* [PATCH V2 Resend 31/92] cpufreq: sh: Use generic cpufreq routines [not found] <cover.1380783079.git.viresh.kumar@linaro.org> @ 2013-10-03 15:10 ` Viresh Kumar 2013-10-03 15:11 ` [PATCH V2 Resend 68/92] cpufreq: sh: don't initialize part of policy that is set by core too Viresh Kumar 1 sibling, 0 replies; 2+ messages in thread From: Viresh Kumar @ 2013-10-03 15:10 UTC (permalink / raw) To: rjw; +Cc: cpufreq, linux-pm, Viresh Kumar, Paul Mundt, linux-sh Most of the CPUFreq drivers do similar things in .exit() and .verify() routines and .attr. So its better if we have generic routines for them which can be used by cpufreq drivers then. This patch uses these generic routines for this driver. Cc: Paul Mundt <lethal@linux-sh.org> Cc: linux-sh@vger.kernel.org Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> --- drivers/cpufreq/sh-cpufreq.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/cpufreq/sh-cpufreq.c b/drivers/cpufreq/sh-cpufreq.c index 1362e88..91c6446 100644 --- a/drivers/cpufreq/sh-cpufreq.c +++ b/drivers/cpufreq/sh-cpufreq.c @@ -154,11 +154,6 @@ static int sh_cpufreq_cpu_exit(struct cpufreq_policy *policy) return 0; } -static struct freq_attr *sh_freq_attr[] = { - &cpufreq_freq_attr_scaling_available_freqs, - NULL, -}; - static struct cpufreq_driver sh_cpufreq_driver = { .name = "sh", .get = sh_cpufreq_get, @@ -166,7 +161,7 @@ static struct cpufreq_driver sh_cpufreq_driver = { .verify = sh_cpufreq_verify, .init = sh_cpufreq_cpu_init, .exit = sh_cpufreq_cpu_exit, - .attr = sh_freq_attr, + .attr = cpufreq_generic_attr, }; static int __init sh_cpufreq_module_init(void) -- 1.7.12.rc2.18.g61b472e ^ permalink raw reply related [flat|nested] 2+ messages in thread
* [PATCH V2 Resend 68/92] cpufreq: sh: don't initialize part of policy that is set by core too [not found] <cover.1380783079.git.viresh.kumar@linaro.org> 2013-10-03 15:10 ` [PATCH V2 Resend 31/92] cpufreq: sh: Use generic cpufreq routines Viresh Kumar @ 2013-10-03 15:11 ` Viresh Kumar 1 sibling, 0 replies; 2+ messages in thread From: Viresh Kumar @ 2013-10-03 15:11 UTC (permalink / raw) To: rjw; +Cc: cpufreq, linux-pm, Viresh Kumar, Paul Mundt, linux-sh Many common initializations of struct policy are moved to core now and hence this driver doesn't need to do it. This patch removes such code. Most recent of those changes is to call ->get() in the core after calling ->init(). Cc: Paul Mundt <lethal@linux-sh.org> Cc: linux-sh@vger.kernel.org Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> --- drivers/cpufreq/sh-cpufreq.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/cpufreq/sh-cpufreq.c b/drivers/cpufreq/sh-cpufreq.c index 91c6446..018348b 100644 --- a/drivers/cpufreq/sh-cpufreq.c +++ b/drivers/cpufreq/sh-cpufreq.c @@ -114,8 +114,6 @@ static int sh_cpufreq_cpu_init(struct cpufreq_policy *policy) return PTR_ERR(cpuclk); } - policy->cur = sh_cpufreq_get(cpu); - freq_table = cpuclk->nr_freqs ? cpuclk->freq_table : NULL; if (freq_table) { int result; -- 1.7.12.rc2.18.g61b472e ^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-10-03 15:11 UTC | newest] Thread overview: 2+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- [not found] <cover.1380783079.git.viresh.kumar@linaro.org> 2013-10-03 15:10 ` [PATCH V2 Resend 31/92] cpufreq: sh: Use generic cpufreq routines Viresh Kumar 2013-10-03 15:11 ` [PATCH V2 Resend 68/92] cpufreq: sh: don't initialize part of policy that is set by core too Viresh Kumar
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).