On Sat, Feb 12, 2011 at 4:09 PM, Santosh Shilimkar
[off-list ref] wrote:

quoted

-----Original Message-----
From: Russell King - ARM Linux [mailto:linux at arm.linux.org.uk]
Sent: Saturday, February 12, 2011 8:20 PM
To: Santosh Shilimkar
Cc: Colin Cross; Kukjin Kim; saeed bishara; linux-arm-
kernel at lists.infradead.org
Subject: Re: [PATCH v3 2/5] ARM: pm: add generic CPU suspend/resume
support

On Fri, Feb 11, 2011 at 05:37:04PM +0530, Santosh Shilimkar wrote:

quoted

There is a Monitor secure API, needs to be called from non-secure
software to set this diagnostic registers in resume path.

It would be an idea to get the OMAP sleep code up to date so that I
can
look at OMAPs requirements for this to be useful.

As the current code stands, I don't see any reason why the sleep34xx
code
can't use this infrastructure, but I'm loathed to start modifying
that if there's outstanding code changes in that area.

Yep. There are few issues out there with sleep34xx code.
? ? ? ?- Secure APIs
? ? ? ?- Current code needs to be cleaned up to remove
? ? ? ?unwanted registers save restore
? ? ? ?- Some part of the code on OMAP3 must be run from
? ? ? ?SRAM. It can't run from DDR

This is not the highest priority for now, I will have to come back
later on this.

? ? ? ?- AUXCTLR, Diagnostic registers aren't accessible
? ? ? ?in secure mode.
? ? ? ?- L2 cache needs to be handled with secure APIs.
? ? ? ?- Code sequence needs to handle errata's handling
? ? ? ?which accesses OMAP PM registers.

Also there is some on-going work to support Thumb-2, adding Dave.

Few of the above are getting addressed for this merge window.

So my plan was to take a look at generic suspend after the
merge window. By that time your generic stuff and omap
cleanup would have got merged hopefully.

Agree!

Regards,
Santosh

Regards,
Jean

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel at lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel