RE: [PATCH 0/5] soc: renesas: Add system controller support for RZ/G3E SoC

[John Madieu]

> -----Original Message-----
> From: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
> Sent: Friday, December 13, 2024 5:25 PM
> Subject: Re: [PATCH 0/5] soc: renesas: Add system controller support for
> RZ/G3E SoC
> 
> Hi John,

Hello Geert,

> 
> On Fri, Dec 6, 2024 at 10:26 PM John Madieu
> <john.madieu.xa@xxxxxxxxxxxxxx> wrote:
> > This patch series adds support for the RZ/G3E system controller and
> > extends the existing RZ/V2H(P) system controller to support syscon.
> > The RZ/G3E system controller allows detecting various SoC features
> > like core count, NPU availability, and CA55 PLL configuration.
> >
> > Key features:
> > - Syscon support for both RZ/V2H and RZ/G3E system controllers
> > - Detection of quad/dual core configuration
> > - Detection of Ethos-U55 NPU presence
> > - Validation of CA55 PLL frequency setting
> > - SoC-specific extended identification through callbacks
> 
> Thanks for your series!
> 
> > This patch series depends upon [1] and [2].
> >
> > Tested:
> > - Example of SoC detection:
> > [    0.065608] renesas-rz-sysc 10430000.system-controller: Detected
> Renesas Quad Core RZ/G3E r9a09g047 Rev 0  with Ethos-U55
> > - Example of PLL misconfiguration warning:
> >  [    0.065616] renesas-rz-sysc 10430000.system-controller: CA55 PLL is
> not set to 1.7GHz
> >
> 
> The latter points to an already-applied unrelated series.  I assume you
> meant "[PATCH v2 00/15] Add initial USB support for the Renesas RZ/G3S
> SoC"[3]?

You are right about [3], as it's the series that adds initial support for
the SYSC driver.

> 
> 
> 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