Re: [PATCH v7] cpufreq: Add SFI based cpufreq driver support

["Kasagar, Srinidhi" <srinidhi.kasagar@intel.com>]

On Tue, Dec 16, 2014 at 03:15:33AM -0500, Len Brown wrote:

[...]

> replace line above with...
> 
> policy->cpuinfo.transition_latency = CPUFREQ_ETERNAL;

But we know the transition latency of this system. Why should we
claim it as "unknown"? The sfi_freq_table_entry has latency
field - does it not make irrelevant then?

> 
> ondemand will compare this to TRANSITION_LATENCY_LIMIT (10 * 1000 * 1000)
> and since -1 is less than 10,000,000; it will be happy.
> 
> > +       for (i = 0; i < num_freq_table_entries; i++) {
> > +               /* detect transition latency */
> > +               if ((sfi_cpufreq_array[i].latency * 1000) >
> > +                   policy->cpuinfo.transition_latency)
> > +                       policy->cpuinfo.transition_latency =
> > +                               sfi_cpufreq_array[i].latency * 1000;
> 
> delete above 5 lines.  every SFI system will have latency
> plenty lower than that required by ondemand governor.

Srinidhi