On Fri, Aug 31, 2018 at 10:44:10AM +0200, Geert Uytterhoeven wrote: > Hi Fabrizio, > > On Thu, Aug 23, 2018 at 4:22 PM Fabrizio Castro > <fabrizio.castro@xxxxxxxxxxxxxx> wrote: > > From: Chris Paterson <chris.paterson2@xxxxxxxxxxx> > > > > Add the device nodes for both RZ/G2M CAN channels. > > > > Signed-off-by: Chris Paterson <chris.paterson2@xxxxxxxxxxx> > > Reviewed-by: Biju Das <biju.das@xxxxxxxxxxxxxx> > > --- > > > > This patch depends on: > > https://lkml.org/lkml/2018/8/23/1049 > > https://www.mail-archive.com/linux-renesas-soc@xxxxxxxxxxxxxxx/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_WpX0bAOxPBX76SKvyXCQ@xxxxxxxxxxxxxx/ Thanks Geert, I was ok with the compat string but it was unwise to pick up this patch before that discussion had concluded. I have dropped this patch accordingly. Fabrizio, please post a v2 once the binding is agreed on.
