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

Reply via email to