Hi Shimoda-san, On Wed, Sep 9, 2020 at 3:13 PM Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx> wrote: > Geert suggested defining multiple registers layout variants using > an enum [1] to support further devices like R-Car V3U. So, use enum > clk_reg_layout instead of a boolean .stbyctrl flag. No behavior change. > > [1] > https://lore.kernel.org/linux-renesas-soc/CAMuHMdVAgN69p9FFnQdO4iHk2CHkeNaVui2Q-FOY6_BFVjQ-Nw@xxxxxxxxxxxxxx/ > > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx> > --- a/drivers/clk/renesas/renesas-cpg-mssr.h > +++ b/drivers/clk/renesas/renesas-cpg-mssr.h > @@ -130,7 +135,7 @@ struct cpg_mssr_info { > unsigned int num_core_clks; > unsigned int last_dt_core_clk; > unsigned int num_total_core_clks; > - bool stbyctrl; > + enum clk_reg_layout reg_layout; Seems like you forgot to update the struct's kerneldoc for this rename? > > /* Module Clocks */ > const struct mssr_mod_clk *mod_clks; 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
