| Message ID | 8b7f30f52b99c21697168d77f1fae298dad50226.1375964117.git.viresh.kumar@linaro.org |
|---|---|
| State | New |
| Headers | show |
On Thu, Aug 08, 2013 at 07:18:19PM +0530, Viresh Kumar wrote: > Lets use cpufreq_table_validate_and_show() instead of calling > cpufreq_frequency_table_cpuinfo() and cpufreq_frequency_table_get_attr(). > > Cc: Andrew Lunn <andrew@lunn.ch> > Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> > --- > drivers/cpufreq/kirkwood-cpufreq.c | 10 +--------- > 1 file changed, 1 insertion(+), 9 deletions(-) > > diff --git a/drivers/cpufreq/kirkwood-cpufreq.c b/drivers/cpufreq/kirkwood-cpufreq.c > index 45e4d7f..336f171 100644 > --- a/drivers/cpufreq/kirkwood-cpufreq.c > +++ b/drivers/cpufreq/kirkwood-cpufreq.c > @@ -125,19 +125,11 @@ static int kirkwood_cpufreq_target(struct cpufreq_policy *policy, > /* Module init and exit code */ > static int kirkwood_cpufreq_cpu_init(struct cpufreq_policy *policy) > { > - int result; > - > /* cpuinfo and default policy values */ > policy->cpuinfo.transition_latency = 5000; /* 5uS */ > policy->cur = kirkwood_cpufreq_get_cpu_frequency(0); > > - result = cpufreq_frequency_table_cpuinfo(policy, kirkwood_freq_table); > - if (result) > - return result; > - > - cpufreq_frequency_table_get_attr(kirkwood_freq_table, policy->cpu); > - > - return 0; > + return cpufreq_table_validate_and_show(policy, kirkwood_freq_table); > } > > static int kirkwood_cpufreq_cpu_exit(struct cpufreq_policy *policy) > -- > 1.7.12.rc2.18.g61b472e Reviewed-by: Andrew Lunn <andrew@lunn.ch> Thanks Andrew
diff --git a/drivers/cpufreq/kirkwood-cpufreq.c b/drivers/cpufreq/kirkwood-cpufreq.c index 45e4d7f..336f171 100644 --- a/drivers/cpufreq/kirkwood-cpufreq.c +++ b/drivers/cpufreq/kirkwood-cpufreq.c @@ -125,19 +125,11 @@ static int kirkwood_cpufreq_target(struct cpufreq_policy *policy, /* Module init and exit code */ static int kirkwood_cpufreq_cpu_init(struct cpufreq_policy *policy) { - int result; - /* cpuinfo and default policy values */ policy->cpuinfo.transition_latency = 5000; /* 5uS */ policy->cur = kirkwood_cpufreq_get_cpu_frequency(0); - result = cpufreq_frequency_table_cpuinfo(policy, kirkwood_freq_table); - if (result) - return result; - - cpufreq_frequency_table_get_attr(kirkwood_freq_table, policy->cpu); - - return 0; + return cpufreq_table_validate_and_show(policy, kirkwood_freq_table); } static int kirkwood_cpufreq_cpu_exit(struct cpufreq_policy *policy)
Lets use cpufreq_table_validate_and_show() instead of calling cpufreq_frequency_table_cpuinfo() and cpufreq_frequency_table_get_attr(). Cc: Andrew Lunn <andrew@lunn.ch> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> --- drivers/cpufreq/kirkwood-cpufreq.c | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-)