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

[Myles Watson]

On Mon, Oct 26, 2009 at 2:14 PM, Uwe Hermann  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

Thanks,
Myles