Re: Voltage tweaking in powernow_k8

[Roberto Gordo Saez <roberto.gordo@gmail.com>]

On Mon, Apr 16, 2007 at 08:55:38AM -0500, Langsdorf, Mark wrote:
> AMD policy is that the ACPI values must be used.  I
> can't support a patch that overrides the ACPI values.
> 
> I have no objection to it being in the code, but I
> can't support it.

Hi, thanks for replying.

I understand your position about this. Please, correct me if I'm
giving a wrong interpretation to your message; in the way I read it,
you are unable to support those changes (because of your position at
AMD you are required to strictly follow the policy), but you could
live with those changes provided that your name is not attached to
them...

Admittely, it is not such an elegant solution, but people are indeed
doing other hacks anyway, like statically overriding the ACPI table
with a hacked DSDT at kernel compile time (distributions like gentoo
have HOWTOs for this). I'm not sure if this would be a preferred
approach, but it does look even uglier to me. If you have any
suggestion or alternative proposal, I would be interested in your
opinion.

It may be a good idea to print warnings when the parameter is used,
to clearly state the unsupported and potentally dangerous effect of
the parameter (though it would not be enough to change your view,
I guess...).