Re: [PATCH 5/7] powerpc: Replace ppc_md.power_off with pm_power_off

2007-12-04 Thread Mark A. Greer
On Tue, Dec 04, 2007 at 06:23:09PM +1100, Benjamin Herrenschmidt wrote: On Mon, 2007-12-03 at 22:48 -0700, Mark A. Greer wrote: From: Mark A. Greer [EMAIL PROTECTED] The ppc_md.power_off hook performs the same function that the pm_power_off hook is supposed to. However, it is

Re: [PATCH 5/7] powerpc: Replace ppc_md.power_off with pm_power_off

2007-12-04 Thread Benjamin Herrenschmidt
On Tue, 2007-12-04 at 11:01 -0700, Mark A. Greer wrote: On Tue, Dec 04, 2007 at 06:23:09PM +1100, Benjamin Herrenschmidt wrote: On Mon, 2007-12-03 at 22:48 -0700, Mark A. Greer wrote: From: Mark A. Greer [EMAIL PROTECTED] The ppc_md.power_off hook performs the same function that

Re: [PATCH 5/7] powerpc: Replace ppc_md.power_off with pm_power_off

2007-12-04 Thread Grant Likely
On 12/4/07, Benjamin Herrenschmidt [EMAIL PROTECTED] wrote: On Tue, 2007-12-04 at 11:01 -0700, Mark A. Greer wrote: On Tue, Dec 04, 2007 at 06:23:09PM +1100, Benjamin Herrenschmidt wrote: On Mon, 2007-12-03 at 22:48 -0700, Mark A. Greer wrote: From: Mark A. Greer [EMAIL PROTECTED]

Re: [PATCH 5/7] powerpc: Replace ppc_md.power_off with pm_power_off

2007-12-04 Thread Benjamin Herrenschmidt
On Tue, 2007-12-04 at 13:05 -0700, Grant Likely wrote: We could simply have the setup code copy the ppc_md.power_off pointer into pm_power_off; that we retain the nice assignment in define_machine(), but eliminate the duplicated calls. Good idea. Ben.

Re: [PATCH 5/7] powerpc: Replace ppc_md.power_off with pm_power_off

2007-12-04 Thread Mark A. Greer
On Tue, Dec 04, 2007 at 01:05:46PM -0700, Grant Likely wrote: On 12/4/07, Benjamin Herrenschmidt [EMAIL PROTECTED] wrote: On Tue, 2007-12-04 at 11:01 -0700, Mark A. Greer wrote: On Tue, Dec 04, 2007 at 06:23:09PM +1100, Benjamin Herrenschmidt wrote: On Mon, 2007-12-03 at 22:48

[PATCH 5/7] powerpc: Replace ppc_md.power_off with pm_power_off

2007-12-03 Thread Mark A. Greer
From: Mark A. Greer [EMAIL PROTECTED] The ppc_md.power_off hook performs the same function that the pm_power_off hook is supposed to. However, it is powerpc-specific and prevents kernel drivers (e.g., IPMI) from changing how a platform is powered off. So, get rid of ppc_md.power_off and replace

Re: [PATCH 5/7] powerpc: Replace ppc_md.power_off with pm_power_off

2007-12-03 Thread Benjamin Herrenschmidt
On Mon, 2007-12-03 at 22:48 -0700, Mark A. Greer wrote: From: Mark A. Greer [EMAIL PROTECTED] The ppc_md.power_off hook performs the same function that the pm_power_off hook is supposed to. However, it is powerpc-specific and prevents kernel drivers (e.g., IPMI) from changing how a