Re: [PATCH] MIPS: EARLY_PRINTK: Fixup of dependency

[Wu Zhangjin <wuzhangjin@gmail.com>]

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@gmail.com> 
> > > 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
> > 
>