On Saturday 20 January 2018 10:44 PM, David Lechner wrote: > void __init dm646x_init_time(unsigned long ref_clk_rate, > unsigned long aux_clkin_rate) > { > +#ifdef CONFIG_COMMON_CLK > + void __iomem *pll1, *pll2, *psc; > + struct clk *clk; > + > + pll1 = ioremap(DAVINCI_PLL1_BASE, SZ_4K); > + pll2 = ioremap(DAVINCI_PLL2_BASE, SZ_4K); > + psc = ioremap(DAVINCI_PWR_SLEEP_CNTRL_BASE, SZ_4K); > + > + clk_register_fixed_rate(NULL, "ref_clk", NULL, 0, ref_clk_rate); > + clk_register_fixed_rate(NULL, "aux_clkin", NULL, 0, aux_clkin_rate); > + > + dm646x_pll_clk_init(pll1, pll2); > + > + dm646x_psc_clk_init(psc); > + /* no LPSC, always enabled; c.f. spruep9a */ > + clk = clk_register_fixed_factor(NULL, "timer2", "pll1_sysclk3", 0, 1, 1); > + clk_register_clkdev(clk, NULL, "davinci-wdt"); Lets move this to dm646x_pll_clk_init() and directly register to clkdev to pll1_sysclk3? Thanks, Sekhar -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html