Hi Simon

Thank you for your patch

> Add fallback compatibility strings for R-Car Gen2 and Gen3.
> This is in keeping with the fallback scheme being adopted wherever
> appropriate for drivers for Renesas SoCs.
> 
> Also add SoC names.
> 
> Signed-off-by: Simon Horman <horms+rene...@verge.net.au>
> ---
(snip)
>  Required properties:
>    - compatible: Must contain one of the following:
> -     - "renesas,usbhs-r8a7790"
> -     - "renesas,usbhs-r8a7791"
> -     - "renesas,usbhs-r8a7794"
> -     - "renesas,usbhs-r8a7795"
> +
> +     - "renesas,usbhs-r8a7790" for r8a7790 (R-Car H2) compatible device
> +     - "renesas,usbhs-r8a7791" for r8a7791 (R-Car M2-W) compatible device
> +     - "renesas,usbhs-r8a7794" for r8a7794 (R-Car E2) compatible device
> +     - "renesas,usbhs-r8a7795" for r8a7795 (R-Car H3) compatible device
> +     - "renesas,rcar-gen2-usbhs" for R-Car Gen2 compatibile device
> +     - "renesas,rcar-gen3-usbhs" for R-Car Gen3 compatibile device
> +
> +     When compatible with the generic version, nodes must list the
> +     SoC-specific version corresponding to the platform first followed
> +     by the generic version.

I think these can be separated ?

 1. document update for "renesas,usbhs-r8a77xx"
 2. add new "rcar-genX" (this patch)

>  Example:
>       usbhs: usb@e6590000 {
> -             compatible = "renesas,usbhs-r8a7790";
> +             compatible = "renesas,usbhs-r8a7790", "renesas,rcar-usbhs";

I think you want

 -              compatible = "renesas,usbhs-r8a7790", "renesas,rcar-usbhs";
 +              compatible = "renesas,usbhs-r8a7790", "renesas,rcar-gen2-usbhs";
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to