On Friday 27 April 2018 05:47 AM, David Lechner wrote: > void __init dm644x_init_time(void) > { > +#ifdef CONFIG_COMMON_CLK > + void __iomem *pll1, *psc; > + struct clk *clk; > + > + clk_register_fixed_rate(NULL, "ref_clk", NULL, 0, DM644X_REF_FREQ); > + > + pll1 = ioremap(DAVINCI_PLL1_BASE, SZ_1K); > + dm355_pll1_init(NULL, pll1, NULL); This should be dm644x_*() > + > + psc = ioremap(DAVINCI_PWR_SLEEP_CNTRL_BASE, SZ_4K); > + dm355_psc_init(NULL, psc); This one should be dm644x_*() With those fixes, I was able to bootup on DM644x EVM and a visual inspection of clock debug dump shows it remains same before and after the conversion. 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