Re: powernow-k8: support acpi

[Pavel Machek <pavel@ucw.cz>]

Hi!

>  > > I'm about to disappear for a day or two. I'll look into integrating whats
>  > > outstanding on Sunday.  Pavel, if you have time between now and then
>  > > to do a diff of whats remaining between your tree and the driver I posted
>  > > this afternoon, that would make my life a little easier.
>  > 
>  > Here it is: (I was not able to locate your changes, so it is possible
>  > I'm reverting some of those; but few extra inlines should not hurt
>  > that bad :-).
> 
> Ok, the merge-from-hell is complete (split up into 23 csets).
> The only diffs between your tree and mine should now be trivial
> whitespace changes, and some small bits that looked bogus.

Nice job! I thought it is impossible to split at all.

> One thing outstanding that probably does need fixing. Andi Kleen
> mentioned on IRC yesterday ..
> 
> <f> found a funny bug in the powernow-k8 acpi changes
> <f> when  acpi is disabled processor.ko won't load, but the new module needs symbols from that
> <f> i added a hack to keep it loaded anyways

I thought about it... Would it be fair to make ACPI support
in powernow-k8 depend on ACPI_PROCESSOR=y?

We used to have no ACPI support at all, so it still would be strict improvement
(and unlike other solutions it is one-liner).

				Pavel 
-- 
64 bytes from 195.113.31.123: icmp_seq=28 ttl=51 time=448769.1 ms