Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 65922465 authored by Viresh Kumar's avatar Viresh Kumar Committed by Rafael J. Wysocki
Browse files

cpufreq: Remove unused HOTPLUG_CPU code 



Because the sibling cpu of any online cpu is identified very early in
cpufreq_add_dev(), below code is never executed. And so can be removed.

Signed-off-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent 8e53695f

drivers/cpufreq/cpufreq.c

+2 −15
Original line number Diff line number Diff line
@@ -852,7 +852,7 @@ static int cpufreq_add_policy_cpu(unsigned int cpu, unsigned int sibling, 
static int cpufreq_add_dev(struct device *dev, struct subsys_interface *sif)

{

	unsigned int j, cpu = dev->id;

	int ret = -ENOMEM, found = 0;

	int ret = -ENOMEM;

	struct cpufreq_policy *policy;

	unsigned long flags;

#ifdef CONFIG_HOTPLUG_CPU
@@ -900,6 +900,7 @@ static int cpufreq_add_dev(struct device *dev, struct subsys_interface *sif) 
		goto err_free_cpumask;



	policy->cpu = cpu;

	policy->governor = CPUFREQ_DEFAULT_GOVERNOR;

	cpumask_copy(policy->cpus, cpumask_of(cpu));



	/* Initially set CPU itself as the policy_cpu */
@@ -910,20 +911,6 @@ static int cpufreq_add_dev(struct device *dev, struct subsys_interface *sif) 
	init_completion(&policy->kobj_unregister);

	INIT_WORK(&policy->update, handle_update);



	/* Set governor before ->init, so that driver could check it */

#ifdef CONFIG_HOTPLUG_CPU

	for_each_online_cpu(sibling) {

		struct cpufreq_policy *cp = per_cpu(cpufreq_cpu_data, sibling);

		if (cp && cp->governor &&

		    (cpumask_test_cpu(cpu, cp->related_cpus))) {

			policy->governor = cp->governor;

			found = 1;

			break;

		}

	}

#endif

	if (!found)

		policy->governor = CPUFREQ_DEFAULT_GOVERNOR;

	/* call driver. From then on the cpufreq must be able

	 * to accept all calls to ->verify and ->setpolicy for this CPU

	 */