Commit 22569848 authored by Dave Jones's avatar Dave Jones Committed by Dave Jones

[CPUFREQ] Fix cosmetic issue in powernow-k8 error handling

I tried to load the new SMP capable powernow k8 driver on a machine
with an too old BIOS. I got a lot of cryptic "init not cpu 0"
messages until the "MP systems not supported by PSB ..." message.

This patch switches the order of the checks around to print the
other message first. It will be still printed multiple times
because the cpufreq setup will probe all CPUs, but that's probably
ok.

From: Andi Kleen <ak@muc.de>
Signed-off-by: default avatarDave Jones <davej@redhat.com>
parent a4d3102b
...@@ -948,13 +948,13 @@ static int __init powernowk8_cpu_init(struct cpufreq_policy *pol) ...@@ -948,13 +948,13 @@ static int __init powernowk8_cpu_init(struct cpufreq_policy *pol)
* an UP version, and is deprecated by AMD. * an UP version, and is deprecated by AMD.
*/ */
if (pol->cpu != 0) { if ((num_online_cpus() != 1) || (num_possible_cpus() != 1)) {
printk(KERN_ERR PFX "init not cpu 0\n"); printk(KERN_INFO PFX "MP systems not supported by PSB BIOS structure\n");
kfree(data); kfree(data);
return -ENODEV; return -ENODEV;
} }
if ((num_online_cpus() != 1) || (num_possible_cpus() != 1)) { if (pol->cpu != 0) {
printk(KERN_INFO PFX "MP systems not supported by PSB BIOS structure\n"); printk(KERN_ERR PFX "init not cpu 0\n");
kfree(data); kfree(data);
return -ENODEV; return -ENODEV;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment