| Message ID | 20220106182518.1435497-3-robh@kernel.org (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | dt-bindings: iio/adc: ti,palmas-gpadc: Split interrupt fields in example | expand |
On Thu, 6 Jan 2022 12:25:11 -0600 Rob Herring <robh@kernel.org> wrote: > Best practice for multi-cell property values is to bracket each multi-cell > value. > > Signed-off-by: Rob Herring <robh@kernel.org> Applied to the togreg branch of iio.git. I'm assuming there is no rush on this as it's best practice rather than a bug. Thanks, Jonathan > --- > .../devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml b/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml > index 7b895784e008..57a31356082e 100644 > --- a/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml > +++ b/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml > @@ -74,9 +74,9 @@ examples: > compatible = "ti,twl6035-pmic", "ti,palmas-pmic"; > adc { > compatible = "ti,palmas-gpadc"; > - interrupts = <18 0 > - 16 0 > - 17 0>; > + interrupts = <18 0>, > + <16 0>, > + <17 0>; > #io-channel-cells = <1>; > ti,channel0-current-microamp = <5>; > ti,channel3-current-microamp = <10>;
diff --git a/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml b/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml index 7b895784e008..57a31356082e 100644 --- a/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml +++ b/Documentation/devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml @@ -74,9 +74,9 @@ examples: compatible = "ti,twl6035-pmic", "ti,palmas-pmic"; adc { compatible = "ti,palmas-gpadc"; - interrupts = <18 0 - 16 0 - 17 0>; + interrupts = <18 0>, + <16 0>, + <17 0>; #io-channel-cells = <1>; ti,channel0-current-microamp = <5>; ti,channel3-current-microamp = <10>;
Best practice for multi-cell property values is to bracket each multi-cell value. Signed-off-by: Rob Herring <robh@kernel.org> --- .../devicetree/bindings/iio/adc/ti,palmas-gpadc.yaml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)