Hi Niklas, On Mon, May 27, 2024 at 3:16 PM Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> wrote: > Add the CSI40 and CSI41 module clocks, which are used by the CSI-2 > interfaces on the Renesas R-Car V4M (R8A779H0) SoC. > > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> Thanks for your patch! > --- a/drivers/clk/renesas/r8a779h0-cpg-mssr.c > +++ b/drivers/clk/renesas/r8a779h0-cpg-mssr.c > @@ -176,6 +176,8 @@ static const struct mssr_mod_clk r8a779h0_mod_clks[] = { > DEF_MOD("avb0:rgmii0", 211, R8A779H0_CLK_S0D8_HSC), > DEF_MOD("avb1:rgmii1", 212, R8A779H0_CLK_S0D8_HSC), > DEF_MOD("avb2:rgmii2", 213, R8A779H0_CLK_S0D8_HSC), > + DEF_MOD("csi40", 331, R8A779H0_CLK_CSI), > + DEF_MOD("csi41", 400, R8A779H0_CLK_CSI), According to the documentation for the Module Stop Control Registers 3 and 4, these are called "csitop[01]". > DEF_MOD("hscif0", 514, R8A779H0_CLK_SASYNCPERD1), > DEF_MOD("hscif1", 515, R8A779H0_CLK_SASYNCPERD1), > DEF_MOD("hscif2", 516, R8A779H0_CLK_SASYNCPERD1), 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