RE: [PATCH 10/10] arm64: dts: renesas: Add HiHope RZ/G2M v3.0 SoC sub board support

[Ayman Chaudhry <ayman.chaudhry.kc@renesas.com>]

Hi Geert,

Thanks for your review!

> -----Original Message-----
> From: Geert Uytterhoeven <geert@linux-m68k.org>
> Sent: Friday, September 4, 2026 4:36 PM
> To: Ayman Chaudhry <ayman.chaudhry.kc@renesas.com>
> Cc: magnus.damm <magnus.damm@gmail.com>; Rob Herring
> <robh@kernel.org>; Krzysztof Kozlowski <krzk+dt@kernel.org>; Conor Dooley
> <conor+dt@kernel.org>; Stephen Boyd <sboyd@kernel.org>; Brian Masney
> <bmasney+clk@redhat.com>; Jerome Brunet <jbrunet+clk@baylibre.com>;
> Ulf Hansson <ulfh@kernel.org>; linux-renesas-soc@vger.kernel.org;
> devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux-
> clk@vger.kernel.org; linux-pm@vger.kernel.org; Chris Paterson
> <Chris.Paterson2@renesas.com>; Biju Das <biju.das.jz@bp.renesas.com>;
> Fabrizio Castro <fabrizio.castro.jz@renesas.com>; Prabhakar Mahadev Lad
> <prabhakar.mahadev-lad.rj@bp.renesas.com>
> Subject: Re: [PATCH 10/10] arm64: dts: renesas: Add HiHope RZ/G2M v3.0
> SoC sub board support
> 
> Hi Ayman,
> 
> On Fri, 21 Aug 2026 at 17:06, Ayman Chaudhry
> <ayman.chaudhry.kc@renesas.com> wrote:
> > The HiHope RZ/G2M sub board sits below the HiHope RZ/G2M v3.0 (a.k.a.
> > R8A774A3) SoC main board.
> >
> > The RZ/G2M v3.0 SoC sub board is similar to the RZ/G2M v1.3 SoC sub
> > board, so there are already DTSI files available in the kernel. These
> > DTSI files will be included once additional support (e.g. CAN, PCIe
> > and ethernet) is added for the RZ/G2M v3.0 SoC.
> >
> > Signed-off-by: Ayman Chaudhry <ayman.chaudhry.kc@renesas.com>
> 
> Thanks for your patch!
> 
> > --- /dev/null
> > +++ b/arch/arm64/boot/dts/renesas/r8a774a3-hihope-rzg2m-ex.dts
> > @@ -0,0 +1,14 @@
> > +// SPDX-License-Identifier: GPL-2.0
> 
> GPL-2.0-only OR BSD-2-Clause?

Agreed, I will update the licensing in v2.

> 
> > +/*
> > + * Device Tree Source for the HiHope RZ/G2M v3.0 SoC sub board
> > + *
> > + * Copyright (C) 2026 Renesas Electronics Corp.
> > + */
> > +
> > +#include "r8a774a3-hihope-rzg2m.dts"
> > +
> > +/ {
> > +       model = "HopeRun HiHope RZ/G2M v3.0 with sub board";
> > +       compatible = "hoperun,hihope-rzg2-ex", "hoperun,hihope-rzg2m",
> > +                    "renesas,r8a774a3";
> 
> Please replace each group of 8 spaces by a TAB.

Apologies for this, I will correct the indentation in v2.

Kind Regards,
Ayman

> 
> > +};
> 
> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-
> m68k.org
> 
> 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