<div dir="ltr"><br><br><div class="gmail_quote">On Mon, May 11, 2009 at 3:52 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;">
<div class="im">On Sun, May 10, 2009 at 6:21 AM, Elia Yehuda <<a href="mailto:z4ziggy@gmail.com">z4ziggy@gmail.com</a>> wrote:<br>
> This patch enables the onboard VGA found on 82810 boards and fixes the<br>
> memory procedures to support different kinds of DIMMs. also, support for<br>
> 82810e board had been added. The only drawback from this patch is the<br>
> HAVE_HIGH_TABLES being disabled since it disables (for unknown reason)<br>
> the onboard VGA.<br>
</div>It looks like the problem is that the memory you're reserving for VGA<br>
and the memory reserved for high tables ends up in the same place<br>
(ending at tomk.)  If you decide which one should go at the end and<br>
modify the math, it should work.</blockquote><div><br>i completely agree - there seems to be an overlapped memory region, but i couldn't<br>trace it deep enough to be able to fix it. sorry... im not familiar enough with coreboot<br>
code. i suspect you guys will have better luck.<br> <br></div><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="im"><br>
> Signed-off-by: Elia Yehuda <<a href="mailto:z4ziggy@gmail.com">z4ziggy@gmail.com</a>><br>
</div>> Acked-by: Corey Osgood <<a href="mailto:corey.osgood@gmail.com">corey.osgood@gmail.com</a>><br>
<br>
Since it looks like an easy fix, let's get it right before committing<br>
it.  It breaks SeaBIOS, and our assumption that HAVE_HIGH_TABLES<br>
doesn't break anything.<br>
</blockquote><div><br>i'll keep an eye on HAVE_HIGH_TABLES patches and migrate them to my patches<br>and check if it fix the issue. i'll update the ml as for the results.<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
Thanks,<br>
<font color="#888888">Myles<br>
</font></blockquote></div><br>Elia.<br></div>