[coreboot] [PATCH] consolidate K8M890 VGA code

Luc Verhaegen libv at skynet.be
Thu Oct 29 00:18:08 CET 2009


On Wed, Oct 28, 2009 at 05:00:11PM -0600, Myles Watson wrote:
> On Wed, Oct 28, 2009 at 4:44 PM, Myles Watson <mylesgw at gmail.com> wrote:
> 
> >     fbbits = ((log2(uma_memory_size >> 20) - 2) << 4);
> > Why not use CONFIG_VIDEO_MB here?  If it is going to be fixed at compile
> > time, it seems like you can get rid of uma_memory_size.
> >
> Either way:
> 
> Acked-by: Myles Watson <mylesgw at gmail.com>
> 
> Thanks,
> Myles

NACK.

WHY?

Luc Verhaegen.




More information about the coreboot mailing list