diff mbox series

[v6,1/3] dt-bindings: iio: dac: AD5766 yaml documentation

Message ID 20210115112105.58652-1-cristian.pop@analog.com (mailing list archive)
State New
Headers show
Series [v6,1/3] dt-bindings: iio: dac: AD5766 yaml documentation | expand

Commit Message

Cristian Pop Jan. 15, 2021, 11:21 a.m. UTC
This adds device tree bindings for the AD5766 DAC.

Signed-off-by: Cristian Pop <cristian.pop@analog.com>
---
Changelog v6:
	- Use microvolt unit
	- Remove unrelevant to the binding comment
 .../bindings/iio/dac/adi,ad5766.yaml          | 63 +++++++++++++++++++
 1 file changed, 63 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/iio/dac/adi,ad5766.yaml

Comments

Rob Herring Jan. 15, 2021, 4:46 p.m. UTC | #1
On Fri, 15 Jan 2021 13:21:03 +0200, Cristian Pop wrote:
> This adds device tree bindings for the AD5766 DAC.
> 
> Signed-off-by: Cristian Pop <cristian.pop@analog.com>
> ---
> Changelog v6:
> 	- Use microvolt unit
> 	- Remove unrelevant to the binding comment
>  .../bindings/iio/dac/adi,ad5766.yaml          | 63 +++++++++++++++++++
>  1 file changed, 63 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/iio/dac/adi,ad5766.yaml
> 

Reviewed-by: Rob Herring <robh@kernel.org>
Jonathan Cameron Jan. 16, 2021, 5:28 p.m. UTC | #2
On Fri, 15 Jan 2021 10:46:05 -0600
Rob Herring <robh@kernel.org> wrote:

> On Fri, 15 Jan 2021 13:21:03 +0200, Cristian Pop wrote:
> > This adds device tree bindings for the AD5766 DAC.
> > 
> > Signed-off-by: Cristian Pop <cristian.pop@analog.com>
> > ---
> > Changelog v6:
> > 	- Use microvolt unit
> > 	- Remove unrelevant to the binding comment
> >  .../bindings/iio/dac/adi,ad5766.yaml          | 63 +++++++++++++++++++
> >  1 file changed, 63 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/iio/dac/adi,ad5766.yaml
> >   
> 
> Reviewed-by: Rob Herring <robh@kernel.org>
Fixed up a line with just white space on it in the example and applied to the
togreg branch of iio.git, pushed out as testing for the autobuilders to poke
at it and see if we missed anything.
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/iio/dac/adi,ad5766.yaml b/Documentation/devicetree/bindings/iio/dac/adi,ad5766.yaml
new file mode 100644
index 000000000000..7fdd2c42441d
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/dac/adi,ad5766.yaml
@@ -0,0 +1,63 @@ 
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+# Copyright 2020 Analog Devices Inc.
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/dac/adi,ad5766.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Analog Devices AD5766 DAC device driver
+
+maintainers:
+  - Cristian Pop <cristian.pop@analog.com>
+
+description: |
+  Bindings for the Analog Devices AD5766 current DAC device. Datasheet can be
+  found here:
+    https://www.analog.com/media/en/technical-documentation/data-sheets/ad5766-5767.pdf
+
+properties:
+  compatible:
+    enum:
+      - adi,ad5766
+      - adi,ad5767
+
+  output-range-microvolts:
+    description: Select converter output range.
+
+  reg:
+    maxItems: 1
+
+  spi-max-frequency:
+    maximum: 1000000
+
+  spi-cpol: true
+
+  reset-gpios:
+    description: GPIO spec for the RESET pin. As the line is active low, it
+      should be marked GPIO_ACTIVE_LOW.
+    maxItems: 1
+
+required:
+  - compatible
+  - output-range-microvolts
+  - reg
+  - spi-max-frequency
+  - spi-cpol
+
+additionalProperties: false
+
+examples:
+  - |
+    spi {
+          #address-cells = <1>;
+          #size-cells = <0>;
+          
+          ad5766@0 {
+              compatible = "adi,ad5766";
+              output-range-microvolts = <(-5000) 5000>;
+              reg = <0>;
+              spi-cpol;
+              spi-max-frequency = <1000000>;
+              reset-gpios = <&gpio 22 0>;
+            };
+      };