On 5/13/2015 8:25 PM, Benson Leung wrote: > On Tue, May 12, 2015 at 10:24 AM, Rhyland Klein <rklein@xxxxxxxxxx> wrote: >> +static void pll_clk_start_ss(struct tegra_clk_pll *pll) >> +{ >> + if (pll->params->defaults_set && pll->params->ssc_ctrl_en_mask) { > > Is there any reason you're checking for the existence of > ssc_ctrl_en_mask rather than ssc_ctrl_reg? Not particularly, just a way of seeing if ssc is supported by this pll, I will change it to check the register to be more consistent with other checks in the code. -rhyland -- nvpublic -- 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