imx/clk-pllv2: fix wrong do_div() usage

do_div() is meant to be used with an unsigned dividend.

Signed-off-by: Nicolas Pitre <nico@linaro.org>
Acked-by: Shawn Guo <shawnguo@kernel.org>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>

1 files changed, 5 insertions, 4 deletions

diff --git a/drivers/clk/imx/clk-pllv2.c b/drivers/clk/imx/clk-pllv2.c
index b18f875eac6a..4aeda56ce372 100644
--- a/drivers/clk/imx/clk-pllv2.c
+++ b/drivers/clk/imx/clk-pllv2.c
@@ -79,7 +79,7 @@ static unsigned long __clk_pllv2_recalc_rate(unsigned long parent_rate,
 {
         long mfi, mfn, mfd, pdf, ref_clk;
         unsigned long dbl;
-        s64 temp;
+        u64 temp;
 
         dbl = dp_ctl & MXC_PLL_DP_CTL_DPDCK0_2_EN;
 
@@ -98,8 +98,9 @@ static unsigned long __clk_pllv2_recalc_rate(unsigned long parent_rate,
         temp = (u64) ref_clk * abs(mfn);
         do_div(temp, mfd + 1);
         if (mfn < 0)
-                temp = -temp;
-        temp = (ref_clk * mfi) + temp;
+                temp = (ref_clk * mfi) - temp;
+        else
+                temp = (ref_clk * mfi) + temp;
 
         return temp;
 }
@@ -126,7 +127,7 @@ static int __clk_pllv2_set_rate(unsigned long rate, unsigned long parent_rate,
 {
         u32 reg;
         long mfi, pdf, mfn, mfd = 999999;
-        s64 temp64;
+        u64 temp64;
         unsigned long quad_parent_rate;
 
         quad_parent_rate = 4 * parent_rate;