[coreboot] [PATCH] Start unifying ram_check() usage in coreboot

Peter Stuge peter at stuge.se
Sat Oct 31 23:55:46 CET 2009


Stefan Reinauer wrote:
> NACK

NAK +1.


> I don't think this should be user visible. In fact I think it
> should be dropped from all boards once they are working.

I agree with this.


> The test is bogus and only capable of finding crude coding errors.
> So either you're a developer and know how to put the code there, or
> you shouldn't know about it.

It's also not very nice to have a library function be conditionally
empty depending on configuration. I think it would be better to move
towards fewer lines of code in cache_as_ram_auto.c, not more.


//Peter




More information about the coreboot mailing list