On 12/04/2018 11:56 PM, Rob Herring wrote: > On Sun, Nov 18, 2018 at 06:32:00PM +0100, Marek Vasut wrote: >> Document the support for rcar_canfd on R8A77965 SoC devices. >> >> Signed-off-by: Marek Vasut <marek.vasut+renesas@xxxxxxxxx> >> Cc: Eugeniu Rosca <erosca@xxxxxxxxxxxxxx> >> Cc: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> >> Cc: Marc Kleine-Budde <mkl@xxxxxxxxxxxxxx> >> Cc: Rob Herring <robh@xxxxxxxxxx> >> Cc: Simon Horman <horms+renesas@xxxxxxxxxxxx> >> Cc: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> >> Cc: linux-renesas-soc@xxxxxxxxxxxxxxx >> --- >> .../devicetree/bindings/net/can/rcar_canfd.txt | 13 +++++++------ >> 1 file changed, 7 insertions(+), 6 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/net/can/rcar_canfd.txt b/Documentation/devicetree/bindings/net/can/rcar_canfd.txt >> index ac71daa46195..4720e916fbdd 100644 >> --- a/Documentation/devicetree/bindings/net/can/rcar_canfd.txt >> +++ b/Documentation/devicetree/bindings/net/can/rcar_canfd.txt >> @@ -6,6 +6,7 @@ Required properties: >> - "renesas,rcar-gen3-canfd" for R-Car Gen3 compatible controller. >> - "renesas,r8a7795-canfd" for R8A7795 (R-Car H3) compatible controller. >> - "renesas,r8a7796-canfd" for R8A7796 (R-Car M3-W) compatible controller. >> + - "renesas,r8a77965-canfd" for R8A77965 (R-Car M3-N) compatible controller. >> - "renesas,r8a77970-canfd" for R8A77970 (R-Car V3M) compatible controller. >> - "renesas,r8a77980-canfd" for R8A77980 (R-Car V3H) compatible controller. >> >> @@ -26,12 +27,12 @@ The name of the child nodes are "channel0" and "channel1" respectively. Each >> child node supports the "status" property only, which is used to >> enable/disable the respective channel. >> >> -Required properties for "renesas,r8a7795-canfd" and "renesas,r8a7796-canfd" >> -compatible: >> -In R8A7795 and R8A7796 SoCs, canfd clock is a div6 clock and can be used by both >> -CAN and CAN FD controller at the same time. It needs to be scaled to maximum >> -frequency if any of these controllers use it. This is done using the below >> -properties: >> +Required properties for "renesas,r8a7795-canfd", "renesas,r8a7796-canfd" and >> +"renesas,r8a77965-canfd" compatible: >> +In R8A7795, R8A7796 and R8A77965 SoCs, canfd clock is a div6 clock and can > > Do we have to list the SoCs twice so that the paragraph has to be > reformatted every time? Just the compatibles above should be enough. We don't , but that's about it for the SoCs that need this. >> +be used by both CAN and CAN FD controller at the same time. It needs to be >> +scaled to maximum frequency if any of these controllers use it. This is done >> +using the below properties: >> >> - assigned-clocks: phandle of canfd clock. >> - assigned-clock-rates: maximum frequency of this clock. >> -- >> 2.18.0 >> -- Best regards, Marek Vasut