RE: [PATCH v2 1/3] dt-bindings: iio: dac: Add binding for AD5706R

[Torreno, Alexis Czezar]

> Subject: [PATCH v2 1/3] dt-bindings: iio: dac: Add binding for AD5706R
> 
> Add device tree binding documentation for the Analog Devices AD5706R 4-
> channel 16-bit current output digital-to-analog converter.
> 
> Signed-off-by: Alexis Czezar Torreno <alexisczezar.torreno@xxxxxxxxxx>
> 
> ---
> Changes since v1:
>   - Removed clocks, clock-names, pwms, pwm-names, gpio properties
>   - Simplified example to use plain SPI bus
> ---

...

> +description: |
> +  The AD5706R is a 16-bit, 4-channel current output digital-to-analog
> +  converter with SPI interface.
> +
> +  Datasheet:
> +    https://www.analog.com/en/products/ad5706r.html
> +
> +properties:
> +  compatible:
> +    enum:
> +      - adi,ad5706r
> +
> +  reg:
> +    maxItems: 1
> +
> +  spi-max-frequency:
> +    maximum: 50000000
> +
> +required:
> +  - compatible
> +  - reg
> +
> +additionalProperties: false

Hi All,

Apologies, feel free to skip reviewing this for now. 

This file only trimmed the original yaml and does not contain the feedbacks from
Version 1.

I'll be more careful next time, will apply it on v3.

Did not apply the ff feedback:
- vcc (lacking)
- allOf + ref: (lacking)
- change additionalprop -> unevaluatedprop

> +
> +examples:
> +  - |
> +    spi {
> +        #address-cells = <1>;
> +        #size-cells = <0>;
> +
> +        dac@0 {
> +            compatible = "adi,ad5706r";
> +            reg = <0>;
> +            spi-max-frequency = <50000000>;
> +        };
> +    };
> +...
> 
> --
> 2.34.1