Author: zbao
Date: 2009-08-11 05:18:11 +0200 (Tue, 11 Aug 2009)
New Revision: 4525

Modified:
   trunk/coreboot-v2/src/cpu/x86/mtrr/mtrr.c
Log:
The code between #if and #endif is only about UMA mode. The CONFIG_GFXUMA 
should be 1.
We have another mode called side port mode. It is When the CONFIG_GFXUMA is 0.

Signed-off-by: Zheng Bao <[email protected]>
Acked-by: Myles Watson <[email protected]>


Modified: trunk/coreboot-v2/src/cpu/x86/mtrr/mtrr.c
===================================================================
--- trunk/coreboot-v2/src/cpu/x86/mtrr/mtrr.c   2009-08-10 15:57:05 UTC (rev 
4524)
+++ trunk/coreboot-v2/src/cpu/x86/mtrr/mtrr.c   2009-08-11 03:18:11 UTC (rev 
4525)
@@ -418,7 +418,7 @@
        search_global_resources(
                IORESOURCE_MEM | IORESOURCE_CACHEABLE, IORESOURCE_MEM | 
IORESOURCE_CACHEABLE,
                set_var_mtrr_resource, &var_state);
-#ifdef CONFIG_GFXUMA
+#if (CONFIG_GFXUMA == 1) /* UMA or SP. */
        // For now we assume the UMA space is at the end of memory
        if (var_state.hole_startk || var_state.hole_sizek) {
                printk_debug("Warning: Can't set up MTRR hole for UMA due to 
pre-existing MTRR hole.\n");


-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to