[coreboot] [PATCH] Add kconfig menus for most chipset VIDEO_MB values

Myles Watson mylesgw at gmail.com
Mon Oct 26 21:26:36 CET 2009


On Mon, Oct 26, 2009 at 2:14 PM, Uwe Hermann <uwe at hermann-uwe.de> wrote:

> See patch.
>
I liked the correspondence between the devices menu and src/devices.  Maybe
some other name would be better for the top-level menu?  I don't think
miscellaneous is really what we want to say for src/devices.

I think it's pretty ugly to have 512 be the value for CONFIG_VIDEO_MB that
means 512K.  I know you didn't do that, but...  Maybe we could just say the
512K of RAM doesn't make that much difference and make the minimum 1MB?

With the menu names fixed:
Acked-by: Myles Watson <mylesgw at gmail.com>

Thanks,
Myles
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20091026/c9996614/attachment.html>


More information about the coreboot mailing list