Hi Wolfram, On Wed, Dec 23, 2020 at 6:25 PM Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> wrote: > Signed-off-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> Thanks for your patch! > --- a/drivers/clk/renesas/r8a779a0-cpg-mssr.c > +++ b/drivers/clk/renesas/r8a779a0-cpg-mssr.c > @@ -152,6 +152,13 @@ static const struct mssr_mod_clk r8a779a0_mod_clks[] __initconst = { > DEF_MOD("csi41", 400, R8A779A0_CLK_CSI0), > DEF_MOD("csi42", 401, R8A779A0_CLK_CSI0), > DEF_MOD("csi43", 402, R8A779A0_CLK_CSI0), > + DEF_MOD("i2c0", 518, R8A779A0_CLK_S3D2), > + DEF_MOD("i2c1", 519, R8A779A0_CLK_S3D2), > + DEF_MOD("i2c2", 520, R8A779A0_CLK_S3D2), > + DEF_MOD("i2c3", 521, R8A779A0_CLK_S3D2), > + DEF_MOD("i2c4", 522, R8A779A0_CLK_S3D2), > + DEF_MOD("i2c5", 523, R8A779A0_CLK_S3D2), > + DEF_MOD("i2c6", 524, R8A779A0_CLK_S3D2), The R-Car V3U Series User’s Manual Rev.0.5 says the parent clock is S1D4 for all instances? > DEF_MOD("scif0", 702, R8A779A0_CLK_S1D8), > DEF_MOD("scif1", 703, R8A779A0_CLK_S1D8), > DEF_MOD("scif3", 704, R8A779A0_CLK_S1D8), 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