Am 30.08.2010 20:37, schrieb Aurélien: > a) apply the patch as it is, and also remove RAMBASE 0x4000 to all the > Geode LX based boards, at the risk of maybe breaking coreboot on them. I think the problem is that it hardcodes assumptions about the installed RAM - which _might_ go wrong if there's less memory installed (or maybe not, because it's never touched). At least if we're talking about the same patch :-)
Maybe it's enough to add a big, fat comment to the line in question "WARNING/TODO/FIXME/XXX: hardcodes assumptions about installed RAM. If romstage fails, look here!" ;-) > b) Place this code in the mainboard-specific romstage.c (if you prefer > this, I can make a different patch) I'd prefer global code. Patrick -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
