[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#394971: updated patch, reordered modprobes accordying to depmod and fixed typo.



Hi,

Find attached a new version of the patch. After further investigation, i
noticed a typo which escaped me at first, and which stopped i2c-macintosh from
getting loaded. Fixed in this version of the patch.

Furthermore, i reordered the various windfarm related modules, in order to
have them loaded accordying to their depmod ordering.

On hardware where windfarm is not supported, the core windfarm modules will
not load, and thus the later module will spit out some harmless garbage about
missing symbols, which is not nice.

Maybe the script should be changed to test the successfull modprobing of the
parent modules in the depmod tree, and only load childrens if the parents
where successfull. But this is mostly cosmetic, and will not change the
functionality in any way.

Friendly,

Sven Luther
Index: debian/changelog
===================================================================
--- debian/changelog	(revision 42042)
+++ debian/changelog	(working copy)
@@ -1,3 +1,12 @@
+rootskel (1.42) UNRELEASED; urgency=low
+
+  [ Sven Luther ]
+  * Added S05fancontrol-linux-powerpc, in order to actually load the
+    fancontrol modules, in order to not have G5 apple box go into aircraft
+    noise level a few minutes after the start of the installation.
+
+ -- Sven Luther <sven@tael.powerlinux.fr>  Mon, 23 Oct 2006 20:13:59 +0200
+
 rootskel (1.41) unstable; urgency=low
 
   * Rebuild against klibc 1.4.29-1 to make cpio in rootskel-bootfloppy work
Index: src/lib/debian-installer-startup.d/S05fancontrol-linux-powerpc
===================================================================
--- src/lib/debian-installer-startup.d/S05fancontrol-linux-powerpc	(revision 0)
+++ src/lib/debian-installer-startup.d/S05fancontrol-linux-powerpc	(revision 0)
@@ -0,0 +1,13 @@
+# Load fan control modules, to stop the fans to go into aircraft-db levels
+modprobe i2c-powermac >/dev/null 2>1 || true
+modprobe therm_pm72 >/dev/null 2>1 || true
+modprobe windfarm_core >/dev/null 2>1 || true
+modprobe windfarm_cpufreq_clamp >/dev/null 2>1 || true
+modprobe windfarm_lm75_sensor >/dev/null 2>1 || true
+modprobe windfarm_max6690_sensor >/dev/null 2>1 || true
+modprobe windfarm_pid >/dev/null 2>1 || true
+modprobe windfarm_pm81 >/dev/null 2>1 || true
+modprobe windfarm_pm91 >/dev/null 2>1 || true
+modprobe windfarm_smu_sat >/dev/null 2>1 || true
+modprobe windfarm_pm112 >/dev/null 2>1 || true
+modprobe windfarm_smu_sensors >/dev/null 2>1 || true
Index: src/lib/debian-installer-startup.d/Makefile
===================================================================
--- src/lib/debian-installer-startup.d/Makefile	(revision 42042)
+++ src/lib/debian-installer-startup.d/Makefile	(working copy)
@@ -32,7 +32,9 @@
   
   ifeq ($(DEB_HOST_ARCH_CPU),powerpc)
     files += \
-	S45keyboard-linux-powerpc
+	S05fancontrol-linux-powerpc \
+	S45keyboard-linux-powerpc \
+	S50directfb-linux-powerpc
   endif
   
   ifneq (,$(filter mips mipsel,$(DEB_HOST_ARCH_CPU)))

Reply to: