Hi Peter, > +#define PLL_BASE_LOCK BIT(27) > +#define PLL_MISC_LOCK_ENABLE 18 I don't see these used anywhere. > +static const char *cclk_lp_parents[] = { "clk_m", "pll_c", "clk_32k", "pll_m", > + "pll_p", "pll_p_out4", "unused", > + "unused", "pll_x", "pll_x_out0" }; nit: insert blank line here. > +static void __init tegra_sclk_init(void __iomem *clk_base, > + struct tegra_clk *tegra_clks) > +{ Thanks, Andrew -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html