On Nov 5, 2009, at 1:48 PM, Scott Wood wrote:

Kumar Gala wrote:
On Nov 5, 2009, at 10:57 AM, Scott Wood wrote:
Kumar Gala wrote:
On Sep 23, 2009, at 2:01 PM, Anton Vorontsov wrote:
Currently 83xx PMC driver clears deep_sleeping variable very early, before devices are resumed. This makes fsl_deep_sleep() unusable in
drivers' resume() callback.

Sure, drivers can store fsl_deep_sleep() value on suspend and use
the stored value on resume. But a better solution is to postpone
clearing the deep_sleeping variable, i.e. move it into finish()
callback.

Signed-off-by: Anton Vorontsov <avoront...@ru.mvista.com>
---
arch/powerpc/platforms/83xx/suspend.c |    4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
Scott, any comments or an ack?

ACK
thanks, is that an ACK for all 3?

The first.  Patch 2 looks OK as well.

As for patch 3, Ben objected to the sleep-nexus stuff on IRC.

Is sleep-nexus new?  I thought we've had that for a bit.

- k
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Reply via email to