[PATCH] Ubuntu 7.10 PowerNow! issue with future AMD processors

[Mark Langsdorf]

On Tuesday 27 November 2007, Tim Gardner wrote:
> Langsdorf, Mark wrote:
> > The PowerNow! driver included in Ubuntu 7.10
> > has a flaw that prevents it from loading on
> > AMD family 0x11 processors (which have not
> > been released).  AMD considers Ubuntu 7.10 an
> > important software platform for family 0x11
> > processors.  We would like to get a patch
> > added so that the driver can work.


The equation to find the frequency given the fid and did is family dependant.

Acked-by: Mark Langsdorf <mark.langsdorf at amd.com>
Signed-off-by: Joachim Deguara <joachim.deguara at amd.com>

--
Index: 2.6-git/arch/i386/kernel/cpu/cpufreq/powernow-k8.c
===================================================================
--- 2.6-git.orig/arch/i386/kernel/cpu/cpufreq/powernow-k8.c
+++ 2.6-git/arch/i386/kernel/cpu/cpufreq/powernow-k8.c
@@ -76,7 +76,10 @@ static u32 find_khz_freq_from_fid(u32 fi
 /* Return a frequency in MHz, given an input fid and did */
 static u32 find_freq_from_fiddid(u32 fid, u32 did)
 {
-	return 100 * (fid + 0x10) >> did;
+	if (current_cpu_data.x86 == 0x10)
+		return 100 * (fid + 0x10) >> did;
+	else
+		return 100 * (fid + 0x8) >> did;
 }
 
 static u32 find_khz_freq_from_fiddid(u32 fid, u32 did)