clk: cs2000: Add cs2000_set_saved_rate

Commit Message

Yoshihiro Kaneko April 19, 2017, 5:46 p.m. UTC

From: Gaku Inami <[e-mail]>

This patch adds the common function to reset the clk rate in order to
be able to use it in other cases.

Signed-off-by: Gaku Inami <[e-mail]>
Signed-off-by: Hiroyuki Yokoyama <[e-mail]>
Signed-off-by: Yoshihiro Kaneko <[e-mail]>
---
This patch is based on the clk-next branch of linux-clk tree.

 drivers/clk/clk-cs2000-cp.c | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

Comments

Kuninori Morimoto April 19, 2017, 11:57 p.m. UTC | #1

Hi

> From: Gaku Inami <[e-mail]>
> 
> This patch adds the common function to reset the clk rate in order to
> be able to use it in other cases.
> 
> Signed-off-by: Gaku Inami <[e-mail]>
> Signed-off-by: Hiroyuki Yokoyama <[e-mail]>
> Signed-off-by: Yoshihiro Kaneko <[e-mail]>
> ---

Acked-by: Kuninori Morimoto <[e-mail]>

Stephen Boyd July 21, 2017, 8:16 p.m. UTC | #2

On 04/20, Yoshihiro Kaneko wrote:
> From: Gaku Inami <[e-mail]>
> 
> This patch adds the common function to reset the clk rate in order to
> be able to use it in other cases.
> 
> Signed-off-by: Gaku Inami <[e-mail]>
> Signed-off-by: Hiroyuki Yokoyama <[e-mail]>
> Signed-off-by: Yoshihiro Kaneko <[e-mail]>
> ---
> This patch is based on the clk-next branch of linux-clk tree.

Geert?

Geert Uytterhoeven Aug. 17, 2017, 12:43 p.m. UTC | #3

Hi Stephen,

On Fri, Jul 21, 2017 at 10:16 PM, Stephen Boyd <[e-mail]> wrote:
> On 04/20, Yoshihiro Kaneko wrote:
>> From: Gaku Inami <[e-mail]>
>>
>> This patch adds the common function to reset the clk rate in order to
>> be able to use it in other cases.
>>
>> Signed-off-by: Gaku Inami <[e-mail]>
>> Signed-off-by: Hiroyuki Yokoyama <[e-mail]>
>> Signed-off-by: Yoshihiro Kaneko <[e-mail]>
>> ---
>> This patch is based on the clk-next branch of linux-clk tree.
>
> Geert?

The CS2000 clock generator is not a Renesas part, so please take it directly.
Thanks!

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [e-mail]

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Stephen Boyd Aug. 31, 2017, 6:32 p.m. UTC | #4

On 04/20, Yoshihiro Kaneko wrote:
> From: Gaku Inami <[e-mail]>
> 
> This patch adds the common function to reset the clk rate in order to
> be able to use it in other cases.
> 
> Signed-off-by: Gaku Inami <[e-mail]>
> Signed-off-by: Hiroyuki Yokoyama <[e-mail]>
> Signed-off-by: Yoshihiro Kaneko <[e-mail]>
> ---

Applied to clk-next

Patch

diff --git a/drivers/clk/clk-cs2000-cp.c b/drivers/clk/clk-cs2000-cp.c
index 4df38c5..5b3516d 100644
--- a/drivers/clk/clk-cs2000-cp.c
+++ b/drivers/clk/clk-cs2000-cp.c
@@ -324,6 +324,15 @@  static int cs2000_set_rate(struct clk_hw *hw,
 	return __cs2000_set_rate(priv, ch, rate, parent_rate);
 }
 
+static int cs2000_set_saved_rate(struct cs2000_priv *priv)
+{
+	int ch = 0; /* it uses ch0 only at this point */
+
+	return __cs2000_set_rate(priv, ch,
+				 priv->saved_rate,
+				 priv->saved_parent_rate);
+}
+
 static int cs2000_enable(struct clk_hw *hw)
 {
 	struct cs2000_priv *priv = hw_to_priv(hw);
@@ -516,11 +525,8 @@  static int cs2000_probe(struct i2c_client *client,
 static int cs2000_resume(struct device *dev)
 {
 	struct cs2000_priv *priv = dev_get_drvdata(dev);
-	int ch = 0; /* it uses ch0 only at this point */
 
-	return __cs2000_set_rate(priv, ch,
-				 priv->saved_rate,
-				 priv->saved_parent_rate);
+	return cs2000_set_saved_rate(priv);
 }
 
 static const struct dev_pm_ops cs2000_pm_ops = {