Re: [PATCH][powernow-k7] correcting SGTC. Timer is based upon FSB.

[Dave Jones <davej@redhat.com>]

On Wed, Apr 14, 2004 at 03:28:54PM +0200, Bruno Ducrot wrote:
 > On Wed, Apr 14, 2004 at 02:18:57PM +0200, Bruno Ducrot wrote:
 > > * Integrate acpi perflib from Dominik.
 > > * Use acpi if the PST tables are known to be broken (ASUS Aspire match one PST,
 > >   but give brain damaged values), or use ACPI if no PST found.
 > > 
 > >  arch/i386/kernel/cpu/cpufreq/powernow-k7.c |  180 +++++++++++++++++++++++++++--
 > >  1 files changed, 168 insertions(+), 12 deletions(-)
 > > 
 > > --- linux-dj-cpufreq/arch/i386/kernel/cpu/cpufreq/powernow-k7.c	2004/04/14 11:49:44	1.3
 > > +++ linux-dj-cpufreq/arch/i386/kernel/cpu/cpufreq/powernow-k7.c	2004/04/14 12:11:24
 > ....
 > 
 > > @@ -341,8 +480,14 @@ static int powernow_decode_bios (int max
 > >  			}
 > >  			printk (KERN_INFO PFX "No PST tables match this cpuid (0x%x)\n", etuple);
 > >  			printk (KERN_INFO PFX "This is indicative of a broken BIOS.\n");
 > > -			printk (KERN_INFO PFX "See http://www.codemonkey.org.uk/projects/cpufreq/powernow-k7.shtml\n");
 > > -			return -EINVAL;
 > > +
 > > +			printk (KERN_INFO PFX "Trying ACPI perflib\n");
 > > +			ret = powernow_acpi_init();
 > > +			if (ret) {
 > > +				printk (KERN_INFO PFX "ACPI and legacy methods failed\n");
 > > +				printk (KERN_INFO PFX "See http://www.codemonkey.org.uk/projects/cpufreq/powernow-k7.shtml\n");
 > > +			}
 > > +			return ret;
 > >  		}
 > >  		p++;
 > >  	}
 > 
 > 
 > This chunk is wrong, because I handle that at powernow_cpu_init() now.
 > Just removing that one should be OK.  Do you want I resend the whole
 > patch, or only that part ?

oops. just send me a patch backing out that bit..

		Dave