Re: [PATCH 1/5] dt-bindings: iio: adc: ad7380: add single-ended compatible parts

[Krzysztof Kozlowski]

On 26/07/2024 17:20, Julien Stephan wrote:
> Adding ad7386/7/8 single-ended compatible parts, and the corresponding
> ad7386-4/7-4/8-4 4 channels.
> 
> Signed-off-by: Julien Stephan <jstephan@xxxxxxxxxxxx>
> ---
>  Documentation/devicetree/bindings/iio/adc/adi,ad7380.yaml | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7380.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7380.yaml
> index 899b777017ce..bd19abb867d9 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7380.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7380.yaml
> @@ -15,10 +15,17 @@ description: |
>    * https://www.analog.com/en/products/ad7381.html
>    * https://www.analog.com/en/products/ad7383.html
>    * https://www.analog.com/en/products/ad7384.html
> +  * https://www.analog.com/en/products/ad7386.html
> +  * https://www.analog.com/en/products/ad7387.html
> +  * https://www.analog.com/en/products/ad7388.html
>    * https://www.analog.com/en/products/ad7380-4.html
>    * https://www.analog.com/en/products/ad7381-4.html
>    * https://www.analog.com/en/products/ad7383-4.html
>    * https://www.analog.com/en/products/ad7384-4.html
> +  * https://www.analog.com/en/products/ad7386-4.html
> +  * https://www.analog.com/en/products/ad7387-4.html
> +  * https://www.analog.com/en/products/ad7388-4.html
> +
>  
>  $ref: /schemas/spi/spi-peripheral-props.yaml#
>  
> @@ -29,10 +36,16 @@ properties:
>        - adi,ad7381
>        - adi,ad7383
>        - adi,ad7384
> +      - adi,ad7386
> +      - adi,ad7387
> +      - adi,ad7388
>        - adi,ad7380-4
>        - adi,ad7381-4
>        - adi,ad7383-4
>        - adi,ad7384-4
> +      - adi,ad7386-4

Lists are ordered alphabetically. Do not add new entries to the end of
the lists (like in your commit "add support for ad738x-4 4") because
that is conflict prone.

Best regards,
Krzysztof