[PATCH v2 0/2] dt-bindings: iio: adc: add ad7780 yaml and MAINTAINERS entry

[PATCH v2 0/2] dt-bindings: iio: adc: add ad7780 yaml and MAINTAINERS entry

[Renato Lui Geh]

This patchset converts the old ad7780 device-tree binding to
the new YAML format, and adds an entry to MAINTAINERS.

Renato Lui Geh (2):
  dt-bindings: iio: adc: add adi,ad7780.yaml binding
  MAINTAINERS: add entry for ad7780 adc driver

 .../bindings/iio/adc/adi,ad7780.txt           | 48 ----------
 .../bindings/iio/adc/adi,ad7780.yaml          | 87 +++++++++++++++++++
 MAINTAINERS                                   |  9 ++
 3 files changed, 96 insertions(+), 48 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
 create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml

-- 
2.21.0

[PATCH v2 1/2] dt-bindings: iio: adc: add adi,ad7780.yaml binding

[Renato Lui Geh]

This patch adds a YAML binding for the Analog Devices AD7780/1 and
AD7170/1 analog-to-digital converters.

Signed-off-by: Renato Lui Geh <renatogeh@gmail.com>
---
Changes in v2:
 - vref-supply to avdd-supply
 - remove avdd-supply from required list
 - include adc block in an spi block

 .../bindings/iio/adc/adi,ad7780.txt           | 48 ----------
 .../bindings/iio/adc/adi,ad7780.yaml          | 87 +++++++++++++++++++
 2 files changed, 87 insertions(+), 48 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
 create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml

diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
deleted file mode 100644
index 440e52555349..000000000000
--- a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
+++ /dev/null
@@ -1,48 +0,0 @@
-* 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>;
-};
diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
new file mode 100644
index 000000000000..d1109416963c
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
@@ -0,0 +1,87 @@
+# SPDX-License-Identifier: GPL-2.0
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/adc/adi,ad7780.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Analog Devices AD7170/AD7171/AD7780/AD7781 analog to digital converters
+
+maintainers:
+  - Michael Hennerich <michael.hennerich@analog.com>
+
+description: |
+  The ad7780 is a sigma-delta analog to digital converter. This driver provides
+  reading voltage values and status bits from both the ad778x and ad717x series.
+  Its interface also allows writing on the FILTER and GAIN GPIO pins on the
+  ad778x.
+
+  Specifications on the converters can be found at:
+    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
+
+properties:
+  compatible:
+    enum:
+      - adi,ad7170
+      - adi,ad7171
+      - adi,ad7780
+      - adi,ad7781
+
+  reg:
+    maxItems: 1
+
+  avdd-supply:
+    description:
+      The regulator supply for the ADC reference voltage.
+    maxItems: 1
+
+  powerdown-gpios:
+    description:
+      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.
+    maxItems: 1
+
+  adi,gain-gpios:
+    description:
+      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.
+    maxItems: 1
+
+  adi,filter-gpios:
+    description:
+      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.
+    maxItems: 1
+
+required:
+  - compatible
+  - reg
+
+examples:
+  - |
+    #include <dt-bindings/gpio/gpio.h>
+    spi0 {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        adc@0 {
+            compatible = "adi,ad7780";
+            reg = <0>;
+
+            avdd-supply      = <&vdd_supply>;
+            powerdown-gpios  = <&gpio0 12 GPIO_ACTIVE_HIGH>;
+            adi,gain-gpios   = <&gpio1  5 GPIO_ACTIVE_LOW>;
+            adi,filter-gpios = <&gpio2 15 GPIO_ACTIVE_LOW>;
+        };
+    };
-- 
2.21.0

[PATCH v2 2/2] MAINTAINERS: add entry for ad7780 adc driver

[Renato Lui Geh]

This patch adds a MAINTAINERS entry for the AD7780 ADC driver.

Signed-off-by: Renato Lui Geh <renatogeh@gmail.com>
---
 MAINTAINERS | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 54c8e14fae98..d12685c5b09a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -880,6 +880,15 @@ S:	Supported
 F:	drivers/iio/adc/ad7768-1.c
 F:	Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.txt
 
+ANALOG DEVICES INC AD7780 DRIVER
+M:	Michael Hennerich <Michael.Hennerich@analog.com>
+M:	Renato Lui Geh <renatogeh@gmail.com>
+L:	linux-iio@vger.kernel.org
+W:	http://ez.analog.com/community/linux-device-drivers
+S:	Supported
+F:	drivers/iio/adc/ad7780.c
+F:	Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
+
 ANALOG DEVICES INC AD9389B DRIVER
 M:	Hans Verkuil <hans.verkuil@cisco.com>
 L:	linux-media@vger.kernel.org
-- 
2.21.0

[PATCH v2 2/2] MAINTAINERS: add entry for ad7780 adc driver

[Renato Lui Geh]

This patch adds a MAINTAINERS entry for the AD7780 ADC driver.

Signed-off-by: Renato Lui Geh <renatogeh@gmail.com>
---
 MAINTAINERS | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 54c8e14fae98..d12685c5b09a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -880,6 +880,15 @@ S:	Supported
 F:	drivers/iio/adc/ad7768-1.c
 F:	Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.txt
 
+ANALOG DEVICES INC AD7780 DRIVER
+M:	Michael Hennerich <Michael.Hennerich@analog.com>
+M:	Renato Lui Geh <renatogeh@gmail.com>
+L:	linux-iio@vger.kernel.org
+W:	http://ez.analog.com/community/linux-device-drivers
+S:	Supported
+F:	drivers/iio/adc/ad7780.c
+F:	Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
+
 ANALOG DEVICES INC AD9389B DRIVER
 M:	Hans Verkuil <hans.verkuil@cisco.com>
 L:	linux-media@vger.kernel.org
-- 
2.21.0

Re: [PATCH v2 1/2] dt-bindings: iio: adc: add adi,ad7780.yaml binding

[Jonathan Cameron]

On Fri, 24 May 2019 22:26:30 -0300
Renato Lui Geh <renatogeh@gmail.com> wrote:

> This patch adds a YAML binding for the Analog Devices AD7780/1 and
> AD7170/1 analog-to-digital converters.
> 
> Signed-off-by: Renato Lui Geh <renatogeh@gmail.com>
Looks good to me, but I'm still finding my feet with these so will
leave it for a few days for others to have time to comment.

Michael, looking for a quick reply from you to say if you are happy
being explicitly listed as maintainer for this one, or if you'd
rather land it on someone else.  Same applies for patch 2.

Renato, if I seem to have forgotten this in a week or so, feel
free to give me a poke. I've been known to loose patches entirely!

Thanks,

Jonathan
> ---
> Changes in v2:
>  - vref-supply to avdd-supply
>  - remove avdd-supply from required list
>  - include adc block in an spi block
> 
>  .../bindings/iio/adc/adi,ad7780.txt           | 48 ----------
>  .../bindings/iio/adc/adi,ad7780.yaml          | 87 +++++++++++++++++++
>  2 files changed, 87 insertions(+), 48 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
>  create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> 
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> deleted file mode 100644
> index 440e52555349..000000000000
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> +++ /dev/null
> @@ -1,48 +0,0 @@
> -* 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>;
> -};
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> new file mode 100644
> index 000000000000..d1109416963c
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> @@ -0,0 +1,87 @@
> +# SPDX-License-Identifier: GPL-2.0
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/iio/adc/adi,ad7780.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Analog Devices AD7170/AD7171/AD7780/AD7781 analog to digital converters
> +
> +maintainers:
> +  - Michael Hennerich <michael.hennerich@analog.com>
> +
> +description: |
> +  The ad7780 is a sigma-delta analog to digital converter. This driver provides
> +  reading voltage values and status bits from both the ad778x and ad717x series.
> +  Its interface also allows writing on the FILTER and GAIN GPIO pins on the
> +  ad778x.
> +
> +  Specifications on the converters can be found at:
> +    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
> +
> +properties:
> +  compatible:
> +    enum:
> +      - adi,ad7170
> +      - adi,ad7171
> +      - adi,ad7780
> +      - adi,ad7781
> +
> +  reg:
> +    maxItems: 1
> +
> +  avdd-supply:
> +    description:
> +      The regulator supply for the ADC reference voltage.
> +    maxItems: 1
> +
> +  powerdown-gpios:
> +    description:
> +      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.
> +    maxItems: 1
> +
> +  adi,gain-gpios:
> +    description:
> +      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.
> +    maxItems: 1
> +
> +  adi,filter-gpios:
> +    description:
> +      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.
> +    maxItems: 1
> +
> +required:
> +  - compatible
> +  - reg
> +
> +examples:
> +  - |
> +    #include <dt-bindings/gpio/gpio.h>
> +    spi0 {
> +        #address-cells = <1>;
> +        #size-cells = <0>;
> +
> +        adc@0 {
> +            compatible = "adi,ad7780";
> +            reg = <0>;
> +
> +            avdd-supply      = <&vdd_supply>;
> +            powerdown-gpios  = <&gpio0 12 GPIO_ACTIVE_HIGH>;
> +            adi,gain-gpios   = <&gpio1  5 GPIO_ACTIVE_LOW>;
> +            adi,filter-gpios = <&gpio2 15 GPIO_ACTIVE_LOW>;
> +        };
> +    };

Re: [PATCH v2 1/2] dt-bindings: iio: adc: add adi,ad7780.yaml binding

[Jonathan Cameron]

On Fri, 24 May 2019 22:26:30 -0300
Renato Lui Geh <renatogeh@gmail.com> wrote:

> This patch adds a YAML binding for the Analog Devices AD7780/1 and
> AD7170/1 analog-to-digital converters.
> 
> Signed-off-by: Renato Lui Geh <renatogeh@gmail.com>
Looks good to me, but I'm still finding my feet with these so will
leave it for a few days for others to have time to comment.

Michael, looking for a quick reply from you to say if you are happy
being explicitly listed as maintainer for this one, or if you'd
rather land it on someone else.  Same applies for patch 2.

Renato, if I seem to have forgotten this in a week or so, feel
free to give me a poke. I've been known to loose patches entirely!

Thanks,

Jonathan
> ---
> Changes in v2:
>  - vref-supply to avdd-supply
>  - remove avdd-supply from required list
>  - include adc block in an spi block
> 
>  .../bindings/iio/adc/adi,ad7780.txt           | 48 ----------
>  .../bindings/iio/adc/adi,ad7780.yaml          | 87 +++++++++++++++++++
>  2 files changed, 87 insertions(+), 48 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
>  create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> 
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> deleted file mode 100644
> index 440e52555349..000000000000
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> +++ /dev/null
> @@ -1,48 +0,0 @@
> -* 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>;
> -};
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> new file mode 100644
> index 000000000000..d1109416963c
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> @@ -0,0 +1,87 @@
> +# SPDX-License-Identifier: GPL-2.0
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/iio/adc/adi,ad7780.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Analog Devices AD7170/AD7171/AD7780/AD7781 analog to digital converters
> +
> +maintainers:
> +  - Michael Hennerich <michael.hennerich@analog.com>
> +
> +description: |
> +  The ad7780 is a sigma-delta analog to digital converter. This driver provides
> +  reading voltage values and status bits from both the ad778x and ad717x series.
> +  Its interface also allows writing on the FILTER and GAIN GPIO pins on the
> +  ad778x.
> +
> +  Specifications on the converters can be found at:
> +    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
> +
> +properties:
> +  compatible:
> +    enum:
> +      - adi,ad7170
> +      - adi,ad7171
> +      - adi,ad7780
> +      - adi,ad7781
> +
> +  reg:
> +    maxItems: 1
> +
> +  avdd-supply:
> +    description:
> +      The regulator supply for the ADC reference voltage.
> +    maxItems: 1
> +
> +  powerdown-gpios:
> +    description:
> +      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.
> +    maxItems: 1
> +
> +  adi,gain-gpios:
> +    description:
> +      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.
> +    maxItems: 1
> +
> +  adi,filter-gpios:
> +    description:
> +      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.
> +    maxItems: 1
> +
> +required:
> +  - compatible
> +  - reg
> +
> +examples:
> +  - |
> +    #include <dt-bindings/gpio/gpio.h>
> +    spi0 {
> +        #address-cells = <1>;
> +        #size-cells = <0>;
> +
> +        adc@0 {
> +            compatible = "adi,ad7780";
> +            reg = <0>;
> +
> +            avdd-supply      = <&vdd_supply>;
> +            powerdown-gpios  = <&gpio0 12 GPIO_ACTIVE_HIGH>;
> +            adi,gain-gpios   = <&gpio1  5 GPIO_ACTIVE_LOW>;
> +            adi,filter-gpios = <&gpio2 15 GPIO_ACTIVE_LOW>;
> +        };
> +    };

RE: [PATCH v2 1/2] dt-bindings: iio: adc: add adi,ad7780.yaml binding

[Hennerich, Michael]

> -----Original Message-----
> From: Jonathan Cameron [mailto:jic23@kernel.org]
> Sent: Sonntag, 26. Mai 2019 18:39
> To: Renato Lui Geh <renatogeh@gmail.com>
> Cc: lars@metafoo.de; Hennerich, Michael <Michael.Hennerich@analog.com>; knaack.h@gmx.de; pmeerw@pmeerw.net;
> gregkh@linuxfoundation.org; Popa, Stefan Serban <StefanSerban.Popa@analog.com>; Ardelean, Alexandru
> <alexandru.Ardelean@analog.com>; robh+dt@kernel.org; mark.rutland@arm.com; linux-iio@vger.kernel.org;
> devel@driverdev.osuosl.org; linux-kernel@vger.kernel.org; kernel-usp@googlegroups.com; devicetree@vger.kernel.org
> Subject: Re: [PATCH v2 1/2] dt-bindings: iio: adc: add adi,ad7780.yaml binding
> 
> On Fri, 24 May 2019 22:26:30 -0300
> Renato Lui Geh <renatogeh@gmail.com> wrote:
> 
> > This patch adds a YAML binding for the Analog Devices AD7780/1 and
> > AD7170/1 analog-to-digital converters.
> >
> > Signed-off-by: Renato Lui Geh <renatogeh@gmail.com>
> Looks good to me, but I'm still finding my feet with these so will
> leave it for a few days for others to have time to comment.
> 
> Michael, looking for a quick reply from you to say if you are happy
> being explicitly listed as maintainer for this one, or if you'd
> rather land it on someone else.  Same applies for patch 2.

Hi Jonathan,

Listing me as an maintainer is ok.

Acked-by: Michael Hennerich <michael.hennerich@analog.com>


> 
> Renato, if I seem to have forgotten this in a week or so, feel
> free to give me a poke. I've been known to loose patches entirely!
> 
> Thanks,
> 
> Jonathan
> > ---
> > Changes in v2:
> >  - vref-supply to avdd-supply
> >  - remove avdd-supply from required list
> >  - include adc block in an spi block
> >
> >  .../bindings/iio/adc/adi,ad7780.txt           | 48 ----------
> >  .../bindings/iio/adc/adi,ad7780.yaml          | 87 +++++++++++++++++++
> >  2 files changed, 87 insertions(+), 48 deletions(-)
> >  delete mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> >  create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> >
> > diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> > deleted file mode 100644
> > index 440e52555349..000000000000
> > --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> > +++ /dev/null
> > @@ -1,48 +0,0 @@
> > -* 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>;
> > -};
> > diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> > new file mode 100644
> > index 000000000000..d1109416963c
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> > @@ -0,0 +1,87 @@
> > +# SPDX-License-Identifier: GPL-2.0
> > +%YAML 1.2
> > +---
> > +$id: http://devicetree.org/schemas/iio/adc/adi,ad7780.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > +
> > +title: Analog Devices AD7170/AD7171/AD7780/AD7781 analog to digital converters
> > +
> > +maintainers:
> > +  - Michael Hennerich <michael.hennerich@analog.com>
> > +
> > +description: |
> > +  The ad7780 is a sigma-delta analog to digital converter. This driver provides
> > +  reading voltage values and status bits from both the ad778x and ad717x series.
> > +  Its interface also allows writing on the FILTER and GAIN GPIO pins on the
> > +  ad778x.
> > +
> > +  Specifications on the converters can be found at:
> > +    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
> > +
> > +properties:
> > +  compatible:
> > +    enum:
> > +      - adi,ad7170
> > +      - adi,ad7171
> > +      - adi,ad7780
> > +      - adi,ad7781
> > +
> > +  reg:
> > +    maxItems: 1
> > +
> > +  avdd-supply:
> > +    description:
> > +      The regulator supply for the ADC reference voltage.
> > +    maxItems: 1
> > +
> > +  powerdown-gpios:
> > +    description:
> > +      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.
> > +    maxItems: 1
> > +
> > +  adi,gain-gpios:
> > +    description:
> > +      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.
> > +    maxItems: 1
> > +
> > +  adi,filter-gpios:
> > +    description:
> > +      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.
> > +    maxItems: 1
> > +
> > +required:
> > +  - compatible
> > +  - reg
> > +
> > +examples:
> > +  - |
> > +    #include <dt-bindings/gpio/gpio.h>
> > +    spi0 {
> > +        #address-cells = <1>;
> > +        #size-cells = <0>;
> > +
> > +        adc@0 {
> > +            compatible = "adi,ad7780";
> > +            reg = <0>;
> > +
> > +            avdd-supply      = <&vdd_supply>;
> > +            powerdown-gpios  = <&gpio0 12 GPIO_ACTIVE_HIGH>;
> > +            adi,gain-gpios   = <&gpio1  5 GPIO_ACTIVE_LOW>;
> > +            adi,filter-gpios = <&gpio2 15 GPIO_ACTIVE_LOW>;
> > +        };
> > +    };

Re: [PATCH v2 1/2] dt-bindings: iio: adc: add adi,ad7780.yaml binding

[Renato Lui Geh]

On 05/26, Jonathan Cameron wrote:
>On Fri, 24 May 2019 22:26:30 -0300
>Renato Lui Geh <renatogeh@gmail.com> wrote:
>
>> This patch adds a YAML binding for the Analog Devices AD7780/1 and
>> AD7170/1 analog-to-digital converters.
>>
>> Signed-off-by: Renato Lui Geh <renatogeh@gmail.com>
>Looks good to me, but I'm still finding my feet with these so will
>leave it for a few days for others to have time to comment.
>
>Michael, looking for a quick reply from you to say if you are happy
>being explicitly listed as maintainer for this one, or if you'd
>rather land it on someone else.  Same applies for patch 2.
>
>Renato, if I seem to have forgotten this in a week or so, feel
>free to give me a poke. I've been known to loose patches entirely!

Hi Jonathan,

Just here to give you a poke. :)

By the way, in these cases, which would be easier for you? To send you
an email like I'm doing right now on last week's thread; or to resend
the entire patch(set)?

Thanks,
Renato
>
>Thanks,
>
>Jonathan
>> ---
>> Changes in v2:
>>  - vref-supply to avdd-supply
>>  - remove avdd-supply from required list
>>  - include adc block in an spi block
>>
>>  .../bindings/iio/adc/adi,ad7780.txt           | 48 ----------
>>  .../bindings/iio/adc/adi,ad7780.yaml          | 87 +++++++++++++++++++
>>  2 files changed, 87 insertions(+), 48 deletions(-)
>>  delete mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
>>  create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
>>
>> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
>> deleted file mode 100644
>> index 440e52555349..000000000000
>> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
>> +++ /dev/null
>> @@ -1,48 +0,0 @@
>> -* 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>;
>> -};
>> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
>> new file mode 100644
>> index 000000000000..d1109416963c
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
>> @@ -0,0 +1,87 @@
>> +# SPDX-License-Identifier: GPL-2.0
>> +%YAML 1.2
>> +---
>> +$id: http://devicetree.org/schemas/iio/adc/adi,ad7780.yaml#
>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>> +
>> +title: Analog Devices AD7170/AD7171/AD7780/AD7781 analog to digital converters
>> +
>> +maintainers:
>> +  - Michael Hennerich <michael.hennerich@analog.com>
>> +
>> +description: |
>> +  The ad7780 is a sigma-delta analog to digital converter. This driver provides
>> +  reading voltage values and status bits from both the ad778x and ad717x series.
>> +  Its interface also allows writing on the FILTER and GAIN GPIO pins on the
>> +  ad778x.
>> +
>> +  Specifications on the converters can be found at:
>> +    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
>> +
>> +properties:
>> +  compatible:
>> +    enum:
>> +      - adi,ad7170
>> +      - adi,ad7171
>> +      - adi,ad7780
>> +      - adi,ad7781
>> +
>> +  reg:
>> +    maxItems: 1
>> +
>> +  avdd-supply:
>> +    description:
>> +      The regulator supply for the ADC reference voltage.
>> +    maxItems: 1
>> +
>> +  powerdown-gpios:
>> +    description:
>> +      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.
>> +    maxItems: 1
>> +
>> +  adi,gain-gpios:
>> +    description:
>> +      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.
>> +    maxItems: 1
>> +
>> +  adi,filter-gpios:
>> +    description:
>> +      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.
>> +    maxItems: 1
>> +
>> +required:
>> +  - compatible
>> +  - reg
>> +
>> +examples:
>> +  - |
>> +    #include <dt-bindings/gpio/gpio.h>
>> +    spi0 {
>> +        #address-cells = <1>;
>> +        #size-cells = <0>;
>> +
>> +        adc@0 {
>> +            compatible = "adi,ad7780";
>> +            reg = <0>;
>> +
>> +            avdd-supply      = <&vdd_supply>;
>> +            powerdown-gpios  = <&gpio0 12 GPIO_ACTIVE_HIGH>;
>> +            adi,gain-gpios   = <&gpio1  5 GPIO_ACTIVE_LOW>;
>> +            adi,filter-gpios = <&gpio2 15 GPIO_ACTIVE_LOW>;
>> +        };
>> +    };
>

Re: [PATCH v2 1/2] dt-bindings: iio: adc: add adi,ad7780.yaml binding

[Ardelean, Alexandru]

On Wed, 2019-06-05 at 17:35 -0300, Renato Lui Geh wrote:
> [External]
> 
> 
> On 05/26, Jonathan Cameron wrote:
> > On Fri, 24 May 2019 22:26:30 -0300
> > Renato Lui Geh <renatogeh@gmail.com> wrote:
> > 
> > > This patch adds a YAML binding for the Analog Devices AD7780/1 and
> > > AD7170/1 analog-to-digital converters.
> > > 
> > > Signed-off-by: Renato Lui Geh <renatogeh@gmail.com>
> > Looks good to me, but I'm still finding my feet with these so will
> > leave it for a few days for others to have time to comment.
> > 
> > Michael, looking for a quick reply from you to say if you are happy
> > being explicitly listed as maintainer for this one, or if you'd
> > rather land it on someone else.  Same applies for patch 2.
> > 
> > Renato, if I seem to have forgotten this in a week or so, feel
> > free to give me a poke. I've been known to loose patches entirely!
> 
> Hi Jonathan,
> 
> Just here to give you a poke. :)
> 
> By the way, in these cases, which would be easier for you? To send you
> an email like I'm doing right now on last week's thread; or to resend
> the entire patch(set)?
> 

I think in this case, maybe let's wait a bit longer.
Jonathan has not been active recently.

I think a [RESEND] would be a good idea when he gets back/active and misses your patchset.

Thanks
Alex

> Thanks,
> Renato
> > Thanks,
> > 
> > Jonathan
> > > ---
> > > Changes in v2:
> > >  - vref-supply to avdd-supply
> > >  - remove avdd-supply from required list
> > >  - include adc block in an spi block
> > > 
> > >  .../bindings/iio/adc/adi,ad7780.txt           | 48 ----------
> > >  .../bindings/iio/adc/adi,ad7780.yaml          | 87 +++++++++++++++++++
> > >  2 files changed, 87 insertions(+), 48 deletions(-)
> > >  delete mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> > >  create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> > > 
> > > diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> > > b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> > > deleted file mode 100644
> > > index 440e52555349..000000000000
> > > --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> > > +++ /dev/null
> > > @@ -1,48 +0,0 @@
> > > -* 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>;
> > > -};
> > > diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> > > b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> > > new file mode 100644
> > > index 000000000000..d1109416963c
> > > --- /dev/null
> > > +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> > > @@ -0,0 +1,87 @@
> > > +# SPDX-License-Identifier: GPL-2.0
> > > +%YAML 1.2
> > > +---
> > > +$id: http://devicetree.org/schemas/iio/adc/adi,ad7780.yaml#
> > > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > > +
> > > +title: Analog Devices AD7170/AD7171/AD7780/AD7781 analog to digital converters
> > > +
> > > +maintainers:
> > > +  - Michael Hennerich <michael.hennerich@analog.com>
> > > +
> > > +description: |
> > > +  The ad7780 is a sigma-delta analog to digital converter. This driver provides
> > > +  reading voltage values and status bits from both the ad778x and ad717x series.
> > > +  Its interface also allows writing on the FILTER and GAIN GPIO pins on the
> > > +  ad778x.
> > > +
> > > +  Specifications on the converters can be found at:
> > > +    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
> > > +
> > > +properties:
> > > +  compatible:
> > > +    enum:
> > > +      - adi,ad7170
> > > +      - adi,ad7171
> > > +      - adi,ad7780
> > > +      - adi,ad7781
> > > +
> > > +  reg:
> > > +    maxItems: 1
> > > +
> > > +  avdd-supply:
> > > +    description:
> > > +      The regulator supply for the ADC reference voltage.
> > > +    maxItems: 1
> > > +
> > > +  powerdown-gpios:
> > > +    description:
> > > +      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.
> > > +    maxItems: 1
> > > +
> > > +  adi,gain-gpios:
> > > +    description:
> > > +      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.
> > > +    maxItems: 1
> > > +
> > > +  adi,filter-gpios:
> > > +    description:
> > > +      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.
> > > +    maxItems: 1
> > > +
> > > +required:
> > > +  - compatible
> > > +  - reg
> > > +
> > > +examples:
> > > +  - |
> > > +    #include <dt-bindings/gpio/gpio.h>
> > > +    spi0 {
> > > +        #address-cells = <1>;
> > > +        #size-cells = <0>;
> > > +
> > > +        adc@0 {
> > > +            compatible = "adi,ad7780";
> > > +            reg = <0>;
> > > +
> > > +            avdd-supply      = <&vdd_supply>;
> > > +            powerdown-gpios  = <&gpio0 12 GPIO_ACTIVE_HIGH>;
> > > +            adi,gain-gpios   = <&gpio1  5 GPIO_ACTIVE_LOW>;
> > > +            adi,filter-gpios = <&gpio2 15 GPIO_ACTIVE_LOW>;
> > > +        };
> > > +    };

Re: [PATCH v2 1/2] dt-bindings: iio: adc: add adi,ad7780.yaml binding

[Jonathan Cameron]

On Thu, 6 Jun 2019 11:13:52 +0000
"Ardelean, Alexandru" <alexandru.Ardelean@analog.com> wrote:

> On Wed, 2019-06-05 at 17:35 -0300, Renato Lui Geh wrote:
> > [External]
> > 
> > 
> > On 05/26, Jonathan Cameron wrote:  
> > > On Fri, 24 May 2019 22:26:30 -0300
> > > Renato Lui Geh <renatogeh@gmail.com> wrote:
> > >   
> > > > This patch adds a YAML binding for the Analog Devices AD7780/1 and
> > > > AD7170/1 analog-to-digital converters.
> > > > 
> > > > Signed-off-by: Renato Lui Geh <renatogeh@gmail.com>  
> > > Looks good to me, but I'm still finding my feet with these so will
> > > leave it for a few days for others to have time to comment.
> > > 
> > > Michael, looking for a quick reply from you to say if you are happy
> > > being explicitly listed as maintainer for this one, or if you'd
> > > rather land it on someone else.  Same applies for patch 2.
> > > 
> > > Renato, if I seem to have forgotten this in a week or so, feel
> > > free to give me a poke. I've been known to loose patches entirely!  
> > 
> > Hi Jonathan,
> > 
> > Just here to give you a poke. :)
> > 
> > By the way, in these cases, which would be easier for you? To send you
> > an email like I'm doing right now on last week's thread; or to resend
> > the entire patch(set)?
> >   
> 
> I think in this case, maybe let's wait a bit longer.
> Jonathan has not been active recently.
> 
> I think a [RESEND] would be a good idea when he gets back/active and misses your patchset.
> 
Sorry, was away last weekend and haven't caught up since.

I should be fine to pick this up this weekend.

A ping like this is fine rather than a resend.

Thanks,

Jonathan

> Thanks
> Alex
> 
> > Thanks,
> > Renato  
> > > Thanks,
> > > 
> > > Jonathan  
> > > > ---
> > > > Changes in v2:
> > > >  - vref-supply to avdd-supply
> > > >  - remove avdd-supply from required list
> > > >  - include adc block in an spi block
> > > > 
> > > >  .../bindings/iio/adc/adi,ad7780.txt           | 48 ----------
> > > >  .../bindings/iio/adc/adi,ad7780.yaml          | 87 +++++++++++++++++++
> > > >  2 files changed, 87 insertions(+), 48 deletions(-)
> > > >  delete mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> > > >  create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> > > > 
> > > > diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> > > > b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> > > > deleted file mode 100644
> > > > index 440e52555349..000000000000
> > > > --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> > > > +++ /dev/null
> > > > @@ -1,48 +0,0 @@
> > > > -* 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>;
> > > > -};
> > > > diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> > > > b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> > > > new file mode 100644
> > > > index 000000000000..d1109416963c
> > > > --- /dev/null
> > > > +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> > > > @@ -0,0 +1,87 @@
> > > > +# SPDX-License-Identifier: GPL-2.0
> > > > +%YAML 1.2
> > > > +---
> > > > +$id: http://devicetree.org/schemas/iio/adc/adi,ad7780.yaml#
> > > > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > > > +
> > > > +title: Analog Devices AD7170/AD7171/AD7780/AD7781 analog to digital converters
> > > > +
> > > > +maintainers:
> > > > +  - Michael Hennerich <michael.hennerich@analog.com>
> > > > +
> > > > +description: |
> > > > +  The ad7780 is a sigma-delta analog to digital converter. This driver provides
> > > > +  reading voltage values and status bits from both the ad778x and ad717x series.
> > > > +  Its interface also allows writing on the FILTER and GAIN GPIO pins on the
> > > > +  ad778x.
> > > > +
> > > > +  Specifications on the converters can be found at:
> > > > +    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
> > > > +
> > > > +properties:
> > > > +  compatible:
> > > > +    enum:
> > > > +      - adi,ad7170
> > > > +      - adi,ad7171
> > > > +      - adi,ad7780
> > > > +      - adi,ad7781
> > > > +
> > > > +  reg:
> > > > +    maxItems: 1
> > > > +
> > > > +  avdd-supply:
> > > > +    description:
> > > > +      The regulator supply for the ADC reference voltage.
> > > > +    maxItems: 1
> > > > +
> > > > +  powerdown-gpios:
> > > > +    description:
> > > > +      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.
> > > > +    maxItems: 1
> > > > +
> > > > +  adi,gain-gpios:
> > > > +    description:
> > > > +      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.
> > > > +    maxItems: 1
> > > > +
> > > > +  adi,filter-gpios:
> > > > +    description:
> > > > +      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.
> > > > +    maxItems: 1
> > > > +
> > > > +required:
> > > > +  - compatible
> > > > +  - reg
> > > > +
> > > > +examples:
> > > > +  - |
> > > > +    #include <dt-bindings/gpio/gpio.h>
> > > > +    spi0 {
> > > > +        #address-cells = <1>;
> > > > +        #size-cells = <0>;
> > > > +
> > > > +        adc@0 {
> > > > +            compatible = "adi,ad7780";
> > > > +            reg = <0>;
> > > > +
> > > > +            avdd-supply      = <&vdd_supply>;
> > > > +            powerdown-gpios  = <&gpio0 12 GPIO_ACTIVE_HIGH>;
> > > > +            adi,gain-gpios   = <&gpio1  5 GPIO_ACTIVE_LOW>;
> > > > +            adi,filter-gpios = <&gpio2 15 GPIO_ACTIVE_LOW>;
> > > > +        };
> > > > +    };  

Re: [PATCH v2 1/2] dt-bindings: iio: adc: add adi,ad7780.yaml binding

[Jonathan Cameron]

On Thu, 6 Jun 2019 16:52:14 +0100
Jonathan Cameron <jonathan.cameron@huawei.com> wrote:

> On Thu, 6 Jun 2019 11:13:52 +0000
> "Ardelean, Alexandru" <alexandru.Ardelean@analog.com> wrote:
> 
> > On Wed, 2019-06-05 at 17:35 -0300, Renato Lui Geh wrote:  
> > > [External]
> > > 
> > > 
> > > On 05/26, Jonathan Cameron wrote:    
> > > > On Fri, 24 May 2019 22:26:30 -0300
> > > > Renato Lui Geh <renatogeh@gmail.com> wrote:
> > > >     
> > > > > This patch adds a YAML binding for the Analog Devices AD7780/1 and
> > > > > AD7170/1 analog-to-digital converters.
> > > > > 
> > > > > Signed-off-by: Renato Lui Geh <renatogeh@gmail.com>    
> > > > Looks good to me, but I'm still finding my feet with these so will
> > > > leave it for a few days for others to have time to comment.
> > > > 
> > > > Michael, looking for a quick reply from you to say if you are happy
> > > > being explicitly listed as maintainer for this one, or if you'd
> > > > rather land it on someone else.  Same applies for patch 2.
> > > > 
> > > > Renato, if I seem to have forgotten this in a week or so, feel
> > > > free to give me a poke. I've been known to loose patches entirely!    
> > > 
> > > Hi Jonathan,
> > > 
> > > Just here to give you a poke. :)
> > > 
> > > By the way, in these cases, which would be easier for you? To send you
> > > an email like I'm doing right now on last week's thread; or to resend
> > > the entire patch(set)?
> > >     
> > 
> > I think in this case, maybe let's wait a bit longer.
> > Jonathan has not been active recently.
> > 
> > I think a [RESEND] would be a good idea when he gets back/active and misses your patchset.
> >   
> Sorry, was away last weekend and haven't caught up since.
> 
> I should be fine to pick this up this weekend.
> 
> A ping like this is fine rather than a resend.

I've applied this the togreg branch of iio.git and pushed out as testing.

Note I'm not planning a pull request for a week or so, so welcome
any additional comments anyone has when they are able to make them.

Thanks,

Jonathan

> 
> Thanks,
> 
> Jonathan
> 
> > Thanks
> > Alex
> >   
> > > Thanks,
> > > Renato    
> > > > Thanks,
> > > > 
> > > > Jonathan    
> > > > > ---
> > > > > Changes in v2:
> > > > >  - vref-supply to avdd-supply
> > > > >  - remove avdd-supply from required list
> > > > >  - include adc block in an spi block
> > > > > 
> > > > >  .../bindings/iio/adc/adi,ad7780.txt           | 48 ----------
> > > > >  .../bindings/iio/adc/adi,ad7780.yaml          | 87 +++++++++++++++++++
> > > > >  2 files changed, 87 insertions(+), 48 deletions(-)
> > > > >  delete mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> > > > >  create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> > > > > 
> > > > > diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> > > > > b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> > > > > deleted file mode 100644
> > > > > index 440e52555349..000000000000
> > > > > --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> > > > > +++ /dev/null
> > > > > @@ -1,48 +0,0 @@
> > > > > -* 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>;
> > > > > -};
> > > > > diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> > > > > b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> > > > > new file mode 100644
> > > > > index 000000000000..d1109416963c
> > > > > --- /dev/null
> > > > > +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> > > > > @@ -0,0 +1,87 @@
> > > > > +# SPDX-License-Identifier: GPL-2.0
> > > > > +%YAML 1.2
> > > > > +---
> > > > > +$id: http://devicetree.org/schemas/iio/adc/adi,ad7780.yaml#
> > > > > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > > > > +
> > > > > +title: Analog Devices AD7170/AD7171/AD7780/AD7781 analog to digital converters
> > > > > +
> > > > > +maintainers:
> > > > > +  - Michael Hennerich <michael.hennerich@analog.com>
> > > > > +
> > > > > +description: |
> > > > > +  The ad7780 is a sigma-delta analog to digital converter. This driver provides
> > > > > +  reading voltage values and status bits from both the ad778x and ad717x series.
> > > > > +  Its interface also allows writing on the FILTER and GAIN GPIO pins on the
> > > > > +  ad778x.
> > > > > +
> > > > > +  Specifications on the converters can be found at:
> > > > > +    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
> > > > > +
> > > > > +properties:
> > > > > +  compatible:
> > > > > +    enum:
> > > > > +      - adi,ad7170
> > > > > +      - adi,ad7171
> > > > > +      - adi,ad7780
> > > > > +      - adi,ad7781
> > > > > +
> > > > > +  reg:
> > > > > +    maxItems: 1
> > > > > +
> > > > > +  avdd-supply:
> > > > > +    description:
> > > > > +      The regulator supply for the ADC reference voltage.
> > > > > +    maxItems: 1
> > > > > +
> > > > > +  powerdown-gpios:
> > > > > +    description:
> > > > > +      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.
> > > > > +    maxItems: 1
> > > > > +
> > > > > +  adi,gain-gpios:
> > > > > +    description:
> > > > > +      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.
> > > > > +    maxItems: 1
> > > > > +
> > > > > +  adi,filter-gpios:
> > > > > +    description:
> > > > > +      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.
> > > > > +    maxItems: 1
> > > > > +
> > > > > +required:
> > > > > +  - compatible
> > > > > +  - reg
> > > > > +
> > > > > +examples:
> > > > > +  - |
> > > > > +    #include <dt-bindings/gpio/gpio.h>
> > > > > +    spi0 {
> > > > > +        #address-cells = <1>;
> > > > > +        #size-cells = <0>;
> > > > > +
> > > > > +        adc@0 {
> > > > > +            compatible = "adi,ad7780";
> > > > > +            reg = <0>;
> > > > > +
> > > > > +            avdd-supply      = <&vdd_supply>;
> > > > > +            powerdown-gpios  = <&gpio0 12 GPIO_ACTIVE_HIGH>;
> > > > > +            adi,gain-gpios   = <&gpio1  5 GPIO_ACTIVE_LOW>;
> > > > > +            adi,filter-gpios = <&gpio2 15 GPIO_ACTIVE_LOW>;
> > > > > +        };
> > > > > +    };    
> 
> 

Re: [PATCH v2 1/2] dt-bindings: iio: adc: add adi,ad7780.yaml binding

[Jonathan Cameron]

On Thu, 6 Jun 2019 16:52:14 +0100
Jonathan Cameron <jonathan.cameron@huawei.com> wrote:

> On Thu, 6 Jun 2019 11:13:52 +0000
> "Ardelean, Alexandru" <alexandru.Ardelean@analog.com> wrote:
> 
> > On Wed, 2019-06-05 at 17:35 -0300, Renato Lui Geh wrote:  
> > > [External]
> > > 
> > > 
> > > On 05/26, Jonathan Cameron wrote:    
> > > > On Fri, 24 May 2019 22:26:30 -0300
> > > > Renato Lui Geh <renatogeh@gmail.com> wrote:
> > > >     
> > > > > This patch adds a YAML binding for the Analog Devices AD7780/1 and
> > > > > AD7170/1 analog-to-digital converters.
> > > > > 
> > > > > Signed-off-by: Renato Lui Geh <renatogeh@gmail.com>    
> > > > Looks good to me, but I'm still finding my feet with these so will
> > > > leave it for a few days for others to have time to comment.
> > > > 
> > > > Michael, looking for a quick reply from you to say if you are happy
> > > > being explicitly listed as maintainer for this one, or if you'd
> > > > rather land it on someone else.  Same applies for patch 2.
> > > > 
> > > > Renato, if I seem to have forgotten this in a week or so, feel
> > > > free to give me a poke. I've been known to loose patches entirely!    
> > > 
> > > Hi Jonathan,
> > > 
> > > Just here to give you a poke. :)
> > > 
> > > By the way, in these cases, which would be easier for you? To send you
> > > an email like I'm doing right now on last week's thread; or to resend
> > > the entire patch(set)?
> > >     
> > 
> > I think in this case, maybe let's wait a bit longer.
> > Jonathan has not been active recently.
> > 
> > I think a [RESEND] would be a good idea when he gets back/active and misses your patchset.
> >   
> Sorry, was away last weekend and haven't caught up since.
> 
> I should be fine to pick this up this weekend.
> 
> A ping like this is fine rather than a resend.

I've applied this the togreg branch of iio.git and pushed out as testing.

Note I'm not planning a pull request for a week or so, so welcome
any additional comments anyone has when they are able to make them.

Thanks,

Jonathan

> 
> Thanks,
> 
> Jonathan
> 
> > Thanks
> > Alex
> >   
> > > Thanks,
> > > Renato    
> > > > Thanks,
> > > > 
> > > > Jonathan    
> > > > > ---
> > > > > Changes in v2:
> > > > >  - vref-supply to avdd-supply
> > > > >  - remove avdd-supply from required list
> > > > >  - include adc block in an spi block
> > > > > 
> > > > >  .../bindings/iio/adc/adi,ad7780.txt           | 48 ----------
> > > > >  .../bindings/iio/adc/adi,ad7780.yaml          | 87 +++++++++++++++++++
> > > > >  2 files changed, 87 insertions(+), 48 deletions(-)
> > > > >  delete mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> > > > >  create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> > > > > 
> > > > > diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> > > > > b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> > > > > deleted file mode 100644
> > > > > index 440e52555349..000000000000
> > > > > --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> > > > > +++ /dev/null
> > > > > @@ -1,48 +0,0 @@
> > > > > -* 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>;
> > > > > -};
> > > > > diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> > > > > b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> > > > > new file mode 100644
> > > > > index 000000000000..d1109416963c
> > > > > --- /dev/null
> > > > > +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> > > > > @@ -0,0 +1,87 @@
> > > > > +# SPDX-License-Identifier: GPL-2.0
> > > > > +%YAML 1.2
> > > > > +---
> > > > > +$id: http://devicetree.org/schemas/iio/adc/adi,ad7780.yaml#
> > > > > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > > > > +
> > > > > +title: Analog Devices AD7170/AD7171/AD7780/AD7781 analog to digital converters
> > > > > +
> > > > > +maintainers:
> > > > > +  - Michael Hennerich <michael.hennerich@analog.com>
> > > > > +
> > > > > +description: |
> > > > > +  The ad7780 is a sigma-delta analog to digital converter. This driver provides
> > > > > +  reading voltage values and status bits from both the ad778x and ad717x series.
> > > > > +  Its interface also allows writing on the FILTER and GAIN GPIO pins on the
> > > > > +  ad778x.
> > > > > +
> > > > > +  Specifications on the converters can be found at:
> > > > > +    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
> > > > > +
> > > > > +properties:
> > > > > +  compatible:
> > > > > +    enum:
> > > > > +      - adi,ad7170
> > > > > +      - adi,ad7171
> > > > > +      - adi,ad7780
> > > > > +      - adi,ad7781
> > > > > +
> > > > > +  reg:
> > > > > +    maxItems: 1
> > > > > +
> > > > > +  avdd-supply:
> > > > > +    description:
> > > > > +      The regulator supply for the ADC reference voltage.
> > > > > +    maxItems: 1
> > > > > +
> > > > > +  powerdown-gpios:
> > > > > +    description:
> > > > > +      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.
> > > > > +    maxItems: 1
> > > > > +
> > > > > +  adi,gain-gpios:
> > > > > +    description:
> > > > > +      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.
> > > > > +    maxItems: 1
> > > > > +
> > > > > +  adi,filter-gpios:
> > > > > +    description:
> > > > > +      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.
> > > > > +    maxItems: 1
> > > > > +
> > > > > +required:
> > > > > +  - compatible
> > > > > +  - reg
> > > > > +
> > > > > +examples:
> > > > > +  - |
> > > > > +    #include <dt-bindings/gpio/gpio.h>
> > > > > +    spi0 {
> > > > > +        #address-cells = <1>;
> > > > > +        #size-cells = <0>;
> > > > > +
> > > > > +        adc@0 {
> > > > > +            compatible = "adi,ad7780";
> > > > > +            reg = <0>;
> > > > > +
> > > > > +            avdd-supply      = <&vdd_supply>;
> > > > > +            powerdown-gpios  = <&gpio0 12 GPIO_ACTIVE_HIGH>;
> > > > > +            adi,gain-gpios   = <&gpio1  5 GPIO_ACTIVE_LOW>;
> > > > > +            adi,filter-gpios = <&gpio2 15 GPIO_ACTIVE_LOW>;
> > > > > +        };
> > > > > +    };    
> 
> 

Re: [PATCH v2 2/2] MAINTAINERS: add entry for ad7780 adc driver

[Jonathan Cameron]

On Fri, 24 May 2019 22:26:55 -0300
Renato Lui Geh <renatogeh@gmail.com> wrote:

> This patch adds a MAINTAINERS entry for the AD7780 ADC driver.
> 
> Signed-off-by: Renato Lui Geh <renatogeh@gmail.com>
Applied to the togreg branch of iio.git and pushed out as testing
to be completely ignored by the autobuilders.

Thanks,

Jonathan

> ---
>  MAINTAINERS | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 54c8e14fae98..d12685c5b09a 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -880,6 +880,15 @@ S:	Supported
>  F:	drivers/iio/adc/ad7768-1.c
>  F:	Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.txt
>  
> +ANALOG DEVICES INC AD7780 DRIVER
> +M:	Michael Hennerich <Michael.Hennerich@analog.com>
> +M:	Renato Lui Geh <renatogeh@gmail.com>
> +L:	linux-iio@vger.kernel.org
> +W:	http://ez.analog.com/community/linux-device-drivers
> +S:	Supported
> +F:	drivers/iio/adc/ad7780.c
> +F:	Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> +
>  ANALOG DEVICES INC AD9389B DRIVER
>  M:	Hans Verkuil <hans.verkuil@cisco.com>
>  L:	linux-media@vger.kernel.org

Re: [PATCH v2 1/2] dt-bindings: iio: adc: add adi,ad7780.yaml binding

[Rob Herring]

On Fri, 24 May 2019 22:26:30 -0300, Renato Lui Geh wrote:
> This patch adds a YAML binding for the Analog Devices AD7780/1 and
> AD7170/1 analog-to-digital converters.
> 
> Signed-off-by: Renato Lui Geh <renatogeh@gmail.com>
> ---
> Changes in v2:
>  - vref-supply to avdd-supply
>  - remove avdd-supply from required list
>  - include adc block in an spi block
> 
>  .../bindings/iio/adc/adi,ad7780.txt           | 48 ----------
>  .../bindings/iio/adc/adi,ad7780.yaml          | 87 +++++++++++++++++++
>  2 files changed, 87 insertions(+), 48 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
>  create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> 

Reviewed-by: Rob Herring <robh@kernel.org>

Re: [PATCH v2 1/2] dt-bindings: iio: adc: add adi,ad7780.yaml binding

[Jonathan Cameron]

On Fri, 14 Jun 2019 13:43:02 -0600
Rob Herring <robh@kernel.org> wrote:

> On Fri, 24 May 2019 22:26:30 -0300, Renato Lui Geh wrote:
> > This patch adds a YAML binding for the Analog Devices AD7780/1 and
> > AD7170/1 analog-to-digital converters.
> > 
> > Signed-off-by: Renato Lui Geh <renatogeh@gmail.com>
> > ---
> > Changes in v2:
> >  - vref-supply to avdd-supply
> >  - remove avdd-supply from required list
> >  - include adc block in an spi block
> > 
> >  .../bindings/iio/adc/adi,ad7780.txt           | 48 ----------
> >  .../bindings/iio/adc/adi,ad7780.yaml          | 87 +++++++++++++++++++
> >  2 files changed, 87 insertions(+), 48 deletions(-)
> >  delete mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt
> >  create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> >   
> 
> Reviewed-by: Rob Herring <robh@kernel.org>

Thanks. As I'd not yet pushed out as togreg (rather than testing)
I've rebased to add your RB.

Thanks,

Jonathan