Re: [coreboot] [patch] fix intel 82810 onboard VGA and SDRAM functions

[Myles Watson]

On Sun, May 10, 2009 at 6:21 AM, Elia Yehuda  wrote:
> This patch enables the onboard VGA found on 82810 boards and fixes the
> memory procedures to support different kinds of DIMMs. also, support for
> 82810e board had been added. The only drawback from this patch is the
> HAVE_HIGH_TABLES being disabled since it disables (for unknown reason)
> the onboard VGA.

It looks like the problem is that the memory you're reserving for VGA
and the memory reserved for high tables ends up in the same place
(ending at tomk.)  If you decide which one should go at the end and
modify the math, it should work.
> Signed-off-by: Elia Yehuda 
> Acked-by: Corey Osgood

Since it looks like an easy fix, let's get it right before committing
it.  It breaks SeaBIOS, and our assumption that HAVE_HIGH_TABLES
doesn't break anything.

Thanks,
Myles