[PATCH 06/13] clk: Don't set clk->new_rate twice

Turquette, Mike mturquette at ti.com
Wed Apr 18 17:08:34 EDT 2012


On Tue, Apr 17, 2012 at 4:15 AM, Viresh Kumar <viresh.kumar at st.com> wrote:
> if (!clk->ops->round_rate && (clk->flags & CLK_SET_RATE_PARENT)) is true, then
> we don't need to set clk->new_rate here, as we will call clk_calc_subtree()
> afterwards and it also sets clk->new_rate.
>
> Signed-off-by: Viresh Kumar <viresh.kumar at st.com>

Looks good.  Will take into -next.

Thanks,
Mike

> ---
>  drivers/clk/clk.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c
> index af2bf12..865d0dd 100644
> --- a/drivers/clk/clk.c
> +++ b/drivers/clk/clk.c
> @@ -789,7 +789,7 @@ static struct clk *clk_calc_new_rates(struct clk *clk, unsigned long rate)
>
>        if (!clk->ops->round_rate) {
>                top = clk_calc_new_rates(clk->parent, rate);
> -               new_rate = clk->new_rate = clk->parent->new_rate;
> +               new_rate = clk->parent->new_rate;
>
>                goto out;
>        }
> --
> 1.7.9
>



More information about the linux-arm-kernel mailing list