diff options
author | Thomas Renninger <trenn@suse.de> | 2009-02-04 05:56:11 -0500 |
---|---|---|
committer | Dave Jones <davej@redhat.com> | 2009-02-24 22:47:31 -0500 |
commit | 79cc56af9fdbeaa91f50289b932d0959b41f9467 (patch) | |
tree | ae2f0e9ab28accb1accc047633bd1e5b613173ca /arch/x86/kernel/cpu/cpufreq | |
parent | 112124ab0a9f507a0d7fdbb1e1ed2b9a24f8c4ea (diff) |
[CPUFREQ] powernow-k8: Only print error message once, not per core.
This is the typical message you get if you plug in a CPU
which is newer than your BIOS. It's annoying seeing this
message for each core.
Signed-off-by: Thomas Renninger <trenn@suse.de>
Signed-off-by: Dave Jones <davej@redhat.com>
Diffstat (limited to 'arch/x86/kernel/cpu/cpufreq')
-rw-r--r-- | arch/x86/kernel/cpu/cpufreq/powernow-k8.c | 21 |
1 files changed, 15 insertions, 6 deletions
diff --git a/arch/x86/kernel/cpu/cpufreq/powernow-k8.c b/arch/x86/kernel/cpu/cpufreq/powernow-k8.c index acc06b03194e..c44853fc827a 100644 --- a/arch/x86/kernel/cpu/cpufreq/powernow-k8.c +++ b/arch/x86/kernel/cpu/cpufreq/powernow-k8.c | |||
@@ -794,7 +794,7 @@ static int find_psb_table(struct powernow_k8_data *data) | |||
794 | * BIOS and Kernel Developer's Guide, which is available on | 794 | * BIOS and Kernel Developer's Guide, which is available on |
795 | * www.amd.com | 795 | * www.amd.com |
796 | */ | 796 | */ |
797 | printk(KERN_ERR PFX "BIOS error - no PSB or ACPI _PSS objects\n"); | 797 | printk(KERN_ERR FW_BUG PFX "No PSB or ACPI _PSS objects\n"); |
798 | return -ENODEV; | 798 | return -ENODEV; |
799 | } | 799 | } |
800 | 800 | ||
@@ -1218,6 +1218,7 @@ static int __cpuinit powernowk8_cpu_init(struct cpufreq_policy *pol) | |||
1218 | struct powernow_k8_data *data; | 1218 | struct powernow_k8_data *data; |
1219 | cpumask_t oldmask; | 1219 | cpumask_t oldmask; |
1220 | int rc; | 1220 | int rc; |
1221 | static int print_once; | ||
1221 | 1222 | ||
1222 | if (!cpu_online(pol->cpu)) | 1223 | if (!cpu_online(pol->cpu)) |
1223 | return -ENODEV; | 1224 | return -ENODEV; |
@@ -1240,11 +1241,19 @@ static int __cpuinit powernowk8_cpu_init(struct cpufreq_policy *pol) | |||
1240 | * an UP version, and is deprecated by AMD. | 1241 | * an UP version, and is deprecated by AMD. |
1241 | */ | 1242 | */ |
1242 | if (num_online_cpus() != 1) { | 1243 | if (num_online_cpus() != 1) { |
1243 | printk(KERN_ERR FW_BUG PFX "Your BIOS does not provide" | 1244 | /* |
1244 | " ACPI _PSS objects in a way that Linux " | 1245 | * Replace this one with print_once as soon as such a |
1245 | "understands. Please report this to the Linux " | 1246 | * thing gets introduced |
1246 | "ACPI maintainers and complain to your BIOS " | 1247 | */ |
1247 | "vendor.\n"); | 1248 | if (!print_once) { |
1249 | WARN_ONCE(1, KERN_ERR FW_BUG PFX "Your BIOS " | ||
1250 | "does not provide ACPI _PSS objects " | ||
1251 | "in a way that Linux understands. " | ||
1252 | "Please report this to the Linux ACPI" | ||
1253 | " maintainers and complain to your " | ||
1254 | "BIOS vendor.\n"); | ||
1255 | print_once++; | ||
1256 | } | ||
1248 | kfree(data); | 1257 | kfree(data); |
1249 | return -ENODEV; | 1258 | return -ENODEV; |
1250 | } | 1259 | } |