Re: [coreboot] [PATCH] artecgroup/dbe61: Set up some video memory, as the device has VGA output.

[Carl-Daniel Hailfinger]

On 28.05.2008 02:27, Mart Raudsepp wrote:
> artecgroup/dbe61: Set up some video memory, as the device has VGA output.
>
> This allows me to have a working coreinfo payload on DBE61 with coreboot-v3.
> Signed-off-by: Mart Raudsepp
>
> Outside suggested commit message note:
>
> I think geode_video_mb should be a Kconfig option for boards that have
> video output possible at all. The 16MB is picked because 8MB is not enough
> for WinCE, but this much is often not necessary to be allocated for other
> uses, so configurability would be nice.

Why not include that in the commit message? That way, we'll know in 2
years from now why this was set to 16 MB.

Acked-by: Carl-Daniel Hailfinger

Regards,
Carl-Daniel