From: Kelvin Cheung <keguang.zh...@gmail.com>

Make use of GENMASK instead of open coding the equivalent operation,
and update the PLL formula.

Signed-off-by: Kelvin Cheung <keguang.zh...@gmail.com>
---
 drivers/clk/loongson1/clk-loongson1b.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/loongson1/clk-loongson1b.c 
b/drivers/clk/loongson1/clk-loongson1b.c
index 4b3d9d2..f36a97e 100644
--- a/drivers/clk/loongson1/clk-loongson1b.c
+++ b/drivers/clk/loongson1/clk-loongson1b.c
@@ -26,7 +26,7 @@ static unsigned long ls1x_pll_recalc_rate(struct clk_hw *hw,
        u32 pll, rate;
 
        pll = __raw_readl(LS1X_CLK_PLL_FREQ);
-       rate = 12 + (pll & 0x3f) + (((pll >> 8) & 0x3ff) >> 10);
+       rate = 12 + (pll & GENMASK(5, 0));
        rate *= OSC;
        rate >>= 1;
 
-- 
1.9.1

Reply via email to