On Thu, 28 Feb 2019 11:26:28 -0300 Renato Lui Geh <renatogeh@xxxxxxxxx> wrote: > Adds a device tree binding for the ad7780 driver. > > Signed-off-by: Renato Lui Geh <renatogeh@xxxxxxxxx> Looks good to me, but we'll see if Rob picks up on things I've missed. Jonathan > --- > .../bindings/iio/adc/adi,ad7780.txt | 48 +++++++++++++++++++ > 1 file changed, 48 insertions(+) > create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt > > diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt > new file mode 100644 > index 000000000000..440e52555349 > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt > @@ -0,0 +1,48 @@ > +* Analog Devices AD7170/AD7171/AD7780/AD7781 > + > +Data sheets: > + > +- AD7170: > + * https://www.analog.com/media/en/technical-documentation/data-sheets/AD7170.pdf > +- AD7171: > + * https://www.analog.com/media/en/technical-documentation/data-sheets/AD7171.pdf > +- AD7780: > + * https://www.analog.com/media/en/technical-documentation/data-sheets/ad7780.pdf > +- AD7781: > + * https://www.analog.com/media/en/technical-documentation/data-sheets/AD7781.pdf > + > +Required properties: > + > +- compatible: should be one of > + * "adi,ad7170" > + * "adi,ad7171" > + * "adi,ad7780" > + * "adi,ad7781" > +- reg: spi chip select number for the device > +- vref-supply: the regulator supply for the ADC reference voltage > + > +Optional properties: > + > +- powerdown-gpios: must be the device tree identifier of the PDRST pin. If > + specified, it will be asserted during driver probe. As the > + line is active high, it should be marked GPIO_ACTIVE_HIGH. > +- adi,gain-gpios: must be the device tree identifier of the GAIN pin. Only for > + the ad778x chips. If specified, it will be asserted during > + driver probe. As the line is active low, it should be marked > + GPIO_ACTIVE_LOW. > +- adi,filter-gpios: must be the device tree identifier of the FILTER pin. Only > + for the ad778x chips. If specified, it will be asserted > + during driver probe. As the line is active low, it should be > + marked GPIO_ACTIVE_LOW. > + > +Example: > + > +adc@0 { > + compatible = "adi,ad7780"; > + reg = <0>; > + vref-supply = <&vdd_supply> > + > + powerdown-gpios = <&gpio 12 GPIO_ACTIVE_HIGH>; > + adi,gain-gpios = <&gpio 5 GPIO_ACTIVE_LOW>; > + adi,filter-gpios = <&gpio 15 GPIO_ACTIVE_LOW>; > +};
