On 6/24/2011 5:46 AM, Kevin Hilman wrote:
Upon return from off-mode, the ROM code jumps to a restore function saved in the scratchpad. Based on SoC revision or errata, this restore entry point is different. Current code uses some helper functions in sleep34xx.S (get_*_restore_pointer) to get the restore function entry point. When returning from off-mode, this code is executed from SDRAM, so there's no reason to use these helper functions when using the SDRAM entry points directly would work just fine. This patch uses ENTRY/ENDPROC to create "real" entry points for these functions, and uses those values directly when writing the scratchpad. Tested all three entry points - restore_es3: 3430/n900 - restore_3630: 3630/Zoom3 - restore: 3530/Overo Suggested-by: Russell King<rmk+kernel@xxxxxxxxxxxxxxxx> Cc: Jean Pihet<jean.pihet@xxxxxxxxxxxxxx> Signed-off-by: Kevin Hilman<khilman@xxxxxx> ---
This is a nice change. In general now omap3 suspend code is looking much cleaner and after Jean's DDR patch, it would be even better. For this patch, my ack if you need one Acked-by: Santosh Shilimkar <santosh.shilimkar@xxxxxx> -- 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
