Hi Geert, Thanks for the feedback. > Subject: Re: [PATCH 5/6] clk: renesas: r9a07g043: Add OSTM clock and reset > entries > > Hi Biju, > > On Mon, Apr 25, 2022 at 11:53 AM Biju Das <biju.das.jz@xxxxxxxxxxxxxx> > wrote: > > Add OSTM{0,1,2} clock and reset entries to CPG driver. > > > > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx> > > Thanks for your patch! > > > --- a/drivers/clk/renesas/r9a07g043-cpg.c > > +++ b/drivers/clk/renesas/r9a07g043-cpg.c > > @@ -129,6 +129,12 @@ static struct rzg2l_mod_clk r9a07g043_mod_clks[] = > { > > 0x52c, 0), > > DEF_MOD("dmac_pclk", R9A07G043_DMAC_PCLK, CLK_P1_DIV2, > > 0x52c, 1), > > + DEF_MOD("ostm0", R9A07G043_OSTM0_PCLK, R9A07G043_CLK_P0, > > + 0x534, 0), > > + DEF_MOD("ostm1", R9A07G043_OSTM1_PCLK, R9A07G043_CLK_P0, > > + 0x534, 1), > > + DEF_MOD("ostm2", R9A07G043_OSTM2_PCLK, R9A07G043_CLK_P0, > > + 0x534, 2), > > Do you mind if I change the clock names to "ostm[012_pclk" > while applying, for consistency with r9a07g044-cpg.c. Thanks, Ok for me. Cheers, Biju > > Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> i.e. will queue > in renesas-clk-for-v5.19. > > Gr{oetje,eeting}s, > > Geert > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux- > m68k.org > > 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