From a9492a6169b4636b93de7c9271dd39a16fcc3a5b Mon Sep 17 00:00:00 2001 From: Matt DeVillier Date: Wed, 20 Jun 2018 00:40:48 -0500 Subject: 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 Reviewed-on: https://review.coreboot.org/c/30233 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Rudolph --- src/soc/intel/braswell/gfx.c | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) (limited to 'src/soc/intel/braswell/gfx.c') diff --git a/src/soc/intel/braswell/gfx.c b/src/soc/intel/braswell/gfx.c index 895d2ee7da..9417409358 100644 --- a/src/soc/intel/braswell/gfx.c +++ b/src/soc/intel/braswell/gfx.c @@ -68,15 +68,16 @@ static void gfx_init(struct device *dev) 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); - - /* Post VBIOS Init */ - gfx_post_vbios_init(dev); + /* Run VBIOS */ + pci_dev_init(dev); + /* Post VBIOS Init */ + gfx_post_vbios_init(dev); + } intel_gma_restore_opregion(); } -- cgit v1.2.3