On 5/14/2015 4:23 PM, Benson Leung wrote: > On Tue, May 12, 2015 at 10:24 AM, Rhyland Klein <rklein@xxxxxxxxxx> wrote: >> +static int _p_div_to_hw(struct clk_hw *hw, u8 p_div) >> +{ >> + struct tegra_clk_pll *pll = to_clk_pll(hw); >> + struct pdiv_map *p_tohw = pll->params->pdiv_tohw; >> + >> + if (p_tohw) { >> + while (p_tohw->pdiv) { >> + if (p_div <= p_tohw->pdiv) >> + return p_tohw->hw_val; >> + p_tohw++; >> + } >> + return -EINVAL; >> + } >> + return -EINVAL; >> +} > > This is the same function as the one that lives in clk-pll.c. Can you > make these common? > Sure, I'll add that for v6. -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