On Fri, May 27, 2011 at 1:02 AM, Kevin Hilman <khilman@xxxxxx> wrote: > Inspired by Jean's work to move PM code to modules, I decided it's > time to remove a bunch of ugly and difficult to maintain code from PM > debug. Indeed this code is a spaghetti mess and makes pm-debug difficult to be modularized. > The main chunk here is removing the register dump features for > OMAP2/OMAP3 which are awful to read, and impossible to scale for > OMAP4+. Also, there are now some userspace /dev/mem-based tools (like > omapconf) that can handle this type of thing much better, and with a > much better interface and display. Agree. However we still need the equivalent of [1] to retrieve the PRCM registers just before and after the WFI instructions. I would like to propose some replacement when the current code clean-up is settled down a bit. [1] http://git.kernel.org/?p=linux/kernel/git/khilman/linux-omap-pm.git;a=commitdiff;h=5dd36389eb6ef4837a2aad128e4bd0226468c171 Regards, Jean > Series applies to v2.6.39. > > Kevin Hilman (4): > OMAP3: PM debug: remove sleep_while_idle feature > OMAP2: PM debug: remove register dumping > OMAP3: PM debug: remove register dumping > OMAP2: PM debug: move wakeup timer into clockevent code > > arch/arm/mach-omap2/pm-debug.c | 362 ----------------------------- > arch/arm/mach-omap2/pm.h | 9 - > arch/arm/mach-omap2/pm24xx.c | 6 +- > arch/arm/mach-omap2/pm34xx.c | 6 - > arch/arm/mach-omap2/timer-gp.c | 33 +++- > arch/arm/plat-omap/include/plat/dmtimer.h | 1 - > 6 files changed, 32 insertions(+), 385 deletions(-) > > -- > 1.7.4 > > -- > 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 > -- 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
