RE: [PATCH] clk: renesas: r9a08g046: Add IA55_PCLK to critical module clocks

[Biju Das]

Hi Geert,

Thanks for the feedback.

> -----Original Message-----
> From: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
> Sent: 06 May 2026 15:29
> Subject: Re: [PATCH] clk: renesas: r9a08g046: Add IA55_PCLK to critical module clocks
> 
> Hi Biju,
> 
> On Thu, 30 Apr 2026 at 12:08, Biju <biju.das.au@xxxxxxxxx> wrote:
> > From: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
> >
> > Add R9A08G046_IA55_PCLK to the critical module clocks list to prevent
> > the clock from being gated during suspend, as it is required for the
> > interrupt controller (IA55) to function correctly.
> >
> > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
> 
> Thanks for your patch!
> 
> > --- a/drivers/clk/renesas/r9a08g046-cpg.c
> > +++ b/drivers/clk/renesas/r9a08g046-cpg.c
> > @@ -312,6 +312,7 @@ static const struct rzg2l_reset r9a08g046_resets[]
> > = {
> >
> >  static const unsigned int r9a08g046_crit_mod_clks[] __initconst = {
> >         MOD_CLK_BASE + R9A08G046_GIC600_GICCLK,
> > +       MOD_CLK_BASE + R9A08G046_IA55_PCLK,
> >         MOD_CLK_BASE + R9A08G046_IA55_CLK,
> >         MOD_CLK_BASE + R9A08G046_DMAC_ACLK,  };
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> i.e. will queue in renesas-clk for v7.2.
> Or is this critical, and thus a fix for v7.1?

No, it is not critical, as DTSI support for ICU is not there in 7.1.

Cheers,
Biju