[coreboot-gerrit] Change in coreboot[master]: nb/intel/haswell/gma: Setup OpRegion in nb code

Patrick Rudolph (Code Review) gerrit at coreboot.org
Sat Apr 15 11:18:16 CEST 2017


Patrick Rudolph has uploaded a new change for review. ( https://review.coreboot.org/19308 )

Change subject: nb/intel/haswell/gma: Setup OpRegion in nb code
......................................................................

nb/intel/haswell/gma: Setup OpRegion in nb code

Setup IGD OpRegion in northbridge and fill in GNVS' aslb.
At this point GNVS already hab been setup by SSDT injection.

Change-Id: I66a2e5ba1f62aa17a2f1e54748fa545429739916
Signed-off-by: Patrick Rudolph <siro at das-labor.org>
---
M src/northbridge/intel/haswell/gma.c
1 file changed, 28 insertions(+), 0 deletions(-)


  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/08/19308/1

diff --git a/src/northbridge/intel/haswell/gma.c b/src/northbridge/intel/haswell/gma.c
index 31e3cb6..94b1f24 100644
--- a/src/northbridge/intel/haswell/gma.c
+++ b/src/northbridge/intel/haswell/gma.c
@@ -510,6 +510,33 @@
 	drivers_intel_gma_displays_ssdt_generate(gfx);
 }
 
+static unsigned long
+gma_write_acpi_tables(struct device *const dev,
+					  unsigned long current,
+					  struct acpi_rsdp *const rsdp)
+{
+	igd_opregion_t *opregion;
+	global_nvs_t *gnvs;
+
+	opregion = igd_make_opregion();
+	if (opregion) {
+		/* GNVS has been already setup */
+		gnvs = cbmem_find(CBMEM_ID_ACPI_GNVS);
+		if (gnvs) {
+			/* IGD OpRegion Base Address */
+			gnvs->aslb = (u32)opregion;
+		} else {
+			printk(BIOS_ERR, "Error: GNVS table not found.\n");
+		}
+	}
+
+	current = acpi_align_current(current);
+
+	printk(BIOS_DEBUG, "current = %lx\n", current);
+
+	return current;
+}
+
 static struct pci_operations gma_pci_ops = {
 	.set_subsystem    = gma_set_subsystem,
 };
@@ -523,6 +550,7 @@
 	.scan_bus		= 0,
 	.enable			= 0,
 	.ops_pci		= &gma_pci_ops,
+	.write_acpi_tables	= gma_write_acpi_tables,
 };
 
 static const unsigned short pci_device_ids[] = {

-- 
To view, visit https://review.coreboot.org/19308
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I66a2e5ba1f62aa17a2f1e54748fa545429739916
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Patrick Rudolph <siro at das-labor.org>



More information about the coreboot-gerrit mailing list