From 07e9e6f7376f7378ad2385314db475e9dcede0c5 Mon Sep 17 00:00:00 2001 From: haridhar Date: Fri, 18 Dec 2015 10:50:46 +0530 Subject: 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 Original-Commit-Id: acca68bb5fece58549d762bfaef3e9f2eb0d3066 Original-Change-Id: Idb0cd7d7c789a58d05160968f6448cb59882056c Original-Signed-off-by: haridhar Original-Signed-off-by: Somayaji, Vishwanath Original-Reviewed-on: https://chromium-review.googlesource.com/319221 Original-Commit-Ready: Haridhar Kalvala Original-Tested-by: Haridhar Kalvala Original-Reviewed-by: Aaron Durbin Original-Reviewed-by: Subrata Banik Original-Reviewed-by: Haridhar Kalvala Reviewed-on: https://review.coreboot.org/13001 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth --- src/drivers/intel/fsp1_1/raminit.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) 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 #include #include +#include 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); -- cgit v1.2.3