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

Wolfgang Denk wd at denx.de
Fri Aug 15 23:35:54 CEST 2008


Dear Peter Tyser,

In message <1218831956.1273.114.camel at localhost.localdomain> you wrote:
> > what is the purpose of this patch? As far as I can see we gain
> > nothing, but we lose on memory footpint.  I guess there is some
> > problem you're trying to fix, but you did not mention it?
> 
> I've added a basic "debug serial console" which relies on this patch.
> The debug console is a simple command prompt which allows the user to
> issue commands before relocating to SDRAM.  We've found this feature
> very useful for operations such as initially programming SPD data,
> debugging hardware issues (SDRAM/I2C).  I'll submit the debug console
> patches shortly.

OK, I think what you're trying to do.

> 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.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Lady Nancy Astor:
    "Winston, if you were my husband, I'd put poison in your coffee."
Winston Churchill:
    "Nancy, if you were my wife, I'd drink it."



More information about the U-Boot mailing list