[PATCH] clk: Update comment for clk_round_rate()

Viresh Kumar viresh.kumar at st.com
Thu Apr 5 01:11:58 EDT 2012


On 4/4/2012 11:37 PM, Turquette, Mike wrote:
> The common clk framework doesn't enforce any policy like this, nor do
> I think it should.  The clk framework is far from complete and I
> wouldn't be surprised if we see folks who want their rounded rate to
> represent a minimum value (instead of a maximum as your patch states).

Ok. Just for example, suppose foo_clk can have following rates:
100, 110, 120, 130, 140.

And we call round_rate with a value of 126.

Now i can't visualize why would anybody want it to return 130 (above the
limits requested)? I agree both 100 and 120 can be returned, based on
your below logic.

>  This might be achieved later on by a platform-specific or
> framework-wide flag, similar to how CPUfreq does it today
> (CPUFREQ_RELATION_L and CPUFREQ_RELATION_H).
> 

> Thank you for submitting the patch, but for now it's a NACK.

No Probs.

-- 
viresh



More information about the linux-arm-kernel mailing list