Hi Niklas, On Wed, Jan 15, 2025 at 6:59 PM Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> wrote: > Add the VSPX modules clock for Renesas R-Car V4M. > > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> Thanks for your patch! Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> i.e. will queue in renesas-clk for v6.15. > --- a/drivers/clk/renesas/r8a779h0-cpg-mssr.c > +++ b/drivers/clk/renesas/r8a779h0-cpg-mssr.c > @@ -239,6 +239,7 @@ static const struct mssr_mod_clk r8a779h0_mod_clks[] __initconst = { > DEF_MOD("pfc1", 916, R8A779H0_CLK_CP), > DEF_MOD("pfc2", 917, R8A779H0_CLK_CP), > DEF_MOD("tsc2:tsc1", 919, R8A779H0_CLK_CL16M), > + DEF_MOD("vspx0", 1028, R8A779H0_CLK_S0D4_VIO), Any specific reason you are not using S0D1, like on R-Car V4H? I can update this while applying. > DEF_MOD("fcpvx0", 1100, R8A779H0_CLK_S0D4_VIO), > DEF_MOD("ssiu", 2926, R8A779H0_CLK_S0D6_PER), > DEF_MOD("ssi", 2927, R8A779H0_CLK_S0D6_PER), 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