Re: [PATCH 21/24] dt-bindings:iio:cdc:adi,ad7150 binding doc

[Lars-Peter Clausen <lars@xxxxxxxxxx>]

On 2/7/21 4:46 PM, Jonathan Cameron wrote:
> +required:
> +  - compatible
> +  - reg

Is vdd-supply really optional the way it is implemented in the driver?

> +
> +additionalProperties: false
> +
> +examples:
> +  - |
> +    i2c {
> +        #address-cells = <1>;
> +        #size-cells = <0>;
> +
> +        cdc@48 {
> +            compatible = "adi,ad7150";
> +            reg = <0x48>;
> +            interrupts = <25 2>, <26 2>;

I wonder if we should use the symbolic constants for the IRQ type to 
make the example more clear. E.g.

interrupts = <25 IRQ_TYPE_EDGE_FALLING>, ...

> +            interrupt-parent = <&gpio>;
> +        };
> +    };
> +...