Re: [PATCH/RFC 0/4] OMAP: PM debug: remove register dump, misc cleanups

[Jean Pihet <jean.pihet@newoldbits.com>]

On Fri, May 27, 2011 at 1:02 AM, Kevin Hilman <khilman@ti.com> 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@vger.kernel.org
> 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@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html