Re: [PATCH 1/4] dt-bindings: i2c: nomadik: add mobileye,eyeq6h-i2c bindings

[Krzysztof Kozlowski]

On Tue, Oct 08, 2024 at 12:29:40PM +0200, Théo Lebrun wrote:
> After EyeQ5, it is time for Mobileye EyeQ6H to reuse the Nomadik I2C
> controller. Add a specific compatible because its HW integration is
> slightly different from EyeQ5.
> 
> Do NOT add an example as it looks like EyeQ5 from a DT standpoint
> (without the mobileye,olb property).
> 
> Signed-off-by: Théo Lebrun <theo.lebrun@xxxxxxxxxxx>
> ---
>  Documentation/devicetree/bindings/i2c/st,nomadik-i2c.yaml | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/i2c/st,nomadik-i2c.yaml b/Documentation/devicetree/bindings/i2c/st,nomadik-i2c.yaml
> index 44c54b162bb10741ec7aac70d165403c28176eba..72ecb6efa733f7878bd807df277bfc13153bf71e 100644
> --- a/Documentation/devicetree/bindings/i2c/st,nomadik-i2c.yaml
> +++ b/Documentation/devicetree/bindings/i2c/st,nomadik-i2c.yaml
> @@ -22,6 +22,7 @@ select:
>          enum:
>            - st,nomadik-i2c
>            - mobileye,eyeq5-i2c
> +          - mobileye,eyeq6h-i2c
>    required:
>      - compatible
>  
> @@ -38,6 +39,9 @@ properties:
>        - items:
>            - const: mobileye,eyeq5-i2c

This should be enum in such case.

>            - const: arm,primecell
> +      - items:
> +          - const: mobileye,eyeq6h-i2c
> +          - const: arm,primecell

Best regards,
Krzysztof