diff mbox series

[V2] dt-bindings: iio: adc: maxim,max1118 yaml conversion

Message ID 20200801175850.1140006-1-jic23@kernel.org
State New
Headers show
Series [V2] dt-bindings: iio: adc: maxim,max1118 yaml conversion | expand

Commit Message

Jonathan Cameron Aug. 1, 2020, 5:58 p.m. UTC
From: Jonathan Cameron <Jonathan.Cameron@huawei.com>


Simple device with a simple conversion.  Special handling needed
for the max1118 which is the only supported part that has an external
reference voltage.

Cc: Akinobu Mita <akinobu.mita@gmail.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

---
v2:
* Added additionalProperties: false. Other changes are to make this work.
* Move the vref-supply definition up
* Add an else clause

.../devicetree/bindings/iio/adc/max1118.txt   | 21 -------
 .../bindings/iio/adc/maxim,max1118.yaml       | 62 +++++++++++++++++++
 2 files changed, 62 insertions(+), 21 deletions(-)

-- 
2.27.0

Comments

Rob Herring (Arm) Aug. 3, 2020, 9:58 p.m. UTC | #1
On Sat, 01 Aug 2020 18:58:50 +0100, Jonathan Cameron wrote:
> From: Jonathan Cameron <Jonathan.Cameron@huawei.com>

> 

> Simple device with a simple conversion.  Special handling needed

> for the max1118 which is the only supported part that has an external

> reference voltage.

> 

> Cc: Akinobu Mita <akinobu.mita@gmail.com>

> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

> ---

> v2:

> * Added additionalProperties: false. Other changes are to make this work.

> * Move the vref-supply definition up

> * Add an else clause

> 

> .../devicetree/bindings/iio/adc/max1118.txt   | 21 -------

>  .../bindings/iio/adc/maxim,max1118.yaml       | 62 +++++++++++++++++++

>  2 files changed, 62 insertions(+), 21 deletions(-)

> 


Reviewed-by: Rob Herring <robh@kernel.org>
Jonathan Cameron Aug. 6, 2020, 6:05 p.m. UTC | #2
On Mon, 3 Aug 2020 15:58:26 -0600
Rob Herring <robh@kernel.org> wrote:

> On Sat, 01 Aug 2020 18:58:50 +0100, Jonathan Cameron wrote:

> > From: Jonathan Cameron <Jonathan.Cameron@huawei.com>

> > 

> > Simple device with a simple conversion.  Special handling needed

> > for the max1118 which is the only supported part that has an external

> > reference voltage.

> > 

> > Cc: Akinobu Mita <akinobu.mita@gmail.com>

> > Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

> > ---

> > v2:

> > * Added additionalProperties: false. Other changes are to make this work.

> > * Move the vref-supply definition up

> > * Add an else clause

> > 

> > .../devicetree/bindings/iio/adc/max1118.txt   | 21 -------

> >  .../bindings/iio/adc/maxim,max1118.yaml       | 62 +++++++++++++++++++

> >  2 files changed, 62 insertions(+), 21 deletions(-)

> >   

> 

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

Thanks,  

Applied to the togreg branch off iio.git and pushed out as testing for autobuilders
etc.

Jonathan
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/iio/adc/max1118.txt b/Documentation/devicetree/bindings/iio/adc/max1118.txt
deleted file mode 100644
index cf33d0b15a6d..000000000000
--- a/Documentation/devicetree/bindings/iio/adc/max1118.txt
+++ /dev/null
@@ -1,21 +0,0 @@ 
-* MAX1117/MAX1118/MAX1119 8-bit, dual-channel ADCs
-
-Required properties:
- - compatible: Should be one of
-	* "maxim,max1117"
-	* "maxim,max1118"
-	* "maxim,max1119"
- - reg: spi chip select number for the device
- - (max1118 only) vref-supply: The regulator supply for ADC reference voltage
-
-Recommended properties:
- - spi-max-frequency: Definition as per
-		Documentation/devicetree/bindings/spi/spi-bus.txt
-
-Example:
-adc@0 {
-	compatible = "maxim,max1118";
-	reg = <0>;
-	vref-supply = <&vdd_supply>;
-	spi-max-frequency = <1000000>;
-};
diff --git a/Documentation/devicetree/bindings/iio/adc/maxim,max1118.yaml b/Documentation/devicetree/bindings/iio/adc/maxim,max1118.yaml
new file mode 100644
index 000000000000..e948b3e37b0c
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/adc/maxim,max1118.yaml
@@ -0,0 +1,62 @@ 
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/adc/maxim,max1118.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Maxim MAX1118 and similar ADCs
+
+maintainers:
+  - Akinobu Mita <akinobu.mita@gmail.com>
+
+description: |
+    Dual channel 8bit ADCs.
+
+properties:
+  compatible:
+    enum:
+      - maxim,max1117
+      - maxim,max1118
+      - maxim,max1119
+
+  reg:
+    maxItems: 1
+
+  spi-max-frequency:
+    maximum: 5000000
+
+  vref-supply:
+    description: External reference, needed to establish input scaling
+
+if:
+  properties:
+    compatible:
+      contains:
+        const: maxim,max1118
+then:
+  required:
+    - vref-supply
+else:
+  properties:
+    vref-supply: false
+
+required:
+  - compatible
+  - reg
+
+additionalProperties: false
+
+examples:
+  - |
+    spi {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        adc@0 {
+            compatible = "maxim,max1118";
+            reg = <0>;
+            vref-supply = <&adc_vref>;
+            spi-max-frequency = <1000000>;
+        };
+    };
+...