On Thu, Sep 08, 2011 at 08:47:18AM +0100, Russell King - ARM Linux wrote:

On Thu, Sep 08, 2011 at 02:23:02PM +0800, Shawn Guo wrote:

quoted

Yes, if you lose L2 power in suspend, it's the easiest way for you to
resume L2.

For my case which retains L2, I actually do not want to call this
function which will invalidate L2.  But since we still have problem
to use rmk's generic suspend/resume updates (ARM: pm: add L2 cache
cleaning for suspend), we have to flush the entire L2 on imx6q for now.

OMAP44xx has the same problem but they re-initialize L2 on resume in
their pre-cpu_resume() assembly code.

Are you suggesting that this might be the reason why imx6q has problem
with your patch?  Are we supposed to re-initialize L2 before calling
into generic cpu_resume()?

-- 
Regards,
Shawn