[PATCH] cpufreq annoying warning fix
authorBenjamin Herrenschmidt <benh@kernel.crashing.org>
Mon, 2 May 2005 06:25:10 +0000 (16:25 +1000)
committerLinus Torvalds <torvalds@ppc970.osdl.org>
Mon, 2 May 2005 15:15:22 +0000 (08:15 -0700)
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 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 <benh@kernel.crashing.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
drivers/cpufreq/cpufreq.c

index 4fc0cb7..8e56131 100644 (file)
@@ -1003,9 +1003,10 @@ static int cpufreq_resume(struct sys_device * sysdev)
                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;