cpufreq annoying warning fix

PATCH New Benjamin Herrenschmidt 1 participant 1 message May 2, 2005 CPU frequency scaling framework

lore

Share & tools

lore

Benjamin Herrenschmidt

author

2005-05-02 06:27 UTC #0 lore raw

PATCH 1/1 cpufreq annoying warning fix

Hi !

The cpufreq core patch I sent earlier got only half-applied. I added a flag to let the low level driver disable an annoying warning on suspend/resume that is normal on ppc, but the "resume" part of it wasn't applied. This patch just adds back that missing bit. The original patch also reworked the resume() function to avoid nesting too many if () statements along the way I did the suspend() one, but I didn't include that in the patch below.

Signed-off-by Benjamin Herrenschmidt <[email protected]>

drivers/cpufreq/cpufreq.c +4 -3

Index: linux-work/drivers/cpufreq/cpufreq.c
===================================================================
@@ -1003,9 +1003,10 @@
 		if (unlikely(cur_freq != cpu_policy->cur)) {
 			struct cpufreq_freqs freqs;
 
-			printk(KERN_WARNING "Warning: CPU frequency is %u, "
-					"cpufreq assumed %u kHz.\n",
-					cur_freq, cpu_policy->cur);
+			if (!(cpufreq_driver->flags & CPUFREQ_PM_NO_WARN))
+				printk(KERN_WARNING "Warning: CPU frequency"
+				       "is %u, cpufreq assumed %u kHz.\n",
+				       cur_freq, cpu_policy->cur);
 
 			freqs.cpu = cpu;
 			freqs.old = cpu_policy->cur;