[PATCH v3 2/2] ARM: uncompress debug support for multiplatform build

Shawn Guo shawn.guo at linaro.org
Fri Feb 8 07:45:23 EST 2013


On Fri, Feb 08, 2013 at 12:27:14PM +0000, Russell King - ARM Linux wrote:
> On Fri, Feb 08, 2013 at 07:44:52PM +0800, Shawn Guo wrote:
> > Instead of giving zero support of uncompress debug for multiplatform
> > build, the patch 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.
> > 
> > It reuses the platform DEBUG_LL macros by creating a simple
> > arch/arm/boot/compressed/debug.S with CONFIG_DEBUG_LL_INCLUDE
> > included there, and implements a generic putc() using those macros.
> > 
> > Signed-off-by: Shawn Guo <shawn.guo at linaro.org>
> > Acked-by: Olof Johansson <olof at lixom.net>
> > ---
> > Changes since v2:
> >  * Build arch/arm/boot/compressed/debug.S only for multiplatform kernel
> >    to fix the breakage on traditional platform build
> >  * Include asm/assembler.h in arch/arm/boot/compressed/debug.S, as it
> >    may commonly be needed by platforms to be converted to multiplatform
> > 
> > Russell,
> > 
> > I will put it into patch system if I do not receive any comment
> > tomorrow.
> 
> This doesn't address the problem with OMAP, which is that the OMAP debug
> code places data into the .data section, and that is not allowed in the
> decompressor.

My patches are based on v3.8-rc where OMAP hasn't been converted to
multiplatform.  But you are right, when the patches get into linux-next
it will cause problem for OMAP, since multiplatform is enabled for OMAP
on linux-next.  So should I base my patches on linux-next and fold a
fix for OMAP into the series?

Shawn




More information about the linux-arm-kernel mailing list