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

[Ayman Chaudhry]

Hi Geert,

Thanks for your review!

> -----Original Message-----
> From: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
> Sent: Friday, September 4, 2026 4:29 PM
> To: Ayman Chaudhry <ayman.chaudhry.kc@xxxxxxxxxxx>
> Cc: magnus.damm <magnus.damm@xxxxxxxxx>; Rob Herring
> <robh@xxxxxxxxxx>; Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>; Conor Dooley
> <conor+dt@xxxxxxxxxx>; Stephen Boyd <sboyd@xxxxxxxxxx>; Brian Masney
> <bmasney+clk@xxxxxxxxxx>; Jerome Brunet <jbrunet+clk@xxxxxxxxxxxx>;
> Ulf Hansson <ulfh@xxxxxxxxxx>; linux-renesas-soc@xxxxxxxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; linux-
> clk@xxxxxxxxxxxxxxx; linux-pm@xxxxxxxxxxxxxxx; Chris Paterson
> <Chris.Paterson2@xxxxxxxxxxx>; Biju Das <biju.das.jz@xxxxxxxxxxxxxx>;
> Fabrizio Castro <fabrizio.castro.jz@xxxxxxxxxxx>; Prabhakar Mahadev Lad
> <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx>
> 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@xxxxxxxxxxx> 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@xxxxxxxxxxx>
> 
> > --- /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