[PATCH v4 4/7] cpufreq: add clk-reg cpufreq driver

Richard Zhao richard.zhao at linaro.org
Sun Dec 25 02:52:53 EST 2011


On Sat, Dec 24, 2011 at 01:42:29PM +0000, Mark Brown wrote:
> On Sat, Dec 24, 2011 at 09:28:33PM +0800, Richard Zhao wrote:
> > On Sat, Dec 24, 2011 at 12:24:11PM +0000, Mark Brown wrote:
> 
> >  - trans-latency :  transition latency of cpu freq and related regulator,
> >    in unit of ns.
> 
> > Does it look better?
> 
> I think it shouldn't include the regulator part of things.
> 
> > If you think regulator thansition latency is board specific, then the board
> > dts can overrite it.
> 
> We should just query this information from the regulator subsystem
> (there's hooks but currently nothing implements them).  The regulators
> can define their own bindings if they need to read it from device tree,
> most of them should be able to do this as a function of knowing about
> the device.  None of this is specific to cpufreq so cpufreq shouldn't
> have to define its own support for this.
I'd like to query the latency by call clk and regulator APIs. but as you said
both of them have not implemented it yet. I think, for now, we can use the
property to get the total latency. Once I can get it at runtime, I'll remove
it. So the definition of trans-latency is just the same as cpufreq transition_latency,
people get less confused. What do you think?

Thanks
Richard


More information about the devicetree-discuss mailing list