summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAbhay Kumar <abhay.kumar@intel.com>2016-07-14 18:43:54 -0700
committerMartin Roth <martinroth@google.com>2016-07-29 00:09:05 +0200
commitec2947fb075838d1ea78754113aef6cf000cf522 (patch)
tree862c45dad3c71c00e801db5205141cca2dc5bbc2
parent3707fc2d961720152e7e96bc1606b893023cb577 (diff)
downloadcoreboot-ec2947fb075838d1ea78754113aef6cf000cf522.tar.xz
soc/intel/apollolake: Remove PEIM GFX from normal mode and S3 resume
Do not pass VBT table to fsp in normal mode and S3 resume so that PEIM GFX will not get initialized. Change-Id: Iab7be3cceb0f80ae0273940b36fdd9c41bdb121e Signed-off-by: Abhay Kumar <abhay.kumar@intel.com> Reviewed-on: https://review.coreboot.org/14575 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth <martinroth@google.com>
-rw-r--r--src/soc/intel/apollolake/chip.c3
-rw-r--r--src/soc/intel/common/vbt.c20
-rw-r--r--src/soc/intel/common/vbt.h5
3 files changed, 26 insertions, 2 deletions
diff --git a/src/soc/intel/apollolake/chip.c b/src/soc/intel/apollolake/chip.c
index 544cd133c8..10b71fbc8d 100644
--- a/src/soc/intel/apollolake/chip.c
+++ b/src/soc/intel/apollolake/chip.c
@@ -194,8 +194,7 @@ static void soc_init(void *data)
struct global_nvs_t *gnvs;
/* Save VBT info and mapping */
- if (locate_vbt(&vbt_rdev) != CB_ERR)
- vbt = rdev_mmap_full(&vbt_rdev);
+ vbt = vbt_get(&vbt_rdev);
/* Snapshot the current GPIO IRQ polarities. FSP is setting a
* default policy that doesn't honor boards' requirements. */
diff --git a/src/soc/intel/common/vbt.c b/src/soc/intel/common/vbt.c
index 80b17a2f87..d9bb98a8d1 100644
--- a/src/soc/intel/common/vbt.c
+++ b/src/soc/intel/common/vbt.c
@@ -15,6 +15,8 @@
#include <cbfs.h>
#include <console/console.h>
+#include <arch/acpi.h>
+#include <bootmode.h>
#include "vbt.h"
@@ -40,3 +42,21 @@ enum cb_err locate_vbt(struct region_device *rdev)
return CB_SUCCESS;
}
+
+void *vbt_get(struct region_device *rdev)
+{
+ void *vbt_data;
+
+ /* Normal mode and S3 resume path PEIM GFX init is not needed.
+ * Passing NULL as VBT will not make PEIM GFX to execute. */
+ if (acpi_is_wakeup_s3())
+ return NULL;
+ if (!display_init_required())
+ return NULL;
+ if (locate_vbt(rdev) != CB_ERR) {
+ vbt_data = rdev_mmap_full(rdev);
+ return vbt_data;
+ } else {
+ return NULL;
+ }
+}
diff --git a/src/soc/intel/common/vbt.h b/src/soc/intel/common/vbt.h
index e1a45cc887..9a02e6a467 100644
--- a/src/soc/intel/common/vbt.h
+++ b/src/soc/intel/common/vbt.h
@@ -21,4 +21,9 @@
/* locate .vbt file */
enum cb_err locate_vbt(struct region_device *rdev);
+/*
+ * Returns VBT pointer and mapping after checking prerequisites for Pre OS
+ * Graphics initialization
+ */
+void *vbt_get(struct region_device *rdev);
#endif