Hi Ulrich, On Fri, Nov 17, 2017 at 11:41 AM, Ulrich Hecht <ulrich.hecht+renesas@xxxxxxxxx> wrote: > Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@xxxxxxxxx> Thanks for your patch! With the below fixed: Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > --- a/Documentation/devicetree/bindings/net/can/rcar_canfd.txt > +++ b/Documentation/devicetree/bindings/net/can/rcar_canfd.txt > @@ -24,12 +25,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,r8a77995" compatible: "renesas,r8a77995-canfd" Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds