summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorharidhar <haridhar.kalvala@intel.com>2015-12-18 10:50:46 +0530
committerPatrick Georgi <pgeorgi@google.com>2016-01-19 16:23:06 +0100
commit07e9e6f7376f7378ad2385314db475e9dcede0c5 (patch)
tree1d37a77f705b4b670f2c410f5dba668fa4ac6800
parentfefce185335232aade9e512a1cbe94da27578768 (diff)
intel/skylake: do not save MRC data in recovery mode
If the system is in recovery don't bother saving MRC training data. BRANCH=None BUG=chrome-os-partner:48534 TEST=Built for kunimitsu. Results show MRC data is not saved in recovery mode. Change-Id: I236b7fe1860ac86722562c9a749067496dfe98f8 Signed-off-by: Patrick Georgi <pgeorgi@chromium.org> Original-Commit-Id: acca68bb5fece58549d762bfaef3e9f2eb0d3066 Original-Change-Id: Idb0cd7d7c789a58d05160968f6448cb59882056c Original-Signed-off-by: haridhar <haridhar.kalvala@intel.com> Original-Signed-off-by: Somayaji, Vishwanath <vishwanath.somayaji@intel.com> Original-Reviewed-on: https://chromium-review.googlesource.com/319221 Original-Commit-Ready: Haridhar Kalvala <haridhar.kalvala@intel.com> Original-Tested-by: Haridhar Kalvala <haridhar.kalvala@intel.com> Original-Reviewed-by: Aaron Durbin <adurbin@chromium.org> Original-Reviewed-by: Subrata Banik <subrata.banik@intel.com> Original-Reviewed-by: Haridhar Kalvala <haridhar.kalvala@intel.com> Reviewed-on: https://review.coreboot.org/13001 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth <martinroth@google.com>
-rw-r--r--src/drivers/intel/fsp1_1/raminit.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/src/drivers/intel/fsp1_1/raminit.c b/src/drivers/intel/fsp1_1/raminit.c
index a19f6aec83..2ba77e34ad 100644
--- a/src/drivers/intel/fsp1_1/raminit.c
+++ b/src/drivers/intel/fsp1_1/raminit.c
@@ -24,6 +24,7 @@
#include <soc/romstage.h>
#include <string.h>
#include <timestamp.h>
+#include <bootmode.h>
void raminit(struct romstage_params *params)
{
@@ -287,7 +288,8 @@ void raminit(struct romstage_params *params)
if (mrc_hob == NULL)
printk(BIOS_DEBUG,
"Memory Configuration Data Hob not present\n");
- else {
+ else if (!recovery_mode_enabled()) {
+ /* Do not save MRC data in recovery path */
pei_ptr->data_to_save = GET_GUID_HOB_DATA(mrc_hob);
pei_ptr->data_to_save_size = ALIGN(
((u32)GET_HOB_LENGTH(mrc_hob)), 16);