Skip to contentSkip to conversation

dt-bindings: iio/adc: qcom,spmi-iadc: Fix 'reg' property in example

PATCH In review Rob Herring 2 participants 2 messages Jan 26, 2022 – Jan 30, 2022

Subsystems (2)
lore
Share & tools
lore

Rob Herring

author maintainer
#0 lore raw

PATCH 1/1 dt-bindings: iio/adc: qcom,spmi-iadc: Fix 'reg' property in example

The QCom SPMI PMIC child nodes are defined to have a single address cell, but the example has an erroneous size cell. Remove it.

---
 Documentation/devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Documentation/devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml +1 -1
index 27e3108661c0..2a94db688830 100644
@@ -51,7 +51,7 @@ examples:
         #size-cells = <0>;
         pmic_iadc: adc@3600 {
             compatible = "qcom,spmi-iadc";
-            reg = <0x3600 0x100>;
+            reg = <0x3600>;
             interrupts = <0x0 0x36 0x0 IRQ_TYPE_EDGE_RISING>;
             qcom,external-resistor-micro-ohms = <10000>;
             #io-channel-cells  = <1>;
Signature · 1 line
2.32.0
maintainer
#1 lore raw
Quoted 4 lines · On Wed, 26 Jan 2022 17:12:17 -0600 Rob Herring <[email protected]> wrote:

The QCom SPMI PMIC child nodes are defined to have a single address cell, but the example has an erroneous size cell. Remove it.

Probably should have a fixes tag...

Fixes a4e6bf69418c ("dt-bindings:iio:adc:qcom,spmi-iadc: txt to yaml format conversion.")

Given it's been there a long time I'm guessing no great rush on this so I'll queue it up for then next merge window.

Shout if you want to pick it up sooner.

Thanks,

Jonathan

Quoted 17 lines
---
 Documentation/devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Documentation/devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml +1 -1
index 27e3108661c0..2a94db688830 100644
@@ -51,7 +51,7 @@ examples:
         #size-cells = <0>;
         pmic_iadc: adc@3600 {
             compatible = "qcom,spmi-iadc";
-            reg = <0x3600 0x100>;
+            reg = <0x3600>;
             interrupts = <0x0 0x36 0x0 IRQ_TYPE_EDGE_RISING>;
             qcom,external-resistor-micro-ohms = <10000>;
             #io-channel-cells  = <1>;