[coreboot-gerrit] New patch to review for coreboot: nb/intel/pineview: Make preallocated igd memory a cmos parameter

[Arthur Heymans]

Arthur Heymans ([e-mail]) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18142

-gerrit

commit ffa27426e9ade5fcd3155572a7ab357934d56db7
Author: Arthur Heymans 
Date:   Sat Jan 14 17:32:20 2017 +0100

nb/intel/pineview: Make preallocated igd memory a cmos parameter

Change-Id: Ia7fa2c290e540ff779cf8dc16147db5a248021e2
    Signed-off-by: Arthur Heymans 
---
 src/northbridge/intel/pineview/early_init.c | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/src/northbridge/intel/pineview/early_init.c b/src/northbridge/intel/pineview/early_init.c
index 842ffa1..933d1bc 100644
--- a/src/northbridge/intel/pineview/early_init.c
+++ b/src/northbridge/intel/pineview/early_init.c
@@ -25,6 +25,7 @@
 #include <string.h>
 #include <northbridge/intel/pineview/pineview.h>
 #include <northbridge/intel/pineview/chip.h>
+#include <pc80/mc146818rtc.h>

#define LPC PCI_DEV(0, 0x1f, 0)
 #define D0F0 PCI_DEV(0, 0, 0)
@@ -45,7 +46,16 @@ static void early_graphics_setup(void)
    const struct northbridge_intel_pineview_config *config = d0f0->chip_info;

pci_write_config8(D0F0, DEVEN, BOARD_DEVEN);
-	pci_write_config16(D0F0, GGC, 0x130); /* 1MB GTT 8MB UMA */
+
+	/* vram size from cmos option */
+	if (get_option(&reg8, "gfx_uma_size") != CB_SUCCESS)
+		reg8 = 2;	/* 2 for 8MB */
+	/* make sure no invalid setting is used */
+	if (reg8 > 8)
+		reg8 = 2;
+	/* Select 1M GTT */
+	pci_write_config16(PCI_DEV(0, 0x00, 0), GGC, (1 << 8)
+			   | ((reg8 + 1) << 4));

printk(BIOS_SPEW, "Set GFX clocks...");
    reg16 = MCHBAR16(MCH_GCFGC);