Package: cpufrequtils
Version: 004-2
Severity: normal

Hi,
actually if the user specify FREQDRIVER in /etc/default/loadcpufreq and this 
one is unloadable (because of a typo or because doesn't match the real 
device) no cpu frequency driver are loaded at all.
I attach the patch that make it possible to load the 
MODULE_FALLBACK=acpi-cpufreq instead.
This is very usefull for me because i have two kernel image to load: one with 
a patched version of speedsteep-centrino that load fine (have set 
FREQDRIVER="speedsteep-centrino" in /etc/default/loadcpufreq) and one with 
upstream version that doesn't work in my box: booting this last kernel image 
acpi-cpufreq is loaded instead.

Cheers,
Andrea. 
--- cpufrequtils-004/debian/cpufrequtils.loadcpufreq.init	2008-08-22 20:25:32.000000000 +0200
+++ cpufrequtils-004/debian/cpufrequtils.loadcpufreq.init	2008-08-22 20:43:16.000000000 +0200
@@ -54,7 +54,6 @@
     CPU_FAMILY=$(sed -e '/^cpu family/ {s/.*: //;p;Q};d' $CPUINFO)
 
     MODULE=
-    MODULE_FALLBACK=acpi-cpufreq
 
     # Two modules for PIII-M depending the chipset.
     # modprobe speedstep-ich$EXT || modprobe speestep-smi$EXT
@@ -183,22 +182,22 @@
         fi
 
         #new style detection system
+        MODULE_FALLBACK=acpi-cpufreq
         if [ ! "$FREQDRIVER" = "" ]; then 
 		# user overridden value in /etc/default/loadcpufreq
-                modprobe "$FREQDRIVER"
 		MODULE="$FREQDRIVER"
         else
 	        load_detected_cpufreq_modules
-		if [ ! -z "$MODULE" ] || [ ! -z "$MODULE_FALLBACK" ] ; then
-		    if [ ! -z "$MODULE" ] && modprobe "$MODULE" 2>/dev/null ; then
-			:
-		    elif modprobe "$MODULE_FALLBACK" 2>/dev/null ; then
-			MODULE="$MODULE_FALLBACK"
-		    else
-			unset MODULE
-		    fi
-		fi
-        fi
+	fi
+	if [ ! -z "$MODULE" ] || [ ! -z "$MODULE_FALLBACK" ] ; then
+	    if [ ! -z "$MODULE" ] && modprobe "$MODULE" 2>/dev/null ; then
+		:
+	    elif modprobe "$MODULE_FALLBACK" 2>/dev/null ; then
+		MODULE="$MODULE_FALLBACK"
+	    else
+		unset MODULE
+	    fi
+	fi
 
         echo "$PRINTK" > /proc/sys/kernel/printk
 }

Reply via email to