From: Kuninori Morimoto <kuninori.morimoto...@renesas.com>

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...@renesas.com>
---
 drivers/clk/clk-cs2000-cp.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/drivers/clk/clk-cs2000-cp.c b/drivers/clk/clk-cs2000-cp.c
index a8fa6bd..f3ab0ca 100644
--- a/drivers/clk/clk-cs2000-cp.c
+++ b/drivers/clk/clk-cs2000-cp.c
@@ -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;
 
-- 
1.9.1

Reply via email to