Re: [PATCH] amd76x_pm: C3 powersaving for AMD K7

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



* Andrew Morton <[email protected]> [060131 19:35]:
> Joerg Sommrey <[email protected]> wrote:
> >
> > This is a processor idle module for AMD SMP 760MP(X) based systems.
> >  The patch was originally written by Tony Lindgren and has been around
> >  since 2002.  It enables C2 mode on AMD SMP systems and thus saves
> >  about 70 - 90 W of energy in the idle mode compared to the default idle
> >  mode.  The idle function has been rewritten and is now free of locking
> >  issues and is independent from the number of CPUs.  The impact
> >  from this module on the system clock and on i/o transfer are now fairly
> >  low.
> > 
> >  This patch can also be found at
> >  http://www.sommrey.de/amd76x_pm/amd76x_pm-2.6.15-4.patch
> > 
> >  In this version more locking was added to make sure all or no CPU enter
> >  C3 mode.
> > 
> >  Signed-off-by: Joerg Sommrey <[email protected]>
> 
> Thanks.  I'll merge this into -mm and shall plague the ACPI guys with it. 
> They have said discouraging things about board-specific drivers in the
> past.  We shall see.
> 
> Tony, could you please review section 11 of Documentation/SubmittingPatches
> and if OK, send a Signed-off-by:?

Sure, here you are:

Signed-off-by: Tony Lindgren <[email protected]>
 
> Some minor pointlets, and a bug:
> 
> 
> Should CONFIG_AMD76X_PM depend on ACPI?

It works also without ACPI.

But in the long run something like this could be added for ACPI to allow
easy addition of custom PM modules. That was discussed briefly on the ACPI
list and people seemed to be OK with adding system for custom PM modules.

And if it was integrated with ACPI, only enabling the C states should be
enough, and then ACPI would just know how to use the C states...

Regards,

Tony
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux