On Mon, Aug 30, 2010 at 9:39 PM, Patrick Georgi <span dir="ltr"><<a href="mailto:patrick@georgi-clan.de">patrick@georgi-clan.de</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

Am 30.08.2010 20:37, schrieb Aurélien:<br>
<div class="im">> a) apply the patch as it is, and also remove RAMBASE 0x4000 to all the<br>
> Geode LX based boards, at the risk of maybe breaking coreboot on them.<br>
</div>I think the problem is that it hardcodes assumptions about the installed<br>
RAM - which _might_ go wrong if there's less memory installed (or maybe<br>
not, because it's never touched).<br>
At least if we're talking about the same patch :-)<br>
<br></blockquote><div><br></div><div>I was talking about the patch attached to my message, which is derived from the patch you mentioned before. But, mine does not removes the RAMBASE setting from other LX-based boards' Kconfig, so do not commit it yet, please. Furthermore, I have another idea, which I think is cleaner. I'll check if it's possible to implement.</div>

<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Maybe it's enough to add a big, fat comment to the line in question<br>
"WARNING/TODO/FIXME/XXX: hardcodes assumptions about installed RAM. If<br>
romstage fails, look here!" ;-)<br>
<div class="im"><br></div></blockquote><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">
> b) Place this code in the mainboard-specific romstage.c (if you prefer<br>
> this, I can make a different patch)<br>
</div>I'd prefer global code.<br>
<div><div></div><div class="h5"><br></div></div></blockquote><div><br></div><div>Ok, I'll try to make a patch following these guidelines and resubmit it after some testing. I'll also remove the RAMBASE @0x4000 from the Kconfig of LX-based boards, if mine works.</div>

<div><br></div><div>Best regards,</div></div>-- <br>Aurélien Guillaume<br><br>