On Wed, 2009-11-25 at 08:42 -0600, Kevin Hickey wrote: > On Tue, 2009-11-24 at 16:30 +0000, Ralf Baechle wrote: > > On Tue, Nov 24, 2009 at 05:24:57PM +0100, Manuel Lauss wrote: > > > > > On Tue, Nov 24, 2009 at 10:33 AM, Wu Zhangjin <wuzhangjin@xxxxxxxxx> wrote: > > > [...] > > > > This patch will only enable that option when the DEBUG_KERNEL is > > > > enabled. > > > > > > How about making it independent from DEBUG_KERNEL altogether? If find > > > it useful even without full debug info. > > I agree with Manuel here. I often build release kernels that benefit > from EARLY_PRINTK. Why not make EARLY_PRINTK a selectable option in the > config? Coupling it to DEBUG_KERNEL seems confusing. Hello, Ralf have moved the EARLY_PRINTK to Kconfig.debug and removed the dependency on DEBUG_KERNEL in his -queue repository ;) Just as the X86 and some other ARCHs does: http://www.linux-mips.org/git?p=linux-queue.git [...] +config EARLY_PRINTK + bool "Early printk" if EMBEDDED + depends on SYS_HAS_EARLY_PRINTK + default y [...] So, it is okay now, please ignore this patch ;) Regards, Wu Zhangjin > > =Kevin > > > > DEBUG_INFO controlls the generation of ELF debug information. DEBUG_KERNEL > > only hides most of the debugging options. > > > > Ralf > > >
