diff options
| author | Andre Przywara <andre.przywara@amd.com> | 2012-09-04 08:28:05 +0000 | 
|---|---|---|
| committer | Rafael J. Wysocki <rjw@sisk.pl> | 2012-09-09 22:04:53 +0200 | 
| commit | a2060958569a8e8e36202c373a55c458ff84856d (patch) | |
| tree | 794f7b0536895b26ace84762cbffb4ddf21ed2a2 | |
| parent | 034be8fd645bb77ca623036dc24c790b6cc803e2 (diff) | |
| download | olio-linux-3.10-a2060958569a8e8e36202c373a55c458ff84856d.tar.xz olio-linux-3.10-a2060958569a8e8e36202c373a55c458ff84856d.zip  | |
powernow-k8: delay info messages until initialization has succeeded
powernow-k8 is quite prematurely crying Hooray and outputs diagnostic
messages, although the actual initialization can still fail.
Since now we may have acpi-cpufreq already loaded, we move the
messages at the end of the init routine to avoid confusing output
if the loading of powernow-k8 should not succeed.
Signed-off-by: Andre Przywara <andre.przywara@amd.com>
Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
| -rw-r--r-- | drivers/cpufreq/powernow-k8.c | 24 | 
1 files changed, 14 insertions, 10 deletions
diff --git a/drivers/cpufreq/powernow-k8.c b/drivers/cpufreq/powernow-k8.c index e2defb822bb..f1035a920b0 100644 --- a/drivers/cpufreq/powernow-k8.c +++ b/drivers/cpufreq/powernow-k8.c @@ -1576,9 +1576,6 @@ static int __cpuinit powernowk8_init(void)  	if (supported_cpus != num_online_cpus())  		return -ENODEV; -	printk(KERN_INFO PFX "Found %d %s (%d cpu cores) (" VERSION ")\n", -		num_online_nodes(), boot_cpu_data.x86_model_id, supported_cpus); -  	if (boot_cpu_has(X86_FEATURE_CPB)) {  		cpb_capable = true; @@ -1597,16 +1594,23 @@ static int __cpuinit powernowk8_init(void)  			struct msr *reg = per_cpu_ptr(msrs, cpu);  			cpb_enabled |= !(!!(reg->l & BIT(25)));  		} - -		printk(KERN_INFO PFX "Core Performance Boosting: %s.\n", -			(cpb_enabled ? "on" : "off"));  	}  	rv = cpufreq_register_driver(&cpufreq_amd64_driver); -	if (rv < 0 && boot_cpu_has(X86_FEATURE_CPB)) { -		unregister_cpu_notifier(&cpb_nb); -		msrs_free(msrs); -		msrs = NULL; + +	if (!rv) +		pr_info(PFX "Found %d %s (%d cpu cores) (" VERSION ")\n", +			num_online_nodes(), boot_cpu_data.x86_model_id, +			supported_cpus); + +	if (boot_cpu_has(X86_FEATURE_CPB)) { +		if (rv < 0) { +			unregister_cpu_notifier(&cpb_nb); +			msrs_free(msrs); +			msrs = NULL; +		} else +			pr_info(PFX "Core Performance Boosting: %s.\n", +				(cpb_enabled ? "on" : "off"));  	}  	return rv;  }  |