2010/10/7 Stefan Reinauer <span dir="ltr"><<a href="mailto:stefan.reinauer@coresystems.de">stefan.reinauer@coresystems.de</a>></span><br><div class="gmail_quote"><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 class="im">
    On 10/6/10 2:27 PM, Idwer Vollering wrote:
    <blockquote type="cite">2010/10/6 Uwe Hermann <span dir="ltr"><<a href="mailto:uwe@hermann-uwe.de" target="_blank">uwe@hermann-uwe.de</a>></span><br>
      <div class="gmail_quote">
        <blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
          See patch.<br>
        </blockquote>
        <div><br>
          Here is a fix for building on 32-bit platforms:<br>
          <br>
          Index: src/northbridge/intel/i440bx/raminit.c<br>
===================================================================<br>
          --- src/northbridge/intel/i440bx/raminit.c      (revision
          5917)<br>
          +++ src/northbridge/intel/i440bx/raminit.c      (working copy)<br>
          @@ -657,8 +657,8 @@<br>
           }<br>
          <br>
           struct dimm_size {<br>
          -       unsigned long side1;<br>
          -       unsigned long side2;<br>
          +       uint32_t side1;<br>
          +       uint32_t side2;<br>
           };<br>
        </div>
      </div>
    </blockquote>
    <br></div>
    What's that error?</div></blockquote><div><br>In file included from src/mainboard/asus/p2b/romstage.c:46:<br>src/northbridge/intel/i440bx/raminit.c: In function ‘spd_get_dimm_size’:<br>src/northbridge/intel/i440bx/raminit.c:717: error: format ‘%04x’ expects type ‘unsigned int’, but argument 3 has type ‘long unsigned int’<br>
 </div><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">It seems the above should be about the same on
    32bit platforms, especially it should be the same on 32bit platforms
    and 64bit platforms, since the code is always compiled 32bit...<br>
    What did I miss?<br></div></blockquote><div><br>Should it be something else than uint32_t ? Plain int ?<br><br></div><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">
    <br>
    Stefan<br>
    <br>
  </div>

<br>--<br>
coreboot mailing list: <a href="mailto:coreboot@coreboot.org">coreboot@coreboot.org</a><br>
<a href="http://www.coreboot.org/mailman/listinfo/coreboot" target="_blank">http://www.coreboot.org/mailman/listinfo/coreboot</a><br></blockquote></div><br>