On Mon, Aug 30, 2010 at 9:39 PM, Patrick Georgi <patrick@georgi-clan.de> wrote:
Am 30.08.2010 20:37, schrieb Aurélien:
> a) apply the patch as it is, and also remove RAMBASE 0x4000 to all theI think the problem is that it hardcodes assumptions about the installed
> Geode LX based boards, at the risk of maybe breaking coreboot on them.
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 preferI'd prefer global code.
> this, I can make a different patch)