Re: [PATCH] power-management elements for 603e/fsl (version 2)

[Guennadi Liakhovetski <g.liakhovetski@gmx.de>]

On Mon, 21 May 2007, Guennadi Liakhovetski wrote:

> On Mon, 21 May 2007, Scott Wood wrote:
> 
> > IRQs really shouldn't be enabled here without something like the "Handle
> > HID0_SLEEP in the TLF_NAPPING hack" patch I posted recently.
> 
> Ok, I'd be happy to rely on those your patches... as long as they make it 
> into the kernel. Paulus, what's the plan? I'll try to apply them locally 
> for now and adjust my code accordingly. A recent powerpc.git should be 
> fine, I guess - the arch_suspend_{disable,enable}_irqs hooks are already 
> there, or are there any further dependencies?

Took some time, but now I did rebase my linkstation standby code on your 
patches, and also used your mpc83xx_enter_sleep / ret_from_sleep functions 
(former with a single modification - removed the first mtmsr to set EE and 
POW in MSR with one insn) - all worked. So, we could put them into 
something like arch/powerpc/sysdev/fsl_pm.S and name accordingly. That 
could be an incremental patch after the present versions make it into some 
tree.

Of course, I needed other your patches to, e.g., bypass calling the ISR on 
resume too.

Thanks
Guennadi
---
Guennadi Liakhovetski