Re: [PATCH v4 9/9] staging: iio: ad7780: add device tree binding

From: Jonathan Cameron
Date: Sat Mar 02 2019 - 14:13:26 EST


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>;
> +};