Hi Claudiu, On Sat, Oct 19, 2024 at 10:47 AM Claudiu <claudiu.beznea@xxxxxxxxx> wrote: > From: Claudiu Beznea <claudiu.beznea.uj@xxxxxxxxxxxxxx> > > The RTC and VBATTB don't share the MSTOP control bit (but only the bus > clock and the reset signal). As the MSTOP control is modeled though power > domains add power domain support for the RTC device available on the > Renesas RZ/G3S SoC. > > Signed-off-by: Claudiu Beznea <claudiu.beznea.uj@xxxxxxxxxxxxxx> > --- > > Changes in v4: > - none; this patch is new Thanks for your patch! Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> i.e. will queue in renesas-clk for v6.13... > --- a/drivers/clk/renesas/r9a08g045-cpg.c > +++ b/drivers/clk/renesas/r9a08g045-cpg.c > @@ -304,6 +304,9 @@ static const struct rzg2l_cpg_pm_domain_init_data r9a08g045_pm_domains[] = { > DEF_PD("vbat", R9A08G045_PD_VBAT, > DEF_REG_CONF(CPG_BUS_MCPU3_MSTOP, BIT(8)), > RZG2L_PD_F_ALWAYS_ON), > + DEF_PD("rtc", R9A08G045_PD_RTC, > + DEF_REG_CONF(CPG_BUS_MCPU3_MSTOP, BIT(7)), > + RZG2L_PD_F_NONE), ... with s/RZG2L_PD_F_NONE/0/ due to "[PATCH v4 2/4] clk: renesas: rzg2l-cpg: Use GENPD_FLAG_* flags instead of local ones". > }; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx 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