[coreboot-gerrit] New patch to review for coreboot: soc/intel/common: Do not update MRC cache in recovery mode

[Furquan Shaikh (furquan@google.com)]

Furquan Shaikh (furquan at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15828

-gerrit

commit 3bd467ea953411ec4d0802fa5c6364c1096d1f5a
Author: Furquan Shaikh <furquan at google.com>
Date:   Sun Jul 24 08:48:34 2016 -0700

    soc/intel/common: Do not update MRC cache in recovery mode
    
    If the system is in recovery, there is no need to update the MRC
    cache. On next boot, the normal/developer mode boot will save its own
    MRC data.
    
    BUG=chrome-os-partner:55431
    
    Change-Id: Ib13e8c978dc1b4fc8817fab16d0e606f210f2586
    Signed-off-by: Furquan Shaikh <furquan at google.com>
---
 src/soc/intel/common/mrc_cache.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/soc/intel/common/mrc_cache.c b/src/soc/intel/common/mrc_cache.c
index 4ad4158..0b41ffa 100644
--- a/src/soc/intel/common/mrc_cache.c
+++ b/src/soc/intel/common/mrc_cache.c
@@ -14,6 +14,7 @@
  */
 
 #include <string.h>
+#include <bootmode.h>
 #include <bootstate.h>
 #include <console/console.h>
 #include <cbmem.h>
@@ -280,6 +281,11 @@ static void update_mrc_cache(void *unused)
 	const struct mrc_saved_data *next_slot;
 	struct mrc_data_region region;
 
+	if (recovery_mode_enabled()) {
+		printk(BIOS_DEBUG, "In recovery mode. Skipping MRC update.\n");
+		return;
+	}
+
 	printk(BIOS_DEBUG, "Updating MRC cache data.\n");
 
 	current_boot = cbmem_find(CBMEM_ID_MRCDATA);