Hello Geert, Thank you for your feedback.
> Subject: Re: [PATCH] arm64: dts: renesas: r8a774a1: Add CAN nodes > > Hi Fabrizio, > > On Thu, Aug 23, 2018 at 4:22 PM Fabrizio Castro > <[email protected]> wrote: > > From: Chris Paterson <[email protected]> > > > > Add the device nodes for both RZ/G2M CAN channels. > > > > Signed-off-by: Chris Paterson <[email protected]> > > Reviewed-by: Biju Das <[email protected]> > > --- > > > > This patch depends on: > > https://lkml.org/lkml/2018/8/23/1049 > > https://www.mail-archive.com/[email protected]/msg30550.html > > > > > > arch/arm64/boot/dts/renesas/r8a774a1.dtsi | 24 ++++++++++++++++++++++++ > > 1 file changed, 24 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi > > b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi > > index 5d0109a..cd204f5 100644 > > --- a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi > > +++ b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi > > @@ -816,6 +816,30 @@ > > status = "disabled"; > > }; > > > > + can0: can@e6c30000 { > > + compatible = "renesas,can-r8a774a1", > > + "renesas,rzg-gen2-can"; > > Please don't introduce "renesas,rzg-gen2-can", cfr. my comments in > https://lore.kernel.org/lkml/camuhmdwpuqfxogh_-qp8q6watu3lj_wpx0baoxpbx76skvy...@mail.gmail.com/ Agreed, will send a v2. Thanks, Fab > > Gr{oetje,eeting}s, > > Geert > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- > [email protected] > > 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 Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
