On Thu, Jan 25, 2018 at 7:03 PM, Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx> wrote: > Add "rwdt" clock to r8a7791_mod_clks. Also, since we may need to access > the watchdog registers at any time, declare the clock as critical. > > Signed-off-by: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx> > Signed-off-by: Ramesh Shanmugasundaram <ramesh.shanmugasundaram@xxxxxxxxxxxxxx> With the critical clock handling removed (that can be done by the driver, if really needed) Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > --- a/drivers/clk/renesas/r8a7791-cpg-mssr.c > +++ b/drivers/clk/renesas/r8a7791-cpg-mssr.c > @@ -128,6 +128,7 @@ static const struct mssr_mod_clk r8a7791_mod_clks[] __initconst = { > DEF_MOD("cmt1", 329, R8A7791_CLK_R), > DEF_MOD("usbhs-dmac0", 330, R8A7791_CLK_HP), > DEF_MOD("usbhs-dmac1", 331, R8A7791_CLK_HP), > + DEF_MOD("rwdt", 402, R8A7791_CLK_R), > DEF_MOD("irqc", 407, R8A7791_CLK_CP), > DEF_MOD("intc-sys", 408, R8A7791_CLK_ZS), > DEF_MOD("audio-dmac1", 501, R8A7791_CLK_HP), > @@ -209,6 +210,7 @@ static const struct mssr_mod_clk r8a7791_mod_clks[] __initconst = { > }; > > static const unsigned int r8a7791_crit_mod_clks[] __initconst = { > + MOD_CLK_ID(402), /* RWDT */ > MOD_CLK_ID(408), /* INTC-SYS (GIC) */ > }; 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