Re: Cleanups for powernow-k8

[Dave Jones <davej@redhat.com>]

On Tue, Jan 13, 2004 at 04:37:13PM -0600, paul.devriendt@amd.com wrote:

 > I have a totally new driver, that I am hoping to release within about
 > a month. (I did target the end of the year, but I got distracted on
 > some other stuff). The new driver :
 >   - uses ACPI to figure out the available p-states. I have seen a *lot*
 >     of buggy BIOSs where the PSB/PST info is wrong or missing,

I've seen a ridiculous amount of broken PST's from folks running the K7 driver
too. Given the complete lack of help from some vendors[1], I think I might
add a minimal ACPI parser there too when I get time, as an alternative
source of info when the PST is obviously crap.

 > I would appreciate some advice on a question ... should I leave the old
 > non-ACPI capability there for those people who do not want to enable ACPI
 > in the kernel ? If so, is this a big ifdef, or is there a better way to do
 > it ? Or should I just say that it is dependent on ACPI, got to have ACPI ?

Part of the justification for cpufreq (at least on x86) was an alternative
for when ACPI just doesn't work, or for when folks either don't want to,
or can't run ACPI (through various other AML bugs for eg).

For minimal parsing of the ACPI P state tables, we shouldn't need the
full-blown interpretor IMO.

		Dave