From mboxrd@z Thu Jan  1 00:00:00 1970
From: marek.vasut@gmail.com (Marek Vasut)
Date: Sat, 12 Mar 2011 12:06:13 +0100
Subject: [PATCH 5/8] ARM: PXA: Z2: Add poweroff function
In-Reply-To: <201103121006.12614.anarsoul@gmail.com>
References: <1299839023-16601-1-git-send-email-anarsoul@gmail.com>
	<201103120105.15474.marek.vasut@gmail.com>
	<201103121006.12614.anarsoul@gmail.com>
Message-ID: <201103121206.13251.marek.vasut@gmail.com>
To: linux-arm-kernel@lists.infradead.org
List-Id: linux-arm-kernel.lists.infradead.org

On Saturday 12 March 2011 09:06:12 Vasily Khoruzhick wrote:
> On Saturday 12 March 2011 02:05:15 Marek Vasut wrote:
> > > Well, it looks like memory content is not preserved, or something wrong
> > > with bootloader. Anyway, it does not resume after deepsleep for some
> > > reason, I can't check better, as I have no JTAG. Maybe you can
> > > investigate this problem?
> > 
> > But resume after deepsleep is unrelated to this patch.
> 
> I just want to say that without this patch you can't keep device in low-
> powermode, and it can drain full battery in ~week when it's in sleep mode.

You have a problem with suspend then ? That'd need a different solution.

> 
> > > No need, I understand how it works, PSPR is null, so after u-boot loads
> > > PSPR to pc it performs 'soft reset' (jumps to 0x0, to begin of NOR
> > > flash), RCSR bits are already cleared, so it just boots kernel.
> > 
> > The note in Table 18 -- Exit from sleep or deep-sleep doesn't clear or
> > set this bit -- means PSPR might not be null if you use your
> > deep-sleep-parapowerdown.
> 
> Ok, I can clear PSPR before going to deepsleep, would it be enough?

As Eric pointed out, the wake-up scenario is bootloader-specific ... taking into 
acount only your own version of bootloader is a no-no
> 
> Regards
> Vasily