Re: [PATCH -queue v2] MIPS: Cleanup the debugging of compressed kernel support

[Wu Zhangjin <wuzhangjin@gmail.com>]

On Tue, 2010-01-26 at 16:28 +0100, Manuel Lauss wrote:
> Hi,
> 
> On Tue, Jan 26, 2010 at 4:02 PM, Wu Zhangjin <wuzhangjin@gmail.com> wrote:
> > Changes from v1 (feedbacks from Ralf):
> >
> >  o make DEBUG_ZBOOT also depend on DEBUG_KERNEL
> >
> >  o DEBUG_ZBOOT already depends on SYS_SUPPORTS_ZBOOT_UART16550 so simplify 
> > the
> 
> Not every chip has a standard 16550, unfortunately.  I liked your
> first iteration better:
> DEBUG_ZBOOT visible at all times (or depend on DEBUG_KERNEL)  another
> (invisible)
> config symbol selecting the code to build (i.e. SYS_SUPPORTS_ZBOOT_UART16550
> for your loongson boxes, MACH_ALCHEMY for alchemy, and nothing for unsupported
> chips).
> 

Yes, It will be not convenient for a new serial port debugging support
with the current modification, for example, if you add UARTXXXX, we need
to select it for the new board and also add it under the "depends" of
DEBUG_ZBOOT. how about this?

diff --git a/arch/mips/Kconfig.debug b/arch/mips/Kconfig.debug
index 5e556f7..6fccbf0 100644
--- a/arch/mips/Kconfig.debug
+++ b/arch/mips/Kconfig.debug
@@ -104,7 +104,7 @@ config RUNTIME_DEBUG
 
 config DEBUG_ZBOOT
        bool "Enable compressed kernel support debugging"
-       depends on DEBUG_KERNEL && SYS_SUPPORTS_ZBOOT_UART16550
+       depends on DEBUG_KERNEL

and we keep the old:

ifdef CONFIG_DEBUG_ZBOOT
obj-$(CONFIG_SYS_SUPPORTS_ZBOOT_UART16550) += $(obj)/uart-16550.o
endif

If a new serial port is needed, we just need to add the following lines
to arch/mips/Kconfig

config SYS_SUPPORTS_ZBOOT_UARTXXX
        bool
        select SYS_SUPPORTS_ZBOOT

config MACH_XXXX
        [...]
        select SYS_SUPPORTS_ZBOOT_UARTXXX
        [...]

and in arch/mips/boot/compressed/Makfefile

ifdef CONFIG_DEBUG_ZBOOT
obj-$(CONFIG_SYS_SUPPORTS_ZBOOT_UART16550) += $(obj)/uart-16550.o
+obj-$(CONFIG_SYS_SUPPORTS_ZBOOT_UARTXXX) += $(obj)/uart-xxx.o
endif

and also a new arch/mips/boot/compressed/uar-xxx.c is needed.

If this is okay, I will send a new revision asap.

Regards,
        Wu Zhangjin