clk: tegra: Return the exact clock rate from clk_round_rate
authorRobert Yang <decatf@gmail.com>
Tue, 25 Sep 2018 21:49:40 +0000 (17:49 -0400)
committerStephen Boyd <sboyd@kernel.org>
Fri, 14 Dec 2018 21:32:55 +0000 (13:32 -0800)
The current behavior is that clk_round_rate would return the same clock
rate passed to it for valid PLL configurations. This change will return
the exact rate the PLL will provide in accordance with clk API.

Signed-off-by: Robert Yang <decatf@gmail.com>
Reviewed-by: Dmitry Osipenko <digetx@gmail.com>
Tested-by: Dmitry Osipenko <digetx@gmail.com>
Acked-by: Thierry Reding <treding@nvidia.com>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/tegra/clk-pll.c

index 830d1c8..b50b746 100644 (file)
@@ -590,12 +590,13 @@ static int _calc_rate(struct clk_hw *hw, struct tegra_clk_pll_freq_table *cfg,
        cfg->n = cfg->output_rate / cfreq;
        cfg->cpcon = OUT_OF_TABLE_CPCON;
 
-       if (cfg->m > divm_max(pll) || cfg->n > divn_max(pll) ||
-           (1 << p_div) > divp_max(pll)
-           || cfg->output_rate > pll->params->vco_max) {
+       if (cfg->m == 0 || cfg->m > divm_max(pll) ||
+           cfg->n > divn_max(pll) || (1 << p_div) > divp_max(pll) ||
+           cfg->output_rate > pll->params->vco_max) {
                return -EINVAL;
        }
 
+       cfg->output_rate = cfg->n * DIV_ROUND_UP(parent_rate, cfg->m);
        cfg->output_rate >>= p_div;
 
        if (pll->params->pdiv_tohw) {