Hi Marek, Wolfram, On Mon, Nov 19, 2018 at 12:46 AM Marek Vasut <marek.vasut@xxxxxxxxx> wrote: > On 11/19/2018 12:14 AM, Wolfram Sang wrote: > > On Sun, Nov 18, 2018 at 06:30:56PM +0100, Marek Vasut wrote: > >> Document the support for rcar_can on R8A77990 SoC devices. > >> Add R8A77990 to the list of SoCs which require the "assigned-clocks" > >> and "assigned-clock-rates" properties. > >> > >> Signed-off-by: Marek Vasut <marek.vasut+renesas@xxxxxxxxx> > >> --- a/Documentation/devicetree/bindings/net/can/rcar_can.txt > >> +++ b/Documentation/devicetree/bindings/net/can/rcar_can.txt > >> @@ -16,6 +16,7 @@ Required properties: > >> "renesas,can-r8a7795" if CAN controller is a part of R8A7795 SoC. > >> "renesas,can-r8a7796" if CAN controller is a part of R8A7796 SoC. > >> "renesas,can-r8a77965" if CAN controller is a part of R8A77965 SoC. > > > > I was wondering why you didn't have a patch adding support for R8A77965. > > I see it is already added here but can't find a tree where this line > > exists (no linus/master or renesas-drivers/master). Did you maybe forgot > > to send out a patch? > > Try and grep for 'renesas,rcar-gen3-can' , the drivers are in > drivers/net/can/rcar/ . They both match on the generic compatible. Marek: I think you misunderstood Wolfram's question. He asked about the DT bindings, not about the driver. Wolfram: Eugenu's commit 4f145f14f6b98b5a ("dt-bindings: can: rcar_can: document r8a77965 support") is in next as of next-20181115 . However, it's not part of linux-can-next. Seems to be destined for v4.20 as a fix. 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