Commit d92d50a4 authored by Viresh Kumar's avatar Viresh Kumar Committed by Rafael J. Wysocki

cpufreq: pass policy to __cpufreq_get()

There is no point finding out the 'policy' again within __cpufreq_get()
when all the callers already have it. Just make them pass policy instead.
Signed-off-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent a1e1dc41
...@@ -63,7 +63,7 @@ static DECLARE_RWSEM(cpufreq_rwsem); ...@@ -63,7 +63,7 @@ static DECLARE_RWSEM(cpufreq_rwsem);
/* internal prototypes */ /* internal prototypes */
static int __cpufreq_governor(struct cpufreq_policy *policy, static int __cpufreq_governor(struct cpufreq_policy *policy,
unsigned int event); unsigned int event);
static unsigned int __cpufreq_get(unsigned int cpu); static unsigned int __cpufreq_get(struct cpufreq_policy *policy);
static void handle_update(struct work_struct *work); static void handle_update(struct work_struct *work);
/** /**
...@@ -563,7 +563,7 @@ store_one(scaling_max_freq, max); ...@@ -563,7 +563,7 @@ store_one(scaling_max_freq, max);
static ssize_t show_cpuinfo_cur_freq(struct cpufreq_policy *policy, static ssize_t show_cpuinfo_cur_freq(struct cpufreq_policy *policy,
char *buf) char *buf)
{ {
unsigned int cur_freq = __cpufreq_get(policy->cpu); unsigned int cur_freq = __cpufreq_get(policy);
if (!cur_freq) if (!cur_freq)
return sprintf(buf, "<unknown>"); return sprintf(buf, "<unknown>");
return sprintf(buf, "%u\n", cur_freq); return sprintf(buf, "%u\n", cur_freq);
...@@ -1575,15 +1575,14 @@ unsigned int cpufreq_quick_get_max(unsigned int cpu) ...@@ -1575,15 +1575,14 @@ unsigned int cpufreq_quick_get_max(unsigned int cpu)
} }
EXPORT_SYMBOL(cpufreq_quick_get_max); EXPORT_SYMBOL(cpufreq_quick_get_max);
static unsigned int __cpufreq_get(unsigned int cpu) static unsigned int __cpufreq_get(struct cpufreq_policy *policy)
{ {
struct cpufreq_policy *policy = per_cpu(cpufreq_cpu_data, cpu);
unsigned int ret_freq = 0; unsigned int ret_freq = 0;
if (!cpufreq_driver->get) if (!cpufreq_driver->get)
return ret_freq; return ret_freq;
ret_freq = cpufreq_driver->get(cpu); ret_freq = cpufreq_driver->get(policy->cpu);
if (ret_freq && policy->cur && if (ret_freq && policy->cur &&
!(cpufreq_driver->flags & CPUFREQ_CONST_LOOPS)) { !(cpufreq_driver->flags & CPUFREQ_CONST_LOOPS)) {
...@@ -1611,7 +1610,7 @@ unsigned int cpufreq_get(unsigned int cpu) ...@@ -1611,7 +1610,7 @@ unsigned int cpufreq_get(unsigned int cpu)
if (policy) { if (policy) {
down_read(&policy->rwsem); down_read(&policy->rwsem);
ret_freq = __cpufreq_get(cpu); ret_freq = __cpufreq_get(policy);
up_read(&policy->rwsem); up_read(&policy->rwsem);
cpufreq_cpu_put(policy); cpufreq_cpu_put(policy);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment