On 6/23/2011 2:31 AM, Russell King - ARM Linux wrote:
On Wed, Jun 22, 2011 at 04:08:16PM +0100, Russell King - ARM Linux wrote:
Tested on Assabet (SA1100) and 3430LDP only.
Correction - because suspend only goes into retention mode, these
changes have not been tested on the 3430. Someone who knows what
they're doing with the mega-complicated OMAPs (and so knows how to
trigger the required modes to test these code paths) needs to test
this.
It will currently fail when trying to call cpu_resume() because we're
trying the virtual address for that function, whereas it needs to be
the physical address. That's left as an exercise to solve (easiest
way is probably to pass virt_to_phys(cpu_resume) into _omap_sram_idle
and get it to save that away in SRAM.)
Firstly thanks for taking time to include OMAP this time. Even if the
changes are not tested, we can get them working with necessary fixes.
Have looked at your series briefly and have some comments for the same.
Am planning to test these patches, but it might take bit more time.
Will update the results as soon as I have them.
Regards
Santosh
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
