dt-bindings: yaml conversions of some simple ADC bindings.

From: Jonathan Cameron <[email protected]>

Drops the deprecated compatibles without the vendor name. Whilst the driver continues to support these for old dt blobs, any dt bindings that are actuallly verified against this document should be fixed to add the vendor name.

Enforces rule that spi-cpol == spi-cpha. Is there a cleaner way to do this?

Added the #io-channel-cells property to allow for consumers.

Signed-off-by Jonathan Cameron <[email protected]>

Cc Oskar Andero <[email protected]>

---
 .../devicetree/bindings/iio/adc/mcp320x.txt   | 57 ------------
 .../bindings/iio/adc/microchip,mcp3201.yaml   | 93 +++++++++++++++++++
 2 files changed, 93 insertions(+), 57 deletions(-)

Quoted 7 lines · Jonathan Cameron

From: Jonathan Cameron <[email protected]>

Drops the deprecated compatibles without the vendor name. Whilst the driver continues to support these for old dt blobs, any dt bindings that are actuallly verified against this document should be fixed to add the vendor name.

Quoted 2 lines

Enforces rule that spi-cpol == spi-cpha. Is there a cleaner way to do this?

dependencies:
spi-cpol: [ spi-cpha ]
spi-cpha: [ spi-cpol ]

With that,

Reviewed-by Rob Herring <[email protected]>

Quoted 10 lines

Added the #io-channel-cells property to allow for consumers.

Signed-off-by Jonathan Cameron <[email protected]>

Cc Oskar Andero <[email protected]>

---
 .../devicetree/bindings/iio/adc/mcp320x.txt   | 57 ------------
 .../bindings/iio/adc/microchip,mcp3201.yaml   | 93 +++++++++++++++++++
 2 files changed, 93 insertions(+), 57 deletions(-)  

From: Jonathan Cameron <[email protected]>

These i2c devices have simple bindings, well described by trivial-device.yaml so rather than convert the binding doc to yaml, let us just add them to trivial devices and drop the old binding document.

Signed-off-by Jonathan Cameron <[email protected]>

Cc Angelo Compagnucci <[email protected]>

---
 .../devicetree/bindings/iio/adc/mcp3422.txt   | 19 -------------------
 .../devicetree/bindings/trivial-devices.yaml  | 16 ++++++++++++++++
 2 files changed, 16 insertions(+), 19 deletions(-)

Quoted 7 lines · Jonathan Cameron

From: Jonathan Cameron <[email protected]>

These i2c devices have simple bindings, well described by trivial-device.yaml so rather than convert the binding doc to yaml, let us just add them to trivial devices and drop the old binding document.

Quoted 6 lines

Signed-off-by Jonathan Cameron <[email protected]>

Cc Angelo Compagnucci <[email protected]>

---
 .../devicetree/bindings/iio/adc/mcp3422.txt   | 19 -------------------
 .../devicetree/bindings/trivial-devices.yaml  | 16 ++++++++++++++++
 2 files changed, 16 insertions(+), 19 deletions(-)  

Reviewed-by Rob Herring <[email protected]>

From: Jonathan Cameron <[email protected]>

Very simple conversion of spi device with reference supply.

Added the #io-channel-cells property to allow for consumers.

Signed-off-by Jonathan Cameron <[email protected]>

Cc Jan Kiszka <[email protected]>

---
 .../bindings/iio/adc/ti,adc108s102.yaml       | 47 +++++++++++++++++++
 .../bindings/iio/adc/ti-adc108s102.txt        | 18 -------
 2 files changed, 47 insertions(+), 18 deletions(-)

Quoted 4 lines · Jonathan Cameron

From: Jonathan Cameron <[email protected]>

Very simple conversion of spi device with reference supply.

Quoted 9 lines

Added the #io-channel-cells property to allow for consumers.

Signed-off-by Jonathan Cameron <[email protected]>

Cc Jan Kiszka <[email protected]>

---
 .../bindings/iio/adc/ti,adc108s102.yaml       | 47 +++++++++++++++++++
 .../bindings/iio/adc/ti-adc108s102.txt        | 18 -------
 2 files changed, 47 insertions(+), 18 deletions(-)

Reviewed-by Rob Herring <[email protected]>

From: Jonathan Cameron <[email protected]>

Very simple binding for this i2c device with a reference supply.

Added the #io-channel-cells property to allow for consumers.

Signed-off-by Jonathan Cameron <[email protected]>

Cc Michael Hennerich <[email protected]>

---
 .../bindings/iio/adc/lltc,ltc2497.yaml        | 44 +++++++++++++++++++
 .../devicetree/bindings/iio/adc/ltc2497.txt   | 13 ------
 2 files changed, 44 insertions(+), 13 deletions(-)

Quoted 4 lines · Jonathan Cameron

From: Jonathan Cameron <[email protected]>

Very simple binding for this i2c device with a reference supply.

Quoted 9 lines

Added the #io-channel-cells property to allow for consumers.

Signed-off-by Jonathan Cameron <[email protected]>

Cc Michael Hennerich <[email protected]>

---
 .../bindings/iio/adc/lltc,ltc2497.yaml        | 44 +++++++++++++++++++
 .../devicetree/bindings/iio/adc/ltc2497.txt   | 13 ------
 2 files changed, 44 insertions(+), 13 deletions(-)

Reviewed-by Rob Herring <[email protected]>

From: Jonathan Cameron <[email protected]>

Simple conversion for this ADC driver. Note that I haven't put limits on the spi-max-sampling-frequency because the adc161s626 doesn't state one clearly defined value.

Added the #io-channel-cells property to allow for consumers.

Signed-off-by Jonathan Cameron <[email protected]>

Cc Matt Ranostay <[email protected]>

---
 .../bindings/iio/adc/ti,adc161s626.yaml       | 51 +++++++++++++++++++
 .../bindings/iio/adc/ti-adc161s626.txt        | 18 -------
 2 files changed, 51 insertions(+), 18 deletions(-)

Documentation/devicetree/bindings/iio/adc/ti,adc161s626.yaml new file +51 -0

new file mode 100644
index 000000000000..3f4f334d6f73
@@ -0,0 +1,51 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/adc/ti,adc161s626.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Texas Instruments ADC141S626 and ADC161S626 ADCs
+
+maintainers:
+  - Matt Ranostay <[email protected]>
+
+description: |
+  Single channel 14/16bit differential ADCs
+
+properties:
+  compatible:
+    enum:
+      - ti,adc141s626
+      - ti,adc161s626
+
+  reg:
+    maxItems: 1
+
+  spi-max-frequency: true
+
+  vdda-supply: true
+
+  "#io-channel-cells":
+    const: 1
+
+required:
+  - compatible
+  - reg
+
+additionalProperties: false
+
+examples:
+  - |
+    spi {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        adc@0 {
+            compatible = "ti,adc161s626";
+            vdda-supply = <&vdda_fixed>;
+            reg = <0>;
+            spi-max-frequency = <4300000>;
+            #io-channel-cells = <1>;
+        };
+    };
+...

Documentation/devicetree/bindings/iio/adc/ti-adc161s626.txt deleted +0 -19

deleted file mode 100644
index 3d25011f0c99..000000000000
@@ -1,18 +0,0 @@
-* Texas Instruments ADC141S626 and ADC161S626 chips
-
-Required properties:
- - compatible: Should be "ti,adc141s626" or "ti,adc161s626"
- - reg: spi chip select number for the device
- - vdda-supply: supply voltage to VDDA pin
-
-Recommended properties:
- - spi-max-frequency: Definition as per
-               Documentation/devicetree/bindings/spi/spi-bus.txt
-
-Example:
-adc@0 {
-       compatible = "ti,adc161s626";
-       vdda-supply = <&vdda_fixed>;
-       reg = <0>;
-       spi-max-frequency = <4300000>;
-};
--

2.28.0

From: Jonathan Cameron <[email protected]>

Simple conversion for this ADC driver. Note that I haven't put limits on the spi-max-sampling-frequency because the adc161s626 doesn't state one clearly defined value.

Added the #io-channel-cells property to allow for consumers.

Signed-off-by Jonathan Cameron <[email protected]>

Cc Matt Ranostay <[email protected]>

---
 .../bindings/iio/adc/ti,adc161s626.yaml       | 51 +++++++++++++++++++
 .../bindings/iio/adc/ti-adc161s626.txt        | 18 -------
 2 files changed, 51 insertions(+), 18 deletions(-)

Quoted 6 lines · Jonathan Cameron

From: Jonathan Cameron <[email protected]>

Simple conversion for this ADC driver. Note that I haven't put limits on the spi-max-sampling-frequency because the adc161s626 doesn't state one clearly defined value.

Quoted 9 lines

Added the #io-channel-cells property to allow for consumers.

Signed-off-by Jonathan Cameron <[email protected]>

Cc Matt Ranostay <[email protected]>

---
 .../bindings/iio/adc/ti,adc161s626.yaml       | 51 +++++++++++++++++++
 .../bindings/iio/adc/ti-adc161s626.txt        | 18 -------
 2 files changed, 51 insertions(+), 18 deletions(-)

Reviewed-by Rob Herring <[email protected]>

From: Jonathan Cameron <[email protected]>

Simple binding conversion of this SPI ADC binding, with reference voltage.

Added the optional property #io-channel-cells to allow for consumers of channels if that makes sense for a given board.

Signed-off-by Jonathan Cameron <[email protected]>

Cc Akinobu Mita <[email protected]>

---
 .../bindings/iio/adc/ti,adc0832.yaml          | 56 +++++++++++++++++++
 .../bindings/iio/adc/ti-adc0832.txt           | 19 -------
 2 files changed, 56 insertions(+), 19 deletions(-)

From: Jonathan Cameron <[email protected]>

Simple conversion.

Added the #io-channel-cells optional property to allow for consumer bindings if appropriate on a given board.

Signed-off-by Jonathan Cameron <[email protected]>

Cc Mårten Lindahl <[email protected]>

---
 .../bindings/iio/adc/ti,adc084s021.yaml       | 58 +++++++++++++++++++
 .../bindings/iio/adc/ti-adc084s021.txt        | 19 ------
 2 files changed, 58 insertions(+), 19 deletions(-)

Quoted 7 lines · Jonathan Cameron

From: Jonathan Cameron <[email protected]>

Simple conversion.

Added the #io-channel-cells optional property to allow for consumer bindings if appropriate on a given board.

Quoted 7 lines

Signed-off-by Jonathan Cameron <[email protected]>

Cc Mårten Lindahl <[email protected]>

---
 .../bindings/iio/adc/ti,adc084s021.yaml       | 58 +++++++++++++++++++
 .../bindings/iio/adc/ti-adc084s021.txt        | 19 ------
 2 files changed, 58 insertions(+), 19 deletions(-)

Reviewed-by Rob Herring <[email protected]>

Quoted 16 lines · On Mon, 17 Aug 2020 20:30:00 -0600 Rob Herring <[email protected]> wrote:

Quoted 14 lines · Jonathan Cameron

From: Jonathan Cameron <[email protected]>

Simple conversion.

Added the #io-channel-cells optional property to allow for consumer bindings if appropriate on a given board.

Signed-off-by Jonathan Cameron <[email protected]>

Cc Mårten Lindahl <[email protected]>

---
 .../bindings/iio/adc/ti,adc084s021.yaml       | 58 +++++++++++++++++++
 .../bindings/iio/adc/ti-adc084s021.txt        | 19 ------
 2 files changed, 58 insertions(+), 19 deletions(-)

Quoted 1 line

Reviewed-by Rob Herring <[email protected]>

Applied

From: Jonathan Cameron <[email protected]>

Simple binding conversion. Only addition was #io-channel-cells to allow for potential consumers of the channels on this device.

Signed-off-by Jonathan Cameron <[email protected]>

Cc Akinobu Mita <[email protected]>

---
 .../bindings/iio/adc/ti,adc12138.yaml         | 85 +++++++++++++++++++
 .../bindings/iio/adc/ti-adc12138.txt          | 37 --------
 2 files changed, 85 insertions(+), 37 deletions(-)

Quoted 5 lines · Jonathan Cameron

From: Jonathan Cameron <[email protected]>

Simple binding conversion. Only addition was #io-channel-cells to allow for potential consumers of the channels on this device.

Quoted 7 lines

Signed-off-by Jonathan Cameron <[email protected]>

Cc Akinobu Mita <[email protected]>

---
 .../bindings/iio/adc/ti,adc12138.yaml         | 85 +++++++++++++++++++
 .../bindings/iio/adc/ti-adc12138.txt          | 37 --------
 2 files changed, 85 insertions(+), 37 deletions(-)

My bot found errors running 'make dt_binding_check' on your patch:

Error: Documentation/devicetree/bindings/iio/adc/ti,adc12138.example.dts:26.34-35 syntax error FATAL ERROR: Unable to parse input tree
make[1]: *** [scripts/Makefile.lib:330: Documentation/devicetree/bindings/iio/adc/ti,adc12138.example.dt.yaml] Error 1 make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:1334: dt_binding_check] Error 2

See https://patchwork.ozlabs.org/patch/1342594

If you already ran 'make dt_binding_check' and didn't see the above error(s), then make sure dt-schema is up to date:

pip3 install git+https://github.com/devicetree-org/dt-schema.git@master --upgrade

Please check and re-submit.

From: Jonathan Cameron <[email protected]>

Simple binding conversion. Only addition was #io-channel-cells to allow for potential consumers of the channels on this device.

Signed-off-by Jonathan Cameron <[email protected]>

Cc Akinobu Mita <[email protected]>

---
 .../bindings/iio/adc/ti,adc12138.yaml         | 85 +++++++++++++++++++
 .../bindings/iio/adc/ti-adc12138.txt          | 37 --------
 2 files changed, 85 insertions(+), 37 deletions(-)

Documentation/devicetree/bindings/iio/adc/ti,adc12138.yaml new file +29 -0

new file mode 100644
index 000000000000..3e8c43aa8ef3
@@ -0,0 +1,85 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/adc/ti,adc12138.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Texas Instruments ADC12138 and similar self-calibrating ADCs
+
+maintainers:
+  - Akinobu Mita <[email protected]>
+
+description: |
+  13 bit ADCs with 1, 2 or 8 inputs and self calibrating circuitry to
+  correct for linearity, zero and full scale errors.
+
+properties:
+  compatible:
+    enum:
+      - ti,adc12130
+      - ti,adc12132
+      - ti,adc12138
+
+  reg:
+    maxItems: 1
+
+  interrupts:
+    maxItems: 1
+    description: End of Conversion (EOC) interrupt
+

Documentation/devicetree/bindings/iio/adc/ti,adc12138.yaml new file +38 -0

+    description: |
+      The number of conversion clock periods for the S/H's acquisition time.
+      For high source impedances, this value can be increased to 18 or 34.
+      For less ADC accuracy and/or slower CCLK frequencies this value may be
+      decreased to 6.  See section 6.0 INPUT SOURCE RESISTANCE in the
+      datasheet for details.
+
+  "#io-channel-cells":
+    const: 1
+
+required:
+  - compatible
+  - reg
+  - interrupts
+  - clocks
+  - vref-p-supply
+
+additionalProperties: false
+
+examples:
+  - |
+    spi {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        adc@0 {
+            compatible = "ti,adc12138";
+            reg = <0>;
+            interrupts = <28 IRQ_TYPE_EDGE_RISING>;
+            interrupt-parent = <&gpio1>;
+            clocks = <&cclk>;
+            vref-p-supply = <&ldo4_reg>;
+            spi-max-frequency = <5000000>;
+            ti,acquisition-time = <6>;
+            #io-channel-cells = <1>;
+        };
+    };
+...

Documentation/devicetree/bindings/iio/adc/ti-adc12138.txt deleted +0 -38

deleted file mode 100644
index 049a1d36f013..000000000000
@@ -1,37 +0,0 @@
-* Texas Instruments' ADC12130/ADC12132/ADC12138
-
-Required properties:
- - compatible: Should be one of
-       * "ti,adc12130"
-       * "ti,adc12132"
-       * "ti,adc12138"
- - reg: SPI chip select number for the device
- - interrupts: Should contain interrupt for EOC (end of conversion)
- - clocks: phandle to conversion clock input
- - spi-max-frequency: Definision as per
-       Documentation/devicetree/bindings/spi/spi-bus.txt
- - vref-p-supply: The regulator supply for positive analog voltage reference
-
-Optional properties:
- - vref-n-supply: The regulator supply for negative analog voltage reference
-       (Note that this must not go below GND or exceed vref-p)
-       If not specified, this is assumed to be analog ground.
- - ti,acquisition-time: The number of conversion clock periods for the S/H's
-       acquisition time.  Should be one of 6, 10, 18, 34.  If not specified,
-       default value of 10 is used.
-       For high source impedances, this value can be increased to 18 or 34.
-       For less ADC accuracy and/or slower CCLK frequencies this value may be
-       decreased to 6.  See section 6.0 INPUT SOURCE RESISTANCE in the
-       datasheet for details.
-
-Example:
-adc@0 {
-       compatible = "ti,adc12138";
-       reg = <0>;
-       interrupts = <28 IRQ_TYPE_EDGE_RISING>;
-       interrupt-parent = <&gpio1>;
-       clocks = <&cclk>;
-       vref-p-supply = <&ldo4_reg>;
-       spi-max-frequency = <5000000>;
-       ti,acquisition-time = <6>;
-};
--

2.28.0

From: Jonathan Cameron <[email protected]>

Simple binding. Only addition to txt version is
as a provider of channels to other devices using the consumer binding.

Signed-off-by Jonathan Cameron <[email protected]>

Cc Angelo Compagnucci <[email protected]>

---
 .../bindings/iio/adc/ti,adc128s052.yaml       | 59 +++++++++++++++++++
 .../bindings/iio/adc/ti-adc128s052.txt        | 25 --------
 2 files changed, 59 insertions(+), 25 deletions(-)

Quoted 6 lines · Jonathan Cameron

From: Jonathan Cameron <[email protected]>

Simple binding. Only addition to txt version is
as a provider of channels to other devices using the consumer binding.

Quoted 7 lines

Signed-off-by Jonathan Cameron <[email protected]>

Cc Angelo Compagnucci <[email protected]>

---
 .../bindings/iio/adc/ti,adc128s052.yaml       | 59 +++++++++++++++++++
 .../bindings/iio/adc/ti-adc128s052.txt        | 25 --------
 2 files changed, 59 insertions(+), 25 deletions(-)

Reviewed-by Rob Herring <[email protected]>

From: Jonathan Cameron <[email protected]>

Simple conversion. This binding already had the
that is necessary, I'm happy to leave it as previously documented.

Signed-off-by Jonathan Cameron <[email protected]>

Cc David Lechner <[email protected]>

---
  .../bindings/iio/adc/ti,ads7950.yaml          | 64 +++++++++++++++++++
  .../bindings/iio/adc/ti-ads7950.txt           | 23 -------
  2 files changed, 64 insertions(+), 23 deletions(-)

Documentation/devicetree/bindings/iio/adc/ti,ads7950.yaml new file +24 -0

new file mode 100644
index 000000000000..a759d9064718
@@ -0,0 +1,64 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/adc/ti,ads7950.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Texas Instruments ADS7950 and similar ADCs
+
+maintainers:
+  - David Lechner <[email protected]>
+
+description: |
+  Family of 4-16 channel, 8-12 bit ADCs with SPI interface.
+
+properties:
+  compatible:
+    enum:
+      - ti,ads7950
+      - ti,ads7951
+      - ti,ads7952
+      - ti,ads7953
+      - ti,ads7954
+      - ti,ads7955
+      - ti,ads7956

Documentation/devicetree/bindings/iio/adc/ti,ads7950.yaml new file +26 -0

+
+  "#io-channel-cells":
+    const: 1
+
+required:
+  - compatible
+  - reg
+  - vref-supply
+  - "#io-channel-cells"
+
+additionalProperties: false
+
+examples:
+  - |
+    spi {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        adc@0 {
+            compatible = "ti,ads7957";
+            reg = <0>;
+            vref-supply = <&refin_supply>;
+            spi-max-frequency = <10000000>;
+            #io-channel-cells = <1>;
+        };
+    };

Quoted 4 lines · Jonathan Cameron

From: Jonathan Cameron <[email protected]>

Simple conversion. This binding already had the
that is necessary, I'm happy to leave it as previously documented.

This commit message doesn't make sense to me. It should probably just say "this converts the device tree bindings to yml" or so.

Quoted 39 lines

Signed-off-by Jonathan Cameron <[email protected]>

Cc David Lechner <[email protected]>

---
  .../bindings/iio/adc/ti,ads7950.yaml          | 64 +++++++++++++++++++
  .../bindings/iio/adc/ti-ads7950.txt           | 23 -------
  2 files changed, 64 insertions(+), 23 deletions(-)

Documentation/devicetree/bindings/iio/adc/ti,ads7950.yaml new file +24 -0

new file mode 100644
index 000000000000..a759d9064718
@@ -0,0 +1,64 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/adc/ti,ads7950.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Texas Instruments ADS7950 and similar ADCs
+
+maintainers:
+  - David Lechner <[email protected]>
+
+description: |
+  Family of 4-16 channel, 8-12 bit ADCs with SPI interface.
+
+properties:
+  compatible:
+    enum:
+      - ti,ads7950
+      - ti,ads7951
+      - ti,ads7952
+      - ti,ads7953
+      - ti,ads7954
+      - ti,ads7955
+      - ti,ads7956

Documentation/devicetree/bindings/iio/adc/ti,ads7950.yaml line 34

Quoted 10 lines

Documentation/devicetree/bindings/iio/adc/ti,ads7950.yaml new file +10 -0

+      - ti,ads7957
+      - ti,ads7958
+      - ti,ads7959
+      - ti,ads7960
+      - ti,ads7961
+
+  reg:
+    maxItems: 1
+
+  spi-max-frequency: true  

According to the datasheet, max frequency is 20MHz, so this can be:

spi-max-frequency:

   maximum: 20000000

Documentation/devicetree/bindings/iio/adc/ti,ads7950.yaml line 37

Quoted 3 lines

Documentation/devicetree/bindings/iio/adc/ti,ads7950.yaml new file +3 -0

+
+  vref-supply:
+    description: Supply the 2.5V or 5V reference voltage  

Should this say "Supplies ..."?

Quoted 26 lines

Documentation/devicetree/bindings/iio/adc/ti,ads7950.yaml new file +26 -0

+
+  "#io-channel-cells":
+    const: 1
+
+required:
+  - compatible
+  - reg
+  - vref-supply
+  - "#io-channel-cells"
+
+additionalProperties: false
+
+examples:
+  - |
+    spi {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        adc@0 {
+            compatible = "ti,ads7957";
+            reg = <0>;
+            vref-supply = <&refin_supply>;
+            spi-max-frequency = <10000000>;
+            #io-channel-cells = <1>;
+        };
+    };  

From: Jonathan Cameron <[email protected]>

Simple binding so easy to convert.

Signed-off-by Jonathan Cameron <[email protected]>

---
 .../bindings/iio/adc/ti,ads8344.yaml          | 51 +++++++++++++++++++
 .../bindings/iio/adc/ti-ads8344.txt           | 19 -------
 2 files changed, 51 insertions(+), 19 deletions(-)

Quoted 4 lines · Jonathan Cameron

From: Jonathan Cameron <[email protected]>

Simple binding so easy to convert.

Quoted 6 lines

Signed-off-by Jonathan Cameron <[email protected]>

---
 .../bindings/iio/adc/ti,ads8344.yaml          | 51 +++++++++++++++++++
 .../bindings/iio/adc/ti-ads8344.txt           | 19 -------
 2 files changed, 51 insertions(+), 19 deletions(-)

Reviewed-by Rob Herring <[email protected]>

From: Phil Reid <[email protected]>

JC: Seems that I messed up applying the original driver patches, and this file never actually made it into the tree.

I have picked up original Ack and Sign-off so as to record the history.

Acked-by Rob Herring <[email protected]>

Signed-off-by Phil Reid <[email protected]>

Signed-off-by Jonathan Cameron <[email protected]>

---
 .../devicetree/bindings/iio/adc/ti-tlc4541.txt  | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

Documentation/devicetree/bindings/iio/adc/ti-tlc4541.txt new file +17 -0

new file mode 100644
index 000000000000..6b2692723a04
@@ -0,0 +1,17 @@
+* Texas Instruments' TLC4541
+
+Required properties:
+ - compatible: Should be one of
+	* "ti,tlc4541"
+	* "ti,tlc3541"
+	- reg: SPI chip select number for the device
+ - vref-supply: The regulator supply for ADC reference voltage
+ - spi-max-frequency: Max SPI frequency to use (<= 200000)
+
+Example:
+adc@0 {
+	compatible = "ti,tlc4541";
+	reg = <0>;
+	vref-supply = <&vdd_supply>;
+	spi-max-frequency = <200000>;
+};

From: Jonathan Cameron <[email protected]>

Simple binding so easy to convert.
Dropped the stated value of maximum spi bus frequency as it does not seem to correspond to the datasheet. The value of 200kHz is the max sampling frequency of the ADC, not the clock frequency of the SPI bus.

Added #io-channel-cells to allow use as a provider of channels to other devices via the consumer binding.

Signed-off-by Jonathan Cameron <[email protected]>

Cc Phil Reid <[email protected]>

---
  .../bindings/iio/adc/ti,tlc4541.yaml          | 52 +++++++++++++++++++
  .../bindings/iio/adc/ti-tlc4541.txt           | 17 ------
  2 files changed, 52 insertions(+), 17 deletions(-)

Documentation/devicetree/bindings/iio/adc/ti,tlc4541.yaml new file +52 -0

new file mode 100644
index 000000000000..6c2539b3d707
@@ -0,0 +1,52 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/adc/ti,tlc4541.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Texas Instruments TLC4541 and similar ADCs
+
+maintainers:
+  - Phil Reid <[email protected]>
+
+description: |
+  14/16bit single channel ADC with SPI interface.
+
+properties:
+  compatible:
+    enum:
+      - ti,tlc3541
+      - ti,tlc4541
+
+  reg:
+    maxItems: 1
+
+  spi-max-frequency: true
+
+  vref-supply: true
+
+  "#io-channel-cells":
+    const: 1
+
+required:
+  - compatible
+  - reg
+  - vref-supply
+
+additionalProperties: false
+
+examples:
+  - |
+    spi {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        adc@0 {
+            compatible = "ti,tlc4541";
+            reg = <0>;
+            vref-supply = <&vdd_supply>;
+            spi-max-frequency = <200000>;
+            #io-channel-cells = <1>;
+        };
+    };
+...

Documentation/devicetree/bindings/iio/adc/ti-tlc4541.txt deleted +0 -17

deleted file mode 100644
index 6b2692723a04..000000000000
@@ -1,17 +0,0 @@
-* Texas Instruments' TLC4541
-
-Required properties:
- - compatible: Should be one of
-	* "ti,tlc4541"
-	* "ti,tlc3541"
-	- reg: SPI chip select number for the device
- - vref-supply: The regulator supply for ADC reference voltage
- - spi-max-frequency: Max SPI frequency to use (<= 200000)
-
-Example:
-adc@0 {
-	compatible = "ti,tlc4541";
-	reg = <0>;
-	vref-supply = <&vdd_supply>;
-	spi-max-frequency = <200000>;
-};

From: Jonathan Cameron <[email protected]>

Simple binding so easy to convert.
Dropped the stated value of maximum spi bus frequency as it does not seem to correspond to the datasheet. The value of 200kHz is the max sampling frequency of the ADC, not the clock frequency of the SPI bus.

Added #io-channel-cells to allow use as a provider of channels to other devices via the consumer binding.

Signed-off-by Jonathan Cameron <[email protected]>

Cc Phil Reid <[email protected]>

---
 .../bindings/iio/adc/ti,tlc4541.yaml          | 52 +++++++++++++++++++
 .../bindings/iio/adc/ti-tlc4541.txt           | 17 ------
 2 files changed, 52 insertions(+), 17 deletions(-)

Quoted 11 lines · Jonathan Cameron

From: Jonathan Cameron <[email protected]>

Simple binding so easy to convert.
Dropped the stated value of maximum spi bus frequency as it does not seem to correspond to the datasheet. The value of 200kHz is the max sampling frequency of the ADC, not the clock frequency of the SPI bus.

Added #io-channel-cells to allow use as a provider of channels to other devices via the consumer binding.

Quoted 7 lines

Signed-off-by Jonathan Cameron <[email protected]>

Cc Phil Reid <[email protected]>

---
 .../bindings/iio/adc/ti,tlc4541.yaml          | 52 +++++++++++++++++++
 .../bindings/iio/adc/ti-tlc4541.txt           | 17 ------
 2 files changed, 52 insertions(+), 17 deletions(-)

Reviewed-by Rob Herring <[email protected]>