[coreboot] [PATCH] Make RAM check configurable via Kconfig
stepan at coresystems.de
Mon Mar 17 09:28:41 CET 2008
ron minnich wrote:
> i can't debug without the memtest, but it is true that we don't need
> it once a board works.
So you say you want to change some CONFIG_THIS_THAT for every bit of
_debugging_ you turn on during your bringup?
And: if you ask the user to turn on debugging or just spew, is that what
you want to see in the log? if so, it should maybe be spew?
but to be honest I'd stick with #if 0 during development and replace it
with such a CONFIG_ option once the code is done.
In v2 we have far too many, far too meaningless CONFIG_ options.
Remember XIP_ROM_* ? The _CODE_ should really take care of such stuff.
Asking the user to make a decision is not helpful to adopters nor users.
> That's my main conclusion from this thread, but I'm not worried enough
> about it either way to argue the point too much.
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.de • http://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 249 bytes
Desc: OpenPGP digital signature
More information about the coreboot