Skip to contentSkip to conversation

make cpu_has_cpufreq() work

PATCH New Dirk Mueller 1 participant 1 message Dec 21, 2005 ACPI

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 make cpu_has_cpufreq() work

Hi,

it seems to me the return values of cpu_has_cpufreq in processor_thermal.c are reversed. all callers check for if(!cpu_has_cpufreq()) return -ENODEV, which indicates that it expects a return value of 0 for error, and nonzero for success.

However the current implementation gets that reversed. the patch below fixes it. Spotted by Thomas Renninger, and fixes the ACPI thermal code never actually speedstepping the CPU when it overheats.

processor_thermal.c 2005-12-21 00:18:23.000000000 +0100 renamed from processor_thermal.c.orig 2005-12-21 00:18:06.000000000 +0100 +2 -2
@@ -102,8 +102,8 @@ static int cpu_has_cpufreq(unsigned int
 {
        struct cpufreq_policy policy;
        if (!acpi_thermal_cpufreq_is_init || cpufreq_get_policy(&policy, cpu))
-               return -ENODEV;
-       return 0;
+               return 0;
+       return 1;
 }
 
 static int acpi_thermal_cpufreq_increase(unsigned int cpu)

-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files for problems? Stop! Download the new AJAX search engine that makes searching your log files as easy as surfing the web. DOWNLOAD SPLUNK!
http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click