> -----Original Message----- > From: Viresh Kumar [mailto:viresh.kumar@xxxxxxxxxx] > Sent: Tuesday, March 18, 2014 12:27 PM > To: Zhang Zhuoyu-B46552 > Cc: Rafael J. Wysocki; cpufreq@xxxxxxxxxxxxxxx; linux-pm@xxxxxxxxxxxxxxx; > Tang Yuantian-B29983 > Subject: Re: [PATCH] cpufreq: powerpc: add cpufreq transition latency for > FSL e500mc Socs > > On 17 March 2014 11:54, Zhuoyu Zhang <Zhuoyu.Zhang@xxxxxxxxxxxxx> wrote: > > According to the data provided by HW Team, at least 12 internal > > platform clock cycles are required to stabilize a DFS clock switch on > FSL e500mc Socs. > > This patch replaces the CPUFREQ_ETERNAL with appropriate HW clock > > transition latency to make DFS governors work normally on Freescale > e500mc boards. > > > > Signed-off-by: Zhuoyu Zhang <Zhuoyu.Zhang@xxxxxxxxxxxxx> > > --- > > drivers/cpufreq/ppc-corenet-cpufreq.c | 6 ++++-- > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/cpufreq/ppc-corenet-cpufreq.c > > b/drivers/cpufreq/ppc-corenet-cpufreq.c > > index 051000f..88f08a6 100644 > > --- a/drivers/cpufreq/ppc-corenet-cpufreq.c > > +++ b/drivers/cpufreq/ppc-corenet-cpufreq.c > > @@ -21,6 +21,7 @@ > > #include <linux/of.h> > > #include <linux/slab.h> > > #include <linux/smp.h> > > +#include <sysdev/fsl_soc.h> > > > > /** > > * struct cpu_data - per CPU data struct @@ -133,7 +134,7 @@ static > > int corenet_cpufreq_cpu_init(struct cpufreq_policy *policy) { > > struct device_node *np; > > int i, count, ret; > > - u32 freq, mask; > > + u32 freq, sysclk, mask; > > struct clk *clk; > > struct cpufreq_frequency_table *table; > > struct cpu_data *data; > > @@ -205,7 +206,8 @@ static int corenet_cpufreq_cpu_init(struct > cpufreq_policy *policy) > > for_each_cpu(i, per_cpu(cpu_mask, cpu)) > > per_cpu(cpu_data, i) = data; > > > > - policy->cpuinfo.transition_latency = CPUFREQ_ETERNAL; > > + sysclk = fsl_get_sys_freq(); > > You don't need this extra variable. Simply use fsl_get_sys_freq() in the > below equation.. OK, I will rewrite it. Thanks, Viresh. Zhuoyu > > > + policy->cpuinfo.transition_latency = (12 * NSEC_PER_SEC) / > > + sysclk; > > of_node_put(np); > > Other than that: Acked-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx> > -- To unsubscribe from this list: send the line "unsubscribe cpufreq" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
