From: Mark Brown <broonie@kernel.org>
Date: Mon, 17 Nov 2014 12:36:56 +0000 (+0000)
Subject: cpufreq: Don't iterate over all CPUs when suspending and resuming governors
X-Git-Tag: firefly_0821_release~3680^2~19^2~2^2~2
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=212df5dd3212092602119d03a2f5d3a27be77ce4;p=firefly-linux-kernel-4.4.55.git

cpufreq: Don't iterate over all CPUs when suspending and resuming governors

The governors only expect to be stopped and started once so don't call
them repeatedly.

Signed-off-by: Mark Brown <broonie@linaro.org>
---

diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
index d444bfe83f13..ae9d9dc6b79b 100644
--- a/drivers/cpufreq/cpufreq.c
+++ b/drivers/cpufreq/cpufreq.c
@@ -1293,7 +1293,6 @@ static struct subsys_interface cpufreq_interface = {
 void cpufreq_suspend(void)
 {
 	struct cpufreq_policy *policy;
-	int cpu;
 
 	if (!cpufreq_driver)
 		return;
@@ -1303,20 +1302,15 @@ void cpufreq_suspend(void)
 
 	pr_debug("%s: Suspending Governors\n", __func__);
 
-	for_each_possible_cpu(cpu) {
-		if (!cpu_online(cpu))
-			continue;
-
-		policy = cpufreq_cpu_get(cpu);
+	policy = cpufreq_cpu_get(0);
 
-		if (__cpufreq_governor(policy, CPUFREQ_GOV_STOP))
-			pr_err("%s: Failed to stop governor for policy: %p\n",
-				__func__, policy);
-		else if (cpufreq_driver->suspend
-		    && cpufreq_driver->suspend(policy))
-			pr_err("%s: Failed to suspend driver: %p\n", __func__,
-				policy);
-	}
+	if (__cpufreq_governor(policy, CPUFREQ_GOV_STOP))
+		pr_err("%s: Failed to stop governor for policy: %p\n",
+			__func__, policy);
+	else if (cpufreq_driver->suspend
+	    && cpufreq_driver->suspend(policy))
+		pr_err("%s: Failed to suspend driver: %p\n", __func__,
+			policy);
 
 	cpufreq_suspended = true;
 }
@@ -1330,7 +1324,6 @@ void cpufreq_suspend(void)
 void cpufreq_resume(void)
 {
 	struct cpufreq_policy *policy;
-	int cpu;
 
 	if (!cpufreq_driver)
 		return;
@@ -1342,29 +1335,18 @@ void cpufreq_resume(void)
 
 	cpufreq_suspended = false;
 
-	for_each_possible_cpu(cpu) {
-		if (!cpu_online(cpu))
-			continue;
+	policy = cpufreq_cpu_get(0);
 
-		policy = cpufreq_cpu_get(cpu);
-
-		if (__cpufreq_governor(policy, CPUFREQ_GOV_START)
-		    || __cpufreq_governor(policy, CPUFREQ_GOV_LIMITS))
-			pr_err("%s: Failed to start governor for policy: %p\n",
-				__func__, policy);
-		else if (cpufreq_driver->resume
-		    && cpufreq_driver->resume(policy))
-			pr_err("%s: Failed to resume driver: %p\n", __func__,
-				policy);
-
-		/*
-		 * schedule call cpufreq_update_policy() for boot CPU, i.e. last
-		 * policy in list. It will verify that the current freq is in
-		 * sync with what we believe it to be.
-		 */
-		if (cpu == 0)
-			schedule_work(&policy->update);
-	}
+	if (__cpufreq_governor(policy, CPUFREQ_GOV_START)
+	    || __cpufreq_governor(policy, CPUFREQ_GOV_LIMITS))
+		pr_err("%s: Failed to start governor for policy: %p\n",
+			__func__, policy);
+	else if (cpufreq_driver->resume
+	    && cpufreq_driver->resume(policy))
+		pr_err("%s: Failed to resume driver: %p\n", __func__,
+			policy);
+
+	schedule_work(&policy->update);
 }
 
 /**