[U-Boot] [PATCH] Replace references to extern console_buffer with a function call

Peter Tyser ptyser at xes-inc.com
Sat Aug 16 00:27:33 CEST 2008


> > I'm not sure if the debug console will be accepted to mainline, but
> I
> > think this patch alone is an improvement over the current
> implementation
> 
> Frankly, I don't think so - it just makes the code bigger for no good
> use to 99.9% of the users.
> 
> > of using a global string and multiple external console_buffer
> > references.  I've always thought global variables should be avoided if
> > there is a reasonable alternative
> 
> But this is a boot loader, and size matters.

I had assumed gcc/ld would be smart enough to optimize the function call
away, seeing as there is no conditional code in it.  I just tested, and
the size was increased by 48 bytes for the MPC8548CDS reference board
after applying the patch.  Inlining the console_buffer_addr() didn't get
rid of the 48 bytes of bloat.

I guess this patch's fate will be the same as "[RFC] Add support for
early serial debug console"...

Thanks,
Peter




More information about the U-Boot mailing list