Commit b20a3f3d authored Aug 10, 2017 by Sudeep Holla Committed by Rafael J. Wysocki Aug 18, 2017

cpufreq: remove setting of policy->cpu in policy->cpus during init



policy->cpu is copied into policy->cpus in cpufreq_online() before
calling into cpufreq_driver->init(). So there's no need to set the
same in the individual driver init() functions again.

This patch removes the redundant setting of policy->cpu in policy->cpus
in intel_pstate and cppc drivers.

Reported-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

parent ccc03d86

drivers/cpufreq/cppc_cpufreq.c

+0 −1

Original line number	Diff line number	Diff line
		@@ -172,7 +172,6 @@ static int cppc_cpufreq_cpu_init(struct cpufreq_policy *policy)
		return -EFAULT;
		}

		cpumask_set_cpu(policy->cpu, policy->cpus);
		cpu->cur_policy = policy;

		/* Set policy->cur to max now. The governors will adjust later. */

drivers/cpufreq/intel_pstate.c

+0 −1

Original line number	Diff line number	Diff line
		@@ -2124,7 +2124,6 @@ static int __intel_pstate_cpu_init(struct cpufreq_policy *policy)
		policy->cpuinfo.max_freq *= cpu->pstate.scaling;

		intel_pstate_init_acpi_perf_limits(policy);
		cpumask_set_cpu(policy->cpu, policy->cpus);

		policy->fast_switch_possible = true;