clk: skip unnecessary set_phase if nothing to do
authorShawn Lin <shawn.lin@rock-chips.com>
Fri, 26 Feb 2016 01:25:52 +0000 (09:25 +0800)
committerStephen Boyd <sboyd@codeaurora.org>
Fri, 26 Feb 2016 02:10:51 +0000 (18:10 -0800)
Let's compare the degrees from clk_set_rate with
clk->core->phase. If the requested degrees is already
there, skip the following steps.

Signed-off-by: Shawn Lin <shawn.lin@rock-chips.com>
[sboyd@codeaurora.org: s/drgrees/degrees/ in commit text]
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
drivers/clk/clk.c

index d95d6f924cace4cf3d050f417d568662d034c6f9..437eebf89fcf8d7ed31735ea697498e79dc6a993 100644 (file)
@@ -1841,6 +1841,10 @@ int clk_set_phase(struct clk *clk, int degrees)
 
        clk_prepare_lock();
 
+       /* bail early if nothing to do */
+       if (degrees == clk->core->phase)
+               goto out;
+
        trace_clk_set_phase(clk->core, degrees);
 
        if (clk->core->ops->set_phase)
@@ -1851,6 +1855,7 @@ int clk_set_phase(struct clk *clk, int degrees)
        if (!ret)
                clk->core->phase = degrees;
 
+out:
        clk_prepare_unlock();
 
        return ret;