| Message ID | 20220119015253.2437352-1-robh@kernel.org |
|---|---|
| State | Accepted, archived |
| Headers | show |
| Series | dt-bindings: i2c: imx: Make each example a separate entry | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/dtbs-check | warning | build log |
| robh/dt-meta-schema | success |
On Tue, Jan 18, 2022 at 07:52:53PM -0600, Rob Herring wrote: > Each independent example should be a separate entry. This allows for > 'interrupts' to have different cell sizes. > > Signed-off-by: Rob Herring <robh@kernel.org> Reviewed-by: Oleksij Rempel <o.rempel@pengutronix.de> Thank you! > --- > Documentation/devicetree/bindings/i2c/i2c-imx.yaml | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/i2c/i2c-imx.yaml b/Documentation/devicetree/bindings/i2c/i2c-imx.yaml > index c167958ae2a9..01720e338b4c 100644 > --- a/Documentation/devicetree/bindings/i2c/i2c-imx.yaml > +++ b/Documentation/devicetree/bindings/i2c/i2c-imx.yaml > @@ -88,9 +88,7 @@ unevaluatedProperties: false > examples: > - | > #include <dt-bindings/clock/imx5-clock.h> > - #include <dt-bindings/clock/vf610-clock.h> > - #include <dt-bindings/gpio/gpio.h> > - #include <dt-bindings/interrupt-controller/arm-gic.h> > + #include <dt-bindings/interrupt-controller/irq.h> > > i2c@83fc4000 { > compatible = "fsl,imx51-i2c", "fsl,imx21-i2c"; > @@ -99,6 +97,9 @@ examples: > clocks = <&clks IMX5_CLK_I2C2_GATE>; > }; > > + - | > + #include <dt-bindings/clock/vf610-clock.h> > + > i2c@40066000 { > compatible = "fsl,vf610-i2c"; > reg = <0x40066000 0x1000>; > -- > 2.32.0 > > >
On Tue, 18 Jan 2022 19:52:53 -0600, Rob Herring wrote: > Each independent example should be a separate entry. This allows for > 'interrupts' to have different cell sizes. > > Signed-off-by: Rob Herring <robh@kernel.org> > --- > Documentation/devicetree/bindings/i2c/i2c-imx.yaml | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > Applied, thanks!
diff --git a/Documentation/devicetree/bindings/i2c/i2c-imx.yaml b/Documentation/devicetree/bindings/i2c/i2c-imx.yaml index c167958ae2a9..01720e338b4c 100644 --- a/Documentation/devicetree/bindings/i2c/i2c-imx.yaml +++ b/Documentation/devicetree/bindings/i2c/i2c-imx.yaml @@ -88,9 +88,7 @@ unevaluatedProperties: false examples: - | #include <dt-bindings/clock/imx5-clock.h> - #include <dt-bindings/clock/vf610-clock.h> - #include <dt-bindings/gpio/gpio.h> - #include <dt-bindings/interrupt-controller/arm-gic.h> + #include <dt-bindings/interrupt-controller/irq.h> i2c@83fc4000 { compatible = "fsl,imx51-i2c", "fsl,imx21-i2c"; @@ -99,6 +97,9 @@ examples: clocks = <&clks IMX5_CLK_I2C2_GATE>; }; + - | + #include <dt-bindings/clock/vf610-clock.h> + i2c@40066000 { compatible = "fsl,vf610-i2c"; reg = <0x40066000 0x1000>;
Each independent example should be a separate entry. This allows for 'interrupts' to have different cell sizes. Signed-off-by: Rob Herring <robh@kernel.org> --- Documentation/devicetree/bindings/i2c/i2c-imx.yaml | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-)