Skip to contentSkip to conversation

Longhaul - Fix multipliers table

PATCH In review Rafał Bilski 2 participants 2 messages Jul 14, 2006

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Longhaul - Fix multipliers table

This patch fix multiplier table. Some values are swaped and 9.0*133MHz processor is incorrecly reported as 5.0*133MHz.

---

arch/i386/kernel/cpu/cpufreq/longhaul.c renamed from arch/i386/kernel/cpu/cpufreq/longhaul.c.orig +4 -2
@@ -323,8 +323,10 @@ static int __init longhaul_get_ranges(vo
 {
 	unsigned long invalue;
 	unsigned int multipliers[32]= {
-		50,40,30,100,55,35,45,95,90,70,80,60,120,75,85,65,
-		-1,110,120,-1,135,115,125,105,130,150,160,140,-1,155,-1,145 };
+		 90,  30,  40, 100,  55,  35,  45,  95,
+		 50,  70,  80,  60, 120,  75,  85,  65,
+		 -1, 110, 120,  -1, 135, 115, 125, 105,
+		130, 150, 160, 140,  -1, 155,  -1, 145 };
 	unsigned int j, k = 0;
 	union msr_longhaul longhaul;
 	unsigned long lo, hi;

----------------------------------------------------------------------
Upal jakby mniejszy... http://link.interia.pl/f1975

#1 lore raw
Quoted 3 lines · RafaÅ‚ Bilski

This patch fix multiplier table. Some values are swaped and 9.0*133MHz processor is incorrecly reported as 5.0*133MHz.

Wow, not sure what happened there. Looks like there was some confusion over which CPU this table was for.

Care to rename multipliers to ezra_t_multipliers or c5m_multipliers at the same time to ease the confusion?

Dave

Signature · 1 line
http://www.codemonkey.org.uk