RE: [PATCH 08/10] arm64: dts: renesas: Add initial SoC DTSI for RZ/G2M v3.0

[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:29 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 08/10] arm64: dts: renesas: Add initial SoC DTSI for
> RZ/G2M v3.0
> 
> On Fri, 21 Aug 2026 at 17:06, Ayman Chaudhry
> <ayman.chaudhry.kc@renesas.com> wrote:
> > Add initial Device Tree Source Include (DTSI) file for the Renesas
> > RZ/G2M v3.0 (a.k.a. R8A774A3) SoC with basic support.
> >
> > Signed-off-by: Ayman Chaudhry <ayman.chaudhry.kc@renesas.com>
> 
> > --- /dev/null
> > +++ b/arch/arm64/boot/dts/renesas/r8a774a3.dtsi
> > @@ -0,0 +1,302 @@
> > +// SPDX-License-Identifier: GPL-2.0
> 
> Any change we can get "GPL-2.0-only OR BSD-2-Clause"?

Yes, I will update the licensing in v2.

> 
> > +       soc {
> > +               compatible = "simple-bus";
> > +               bootph-all;
> > +
> > +               #address-cells = <2>;
> > +               #size-cells = <2>;
> > +               ranges;
> > +
> > +               cpg: clock-controller@e6150000 {
> > +                       compatible = "renesas,r8a774a3-cpg-mssr";
> > +                       reg = <0 0xe6150000 0 0x0bb0>;
> 
> No need for odd sizes: 0x1000.

Agreed, I will update the size in v2.

> 
> > +                       clocks = <&extal_clk>, <&extalr_clk>;
> > +                       clock-names = "extal", "extalr";
> > +                       #clock-cells = <2>;
> > +                       #power-domain-cells = <0>;
> > +                       #reset-cells = <1>;
> > +                       bootph-all;
> > +               };
> > +
> > +               rst: reset-controller@e6160000 {
> > +                       compatible = "renesas,r8a774a3-rst";
> > +                       reg = <0 0xe6160000 0 0x018c>;
> 
> 0x200

Agreed, I will update the size in v2.

Kind Regards,
Ayman

> 
> > +                       bootph-all;
> > +               };
> 
> 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