| Message ID | 20210724190449.221894-6-cbranchereau@gmail.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | iio/adc: ingenic: add support for the JZ4760(B) Socs to the ingenic sadc driver | expand |
Hi Christophe, Le sam., juil. 24 2021 at 21:04:49 +0200, Christophe Branchereau <cbranchereau@gmail.com> a écrit : > Add both the jz4760 and jz4760b, plus a property to use the internal > divider on the b variant and document it. > > Signed-off-by: Christophe Branchereau <cbranchereau@gmail.com> Looks good. Reviewed-by: Paul Cercueil <paul@crapouillou.net> > --- > .../bindings/iio/adc/ingenic,adc.yaml | 19 > +++++++++++++++++++ > 1 file changed, 19 insertions(+) > > diff --git > a/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml > b/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml > index 433a3fb55a2e..3eb7aa8822c3 100644 > --- a/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml > +++ b/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml > @@ -23,6 +23,8 @@ properties: > enum: > - ingenic,jz4725b-adc > - ingenic,jz4740-adc > + - ingenic,jz4760-adc > + - ingenic,jz4760b-adc > - ingenic,jz4770-adc > > '#io-channel-cells': > @@ -43,6 +45,23 @@ properties: > interrupts: > maxItems: 1 > > + ingenic,use-internal-divider: > + description: > + If present, battery voltage is read from the VBAT_IR pin, > which has an > + internal 1/4 divider. If absent, it is read through the > VBAT_ER pin, > + which does not have such a divider. > + type: boolean > + > +if: > + not: > + properties: > + compatible: > + contains: > + const: ingenic,jz4760b-adc > +then: > + properties: > + ingenic,use-internal-divider: false > + > required: > - compatible > - '#io-channel-cells' > -- > 2.30.2 >
diff --git a/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml b/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml index 433a3fb55a2e..3eb7aa8822c3 100644 --- a/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml +++ b/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml @@ -23,6 +23,8 @@ properties: enum: - ingenic,jz4725b-adc - ingenic,jz4740-adc + - ingenic,jz4760-adc + - ingenic,jz4760b-adc - ingenic,jz4770-adc '#io-channel-cells': @@ -43,6 +45,23 @@ properties: interrupts: maxItems: 1 + ingenic,use-internal-divider: + description: + If present, battery voltage is read from the VBAT_IR pin, which has an + internal 1/4 divider. If absent, it is read through the VBAT_ER pin, + which does not have such a divider. + type: boolean + +if: + not: + properties: + compatible: + contains: + const: ingenic,jz4760b-adc +then: + properties: + ingenic,use-internal-divider: false + required: - compatible - '#io-channel-cells'
Add both the jz4760 and jz4760b, plus a property to use the internal divider on the b variant and document it. Signed-off-by: Christophe Branchereau <cbranchereau@gmail.com> --- .../bindings/iio/adc/ingenic,adc.yaml | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+)