On 06/20/2010 07:37 AM, Stefan Reinauer wrote:
On 6/20/10 7:59 AM, Joseph Smith wrote:+ if (CONFIG_VIDEO_MB == 512) { + igd_memory = (CONFIG_VIDEO_MB); + printk(BIOS_DEBUG, "%dKB IGD UMA\n", igd_memory>> 10); + } else { + igd_memory = (CONFIG_VIDEO_MB * 1024); + printk(BIOS_DEBUG, "%dMB IGD UMA\n", igd_memory>> 10); + }You might want to read the SMRAM register intead of relying on the setting of the config variable in case an invalid value sneaked in. Other than that: Acked-by: Stefan Reinauer <[email protected]>
Done, thanks. r5635 -- Thanks, Joseph Smith Set-Top-Linux www.settoplinux.org -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot

