[PATCH] clk: zynqmp: move zynqmp_pll_set_mode out of round_rate callback

[quanyang . wang]

From: Quanyang Wang <quanyang.w...@windriver.com>

The round_rate callback should only perform rate calculation and not
involve calling zynqmp_pll_set_mode to change the pll mode. So let's
move zynqmp_pll_set_mode out of round_rate and to set_rate callback.

Reported-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
Signed-off-by: Quanyang Wang <quanyang.w...@windriver.com>
---
 drivers/clk/zynqmp/pll.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/clk/zynqmp/pll.c b/drivers/clk/zynqmp/pll.c
index fb16ddbb7e2d..1aea5c137b69 100644
--- a/drivers/clk/zynqmp/pll.c
+++ b/drivers/clk/zynqmp/pll.c
@@ -104,9 +104,7 @@ static long zynqmp_pll_round_rate(struct clk_hw *hw, 
unsigned long rate,
        /* Enable the fractional mode if needed */
        rate_div = (rate * FRAC_DIV) / *prate;
        f = rate_div % FRAC_DIV;
-       zynqmp_pll_set_mode(hw, !!f);
-
-       if (zynqmp_pll_get_mode(hw) == PLL_MODE_FRAC) {
+       if (f) {
                if (rate > PS_PLL_VCO_MAX) {
                        fbdiv = rate / PS_PLL_VCO_MAX;
                        rate = rate / (fbdiv + 1);
@@ -177,10 +175,12 @@ static int zynqmp_pll_set_rate(struct clk_hw *hw, 
unsigned long rate,
        long rate_div, frac, m, f;
        int ret;
 
-       if (zynqmp_pll_get_mode(hw) == PLL_MODE_FRAC) {
-               rate_div = (rate * FRAC_DIV) / parent_rate;
+       rate_div = (rate * FRAC_DIV) / parent_rate;
+       f = rate_div % FRAC_DIV;
+       zynqmp_pll_set_mode(hw, !!f);
+
+       if (f) {
                m = rate_div / FRAC_DIV;
-               f = rate_div % FRAC_DIV;
                m = clamp_t(u32, m, (PLL_FBDIV_MIN), (PLL_FBDIV_MAX));
                rate = parent_rate * m;
                frac = (parent_rate * f) / FRAC_DIV;
-- 
2.25.1