<br><br><div class="gmail_quote">On Wed, Oct 14, 2009 at 7:49 PM, Myles Watson <span dir="ltr"><<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br><div class="gmail_quote"><div>#if CONFIG_SOUTHBRIDGE_AMD_RS690  // I guess this could be board-specific too, since that's where these variables are defined.  <br></div><div class="im"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

<div>+extern uint64_t uma_memory_base, uma_memory_size;<br></div></blockquote><br></div></div></blockquote><div>I think the nicest thing to do would be to move the definitions of uma_memory_* into the southbridge code.  It doesn't really belong in the mainboard.  We could also protect it with CONFIG_GFXUMA.  I'd like to use CONFIG_VIDEO_MB, but it looks like the code assigns the uma_memory_size based on the size of main memory.  I guess since it wouldn't brick the boards we could just use the defines and set it to 128M since that's for 1G and up.  Are many people running these boards with 256 MB of RAM?<br>
<br>Thanks,<br>Myles  <br></div></div>