On Wed, Sep 29, 2021 at 12:46:07PM +0200, Florian Boor wrote: > New binding documentation for AD799x series of I²C ADC ICs. > > Signed-off-by: Florian Boor <florian.boor@xxxxxxxxxxxxxxxxx> > --- > Changes in v5: > - Correct errors in documentation found by > 'make DT_CHECKER_FLAGS=-m dt_binding_check' > + Reduce title length > + Move information to description > + Add I²C bits to example > > .../bindings/iio/adc/adi,ad799x.yaml | 73 +++++++++++++++++++ > 1 file changed, 73 insertions(+) > create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad799x.yaml > > diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad799x.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad799x.yaml > new file mode 100644 > index 000000000000..cdba1ea91f42 > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad799x.yaml > @@ -0,0 +1,73 @@ > +# SPDX-License-Identifier: GPL-2.0 > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/iio/adc/adi,ad799x.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Analog Devices AD799x analog to digital converters > + > +maintainers: > + - Michael Hennerich, Analog Devices Inc. <Michael.Hennerich@xxxxxxxxxx> I think the ',' here will throw off email parsing. Drop the Analog Devices. > + > +description: | > + Support for Analog Devices AD7991, AD7992, AD7993, AD7994, AD7995, AD7997, AD7998, > + AD7999 and similar analog to digital converters. > + Specifications on the converters can be found at: > + AD7991, AD7995, AD7999: > + https://www.analog.com/media/en/technical-documentation/data-sheets/AD7991_7995_7999.pdf > + AD7992: > + https://www.analog.com/media/en/technical-documentation/data-sheets/AD7992.pdf > + AD7993, AD7994: > + https://www.analog.com/media/en/technical-documentation/data-sheets/AD7993_7994.pdf > + AD7997, AD7998: > + https://www.analog.com/media/en/technical-documentation/data-sheets/AD7997_7998.pdf > + > +properties: > + compatible: > + enum: > + - adi,ad7991 > + - adi,ad7992 > + - adi,ad7993 > + - adi,ad7994 > + - adi,ad7995 > + - adi,ad7997 > + - adi,ad7998 > + - adi,ad7999 > + > + reg: > + maxItems: 1 > + > + interrupts: > + maxItems: 1 > + > + vcc-supply: > + description: > + ADC power supply > + > + vref-supply: > + description: > + ADC reference voltage supply, optional for AD7991, AD7995 and AD7999 > + > +required: > + - compatible > + - reg > + > +additionalProperties: false > + > +examples: > + - | > + i2c { > + #address-cells = <1>; > + #size-cells = <0>; > + > + adc1: ad7991@28 { adc@28 With that, Reviewed-by: Rob Herring <robh@xxxxxxxxxx> > + reg = <0x28>; > + compatible = "adi,ad7991"; > + interrupts = <13 2>; > + interrupt-parent = <&gpio6>; > + > + vcc-supply = <&vcc_3v3>; > + vref-supply = <&adc_vref>; > + }; > + }; > +... > -- > 2.30.2 > >