<br><br><div class="gmail_quote">On Wed, Apr 14, 2010 at 1:22 PM, Stefan Reinauer <span dir="ltr"><<a href="mailto:stepan@coresystems.de">stepan@coresystems.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">



  

<div bgcolor="#ffffff" text="#000000"><div><div></div><div class="h5">
On 4/14/10 6:44 PM, Myles Watson wrote:
<blockquote type="cite"><br>
  <div class="gmail_quote">
  <blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Index:
src/northbridge/amd/gx2/chipsetinit.c<br>
===================================================================<br>
--- src/northbridge/amd/gx2/chipsetinit.c (revision 5425)<br>
+++ src/northbridge/amd/gx2/chipsetinit.c (working copy)<br>
@@ -275,7 +275,7 @@<br>
if ((msr.lo&0xff) == 0x11)<br>
return;<br>
    <br>
- totalmem = sizeram() << 20 - 1;<br>
+ totalmem = (sizeram() << 20) - 1;<br>
  </blockquote>
  <div>I agree that it looks right, but it changes the answer, since
subtraction has a higher precedence than shift.<br>
  <br>
I don't have a board to test which is right.  Did you check v3?<br>
  </div>
  </div>
</blockquote></div></div>
Nor do I...<br>
<br>
v3 never supported gx2.</div>

</blockquote></div><br>I guess this should be the patch, then:<br><br>- totalmem = sizeram() << 20 - 1;<br>

+ totalmem = sizeram() << (20 - 1);<br><br>Anyone with an OLPC board care to chime in?<br><br>Thanks,<br>Myles<br>