[PATCH v4 0/2] Uncompress debug for multiplatform

[Shawn Guo]

On Mon, Mar 11, 2013 at 11:30:02AM -0600, Stephen Warren wrote:
> On 03/11/2013 11:03 AM, Russell King - ARM Linux wrote:
> > On Mon, Mar 11, 2013 at 07:35:35PM +0800, Shawn Guo wrote:
> >> Instead of giving zero support of uncompress debug for multiplatform
> >> build, the series turns uncompress debug into one part of DEBUG_LL
> >> support.  When DEBUG_LL is turned on for a particular platform,
> >> uncompress debug works too for that platform.
> >>
> >> OMAP is an exception here.  OMAP low-level debug code places data
> >> in the .data section, and that is allowed in decompressor.  That's
> >> why Kconfig symbol DEBUG_UNCOMPRESS that controls multiplatform
> >> uncompress debug support is defined with !DEBUG_OMAP2PLUS_UART.
> >>
> >> Changes since v3:
> >>  - Create Kconfig symbol DEBUG_UNCOMPRESS for building in
> >>    multiplatform uncompress debug support, so that we can leave
> >>    out the platform which is part of multiplatform but not ready
> >>    for being part of uncompress debug like OMAP example.
> > 
> > Wasn't Tegra also having problems with this as well?
> 
> Tegra isn't converted to multi-platform yet. I hope to in 3.10.

Yeah, that's why I did not touch Tegra as the series is based on
v3.9-rc1 where Tegra hasn't been part of multi-platform yet.

But I just noticed that debug-macro.S for Tegra has been renamed to
debug/tegra.S, and we already know it will be broken when multi-platform
is enabled together with this series for Tegra.  I should probably just
take a further step to mark Tegra as another exception right now, to
ensure the series will not break thing when getting merged together
with Tegra multi-platform work.  Something like the following:

config DEBUG_UNCOMPRESS
        bool
        default y if ARCH_MULTIPLATFORM && DEBUG_LL && \
                     !DEBUG_OMAP2PLUS_UART && \
                     !DEBUG_TEGRA_UART


Shawn