<p>Shelley Chen has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/22518">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mainboard/google/fizz: Enable separate MRC cache for recovery mode<br><br>Enable separate MRC cache for recovery mode. This requires change in<br>flash layout to accomodate another region for RECOVERY_MRC_CACHE.<br><br>BUG=b:69473883<br>TEST=Verified following scenarios:<br>1. Boot into recovery does not destroy normal mode MRC cache.<br>2. Once recovery MRC cache is populated, all future boots in recovery<br>mode re-use data from the cache.<br>3. Forcing recovery mode to retrain memory causes normal mode to retrain<br>memory as well.<br><br>Change-Id: Icdfac3698507d89d98a51cfc3d756a56d2a2d648<br>Signed-off-by: Shelley Chen <shchen@chromium.org><br>---<br>M src/mainboard/google/fizz/Kconfig<br>M src/mainboard/google/fizz/chromeos.fmd<br>2 files changed, 20 insertions(+), 13 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/18/22518/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/mainboard/google/fizz/Kconfig b/src/mainboard/google/fizz/Kconfig<br>index 6474824..b1f353d 100644<br>--- a/src/mainboard/google/fizz/Kconfig<br>+++ b/src/mainboard/google/fizz/Kconfig<br>@@ -26,6 +26,8 @@<br> select GBB_FLAG_DISABLE_EC_SOFTWARE_SYNC<br> select VBOOT_EC_EFS<br> select VBOOT_PHYSICAL_REC_SWITCH<br>+ select HAS_RECOVERY_MRC_CACHE<br>+ select MRC_CLEAR_NORMAL_CACHE_ON_RECOVERY_RETRAIN<br> <br> config DRIVER_TPM_I2C_BUS<br> depends on FIZZ_USE_I2C_TPM<br>diff --git a/src/mainboard/google/fizz/chromeos.fmd b/src/mainboard/google/fizz/chromeos.fmd<br>index 58b6127..bdadcf7 100644<br>--- a/src/mainboard/google/fizz/chromeos.fmd<br>+++ b/src/mainboard/google/fizz/chromeos.fmd<br>@@ -4,24 +4,29 @@<br> SI_ME@0x1000 0x1ff000<br> }<br> SI_BIOS@0x200000 0xe00000 {<br>- RW_SECTION_A@0x0 0x3f0000 {<br>+ RW_SECTION_A@0x0 0x3e8000 {<br> VBLOCK_A@0x0 0x10000<br>- FW_MAIN_A(CBFS)@0x10000 0x3dffc0<br>- RW_FWID_A@0x3effc0 0x40<br>+ FW_MAIN_A(CBFS)@0x10000 0x3d7fc0<br>+ RW_FWID_A@0x3e7fc0 0x40<br> }<br>- RW_SECTION_B@0x3f0000 0x3f0000 {<br>+ RW_SECTION_B@0x3e8000 0x3e8000 {<br> VBLOCK_B@0x0 0x10000<br>- FW_MAIN_B(CBFS)@0x10000 0x3dffc0<br>- RW_FWID_B@0x3effc0 0x40<br>+ FW_MAIN_B(CBFS)@0x10000 0x3d7fc0<br>+ RW_FWID_B@0x3e7fc0 0x40<br> }<br>- RW_MRC_CACHE@0x7e0000 0x10000<br>- RW_ELOG@0x7f0000 0x4000<br>- RW_SHARED@0x7f4000 0x4000 {<br>- SHARED_DATA@0x0 0x2000<br>- VBLOCK_DEV@0x2000 0x2000<br>+ RW_MISC@0x7d0000 0x30000 {<br>+ UNIFIED_MRC_CACHE@0x0 0x20000 {<br>+ RECOVERY_MRC_CACHE@0x0 0x10000<br>+ RW_MRC_CACHE@0x10000 0x10000<br>+ }<br>+ RW_ELOG@0x20000 0x4000<br>+ RW_SHARED@0x24000 0x4000 {<br>+ SHARED_DATA@0x0 0x2000<br>+ VBLOCK_DEV@0x2000 0x2000<br>+ }<br>+ RW_VPD@0x28000 0x2000<br>+ RW_NVRAM@0x2a000 0x6000<br> }<br>- RW_VPD@0x7f8000 0x2000<br>- RW_NVRAM@0x7fa000 0x6000<br> RW_LEGACY(CBFS)@0x800000 0x200000<br> WP_RO@0xa00000 0x400000 {<br> RO_VPD@0x0 0x4000<br></pre><p>To view, visit <a href="https://review.coreboot.org/22518">change 22518</a>. To unsubscribe, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/22518"/><meta itemprop="name" content="View Change"/></div></div>
<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: Icdfac3698507d89d98a51cfc3d756a56d2a2d648 </div>
<div style="display:none"> Gerrit-Change-Number: 22518 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Shelley Chen <shchen@google.com> </div>