clk: cs2000: tidyup DEVICE_CFG2 settings
authorKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Tue, 18 Apr 2017 02:19:37 +0000 (02:19 +0000)
committerStephen Boyd <sboyd@codeaurora.org>
Wed, 19 Apr 2017 17:18:53 +0000 (10:18 -0700)
DEVICE_CFG2 can select ratio from user defined ratio and LOCKCLK is
for it. But current driver sets fixed 0 value. This patch fixes it.
Note is that current cs2000 driver is using/supporting only ratio0
(= ch0) now.

DEVICE_CFG2 can select STATIC/DYNAMIC ratio mode, and current cs2000
driver is selecting STATIC mode, but it was not understandable on
current code. This patch also solve this issue.

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
drivers/clk/clk-cs2000-cp.c

index a8fa6bdd0e557feba43da00c92a84c3ef7f63532..f3ab0cad2c9d47e64f1d5a4864ae79ca107570a9 100644 (file)
@@ -48,7 +48,9 @@
 #define AUTORMOD       (1 << 3)
 #define LOCKCLK(x)     (((x) & 0x3) << 1)
 #define LOCKCLK_MASK   LOCKCLK(0x3)
-#define FRACNSRC       (1 << 0)
+#define FRACNSRC_MASK  (1 << 0)
+#define FRACNSRC_STATIC                (0 << 0)
+#define FRACNSRC_DYNAMIC       (1 << 1)
 
 /* GLOBAL_CFG */
 #define ENDEV2         (0x1)
@@ -267,8 +269,8 @@ static int cs2000_ratio_select(struct cs2000_priv *priv, int ch)
                return ret;
 
        ret = cs2000_bset(priv, DEVICE_CFG2,
-                         (AUTORMOD | LOCKCLK_MASK | FRACNSRC),
-                         0);
+                         (AUTORMOD | LOCKCLK_MASK | FRACNSRC_MASK),
+                         (LOCKCLK(ch) | FRACNSRC_STATIC));
        if (ret < 0)
                return ret;