| Message ID | 20220119015514.2441231-1-robh@kernel.org (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | dt-bindings: hwmon: lm90: Drop Tegra specifics from example | expand |
On Tue, Jan 18, 2022 at 07:55:13PM -0600, Rob Herring wrote: > There's no need to complicate examples with a platform specific macro. > It also complicates example parsing to figure out the number of > interrupt cells in examples (based on bracketing). > > Signed-off-by: Rob Herring <robh@kernel.org> Applied to hwmon-next. Thanks, Guenter > --- > Documentation/devicetree/bindings/hwmon/national,lm90.yaml | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/hwmon/national,lm90.yaml b/Documentation/devicetree/bindings/hwmon/national,lm90.yaml > index 6e1d54ff5d5b..30db92977937 100644 > --- a/Documentation/devicetree/bindings/hwmon/national,lm90.yaml > +++ b/Documentation/devicetree/bindings/hwmon/national,lm90.yaml > @@ -60,7 +60,6 @@ additionalProperties: false > > examples: > - | > - #include <dt-bindings/gpio/tegra-gpio.h> > #include <dt-bindings/interrupt-controller/irq.h> > > i2c { > @@ -71,8 +70,7 @@ examples: > compatible = "onnn,nct1008"; > reg = <0x4c>; > vcc-supply = <&palmas_ldo6_reg>; > - interrupt-parent = <&gpio>; > - interrupts = <TEGRA_GPIO(O, 4) IRQ_TYPE_LEVEL_LOW>; > + interrupts = <4 IRQ_TYPE_LEVEL_LOW>; > #thermal-sensor-cells = <1>; > }; > };
diff --git a/Documentation/devicetree/bindings/hwmon/national,lm90.yaml b/Documentation/devicetree/bindings/hwmon/national,lm90.yaml index 6e1d54ff5d5b..30db92977937 100644 --- a/Documentation/devicetree/bindings/hwmon/national,lm90.yaml +++ b/Documentation/devicetree/bindings/hwmon/national,lm90.yaml @@ -60,7 +60,6 @@ additionalProperties: false examples: - | - #include <dt-bindings/gpio/tegra-gpio.h> #include <dt-bindings/interrupt-controller/irq.h> i2c { @@ -71,8 +70,7 @@ examples: compatible = "onnn,nct1008"; reg = <0x4c>; vcc-supply = <&palmas_ldo6_reg>; - interrupt-parent = <&gpio>; - interrupts = <TEGRA_GPIO(O, 4) IRQ_TYPE_LEVEL_LOW>; + interrupts = <4 IRQ_TYPE_LEVEL_LOW>; #thermal-sensor-cells = <1>; }; };
There's no need to complicate examples with a platform specific macro. It also complicates example parsing to figure out the number of interrupt cells in examples (based on bracketing). Signed-off-by: Rob Herring <robh@kernel.org> --- Documentation/devicetree/bindings/hwmon/national,lm90.yaml | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)