| Message ID | 5714b9e8a37bd5aea5f77676b75d9e7773e0f0e2.1412048958.git.viresh.kumar@linaro.org |
|---|---|
| State | New |
| Headers | show |
On Tuesday, September 30, 2014 09:19:57 AM Viresh Kumar wrote: > 'cpu_data' is updated for policy->cpu first and then for all CPUs in > policy->cpus. policy->cpus is guaranteed to contain policy->cpu as well and so > the first write to 'cpu_data' for policy->cpu is redundant. Remove it. > > Acked-by: Tang Yuantian <Yuantian.Tang@freescale.com> > Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> Queued up for 3.18, thanks! > --- > drivers/cpufreq/ppc-corenet-cpufreq.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/cpufreq/ppc-corenet-cpufreq.c b/drivers/cpufreq/ppc-corenet-cpufreq.c > index 3607070..bee5df7 100644 > --- a/drivers/cpufreq/ppc-corenet-cpufreq.c > +++ b/drivers/cpufreq/ppc-corenet-cpufreq.c > @@ -199,7 +199,6 @@ static int corenet_cpufreq_cpu_init(struct cpufreq_policy *policy) > } > > data->table = table; > - per_cpu(cpu_data, cpu) = data; > > /* update ->cpus if we have cluster, no harm if not */ > cpumask_copy(policy->cpus, per_cpu(cpu_mask, cpu)); >
diff --git a/drivers/cpufreq/ppc-corenet-cpufreq.c b/drivers/cpufreq/ppc-corenet-cpufreq.c index 3607070..bee5df7 100644 --- a/drivers/cpufreq/ppc-corenet-cpufreq.c +++ b/drivers/cpufreq/ppc-corenet-cpufreq.c @@ -199,7 +199,6 @@ static int corenet_cpufreq_cpu_init(struct cpufreq_policy *policy) } data->table = table; - per_cpu(cpu_data, cpu) = data; /* update ->cpus if we have cluster, no harm if not */ cpumask_copy(policy->cpus, per_cpu(cpu_mask, cpu));