| Message ID | 7125eea4c3386777d2211224c73e38d8f576e4f0.1741268122.git.Jonathan.Santos@analog.com |
|---|---|
| State | New |
| Headers | show |
| Series | iio: adc: ad7768-1: Add features, improvements, and fixes | expand |
On 3/6/25 3:01 PM, Jonathan Santos wrote: > The AD7768-1 provides a buffered common-mode voltage output > on the VCM pin that can be used to bias analog input signals. > > Add regulators property to enable the use of the VCM output, > referenced here as vcm-output, by any other device. > > Acked-by: Conor Dooley <conor.dooley@microchip.com> > Signed-off-by: Jonathan Santos <Jonathan.Santos@analog.com> > --- > v4 Changes: > * replace "vcm_output" property name for "vcm-output". > > v3 Changes: > * VCM is now provided as a regulator within the device, instead of a > custom property. > > v2 Changes: > * New patch in v2. > --- > .../bindings/iio/adc/adi,ad7768-1.yaml | 21 +++++++++++++++++++ > 1 file changed, 21 insertions(+) > > diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.yaml > index e2f9782b5fc8..12358ea9138a 100644 > --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.yaml > +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.yaml > @@ -59,6 +59,19 @@ properties: > in any way, for example if the filter decimation rate changes. > As the line is active low, it should be marked GPIO_ACTIVE_LOW. > > + regulators: > + type: object > + description: > + list of regulators provided by this controller. > + > + properties: > + vcm-output: > + $ref: /schemas/regulator/regulator.yaml# > + type: object > + unevaluatedProperties: false > + > + additionalProperties: false > + > reset-gpios: > maxItems: 1 > > @@ -152,6 +165,14 @@ examples: > reg = <0>; > label = "channel_0"; > }; > + > + regulators { > + vcm_reg: vcm-output { > + regulator-name = "ad7768-1-vcm"; > + regulator-min-microvolt = <900000>; > + regulator-max-microvolt = <2500000>; Why do we have the min and max properties? Aren't these always going to be the same for all chips? It seems unnecessary to have to write that in the devicetree. > + }; > + }; > }; > }; > ...
diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.yaml index e2f9782b5fc8..12358ea9138a 100644 --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.yaml +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.yaml @@ -59,6 +59,19 @@ properties: in any way, for example if the filter decimation rate changes. As the line is active low, it should be marked GPIO_ACTIVE_LOW. + regulators: + type: object + description: + list of regulators provided by this controller. + + properties: + vcm-output: + $ref: /schemas/regulator/regulator.yaml# + type: object + unevaluatedProperties: false + + additionalProperties: false + reset-gpios: maxItems: 1 @@ -152,6 +165,14 @@ examples: reg = <0>; label = "channel_0"; }; + + regulators { + vcm_reg: vcm-output { + regulator-name = "ad7768-1-vcm"; + regulator-min-microvolt = <900000>; + regulator-max-microvolt = <2500000>; + }; + }; }; }; ...