[coreboot-gerrit] Change in ...coreboot[master]: soc/braswell: ensure ACPI opregion restored on S3 with GOP init

[Matt DeVillier (Code Review)]

Matt DeVillier has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/30233


Change subject: soc/braswell: ensure ACPI opregion restored on S3 with GOP init
......................................................................

soc/braswell: ensure ACPI opregion restored on S3 with GOP init

The Intel GMA ACPI opregion address needs to be set on S3 resume,
otherwise the Windows display driver fails to re-initialize correctly.
Fix by ensuring the address is set correctly regardless of display
init type used (GOP or VBIOS).

Test: build/boot on google/edgar, ensure internal display functional
following S3 resume under Windows 10.

Change-Id: I471c44e8ba4514e4a2ddf6739109b759145598ed
Signed-off-by: Matt DeVillier <matt.devillier at gmail.com>
---
M src/soc/intel/braswell/Makefile.inc
M src/soc/intel/braswell/gfx.c
2 files changed, 9 insertions(+), 9 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/33/30233/1

diff --git a/src/soc/intel/braswell/Makefile.inc b/src/soc/intel/braswell/Makefile.inc
index ac1bacd..fabbc2b 100644
--- a/src/soc/intel/braswell/Makefile.inc
+++ b/src/soc/intel/braswell/Makefile.inc
@@ -22,9 +22,8 @@
 ramstage-$(CONFIG_ELOG) += elog.c
 ramstage-y += emmc.c
 ramstage-y += gpio.c
-ifneq ($(CONFIG_RUN_FSP_GOP),y)
 ramstage-y += gfx.c
-endif
+
 ramstage-y += gpio_support.c
 ramstage-y += hda.c
 ramstage-y += iosf.c
diff --git a/src/soc/intel/braswell/gfx.c b/src/soc/intel/braswell/gfx.c
index 895d2ee..9417409 100644
--- a/src/soc/intel/braswell/gfx.c
+++ b/src/soc/intel/braswell/gfx.c
@@ -68,15 +68,16 @@
 	printk(BIOS_SPEW, "%s/%s (%s)\n",
 			__FILE__, __func__, dev_name(dev));
 
-	/* Pre VBIOS Init */
-	gfx_pre_vbios_init(dev);
+	if (!IS_ENABLED(CONFIG_RUN_FSP_GOP)) {
+		/* Pre VBIOS Init */
+		gfx_pre_vbios_init(dev);
 
-	/* Run VBIOS */
-	pci_dev_init(dev);
+		/* Run VBIOS */
+		pci_dev_init(dev);
 
-	/* Post VBIOS Init */
-	gfx_post_vbios_init(dev);
-
+		/* Post VBIOS Init */
+		gfx_post_vbios_init(dev);
+	}
 	intel_gma_restore_opregion();
 }
 

-- 
To view, visit https://review.coreboot.org/c/coreboot/+/30233
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I471c44e8ba4514e4a2ddf6739109b759145598ed
Gerrit-Change-Number: 30233
Gerrit-PatchSet: 1
Gerrit-Owner: Matt DeVillier <matt.devillier at gmail.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20181215/2b2bb831/attachment-0001.html>