<p>Aamir Bohra has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/23156">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">intel/fsp2_0: Set boot mode default as per s3wake status<br><br>Currently bootmode default is set to FSP_BOOT_WITH_FULL_CONFIGURATION<br>and this upd is updated in fsp_fill_mrc_cache based on mrc cache data<br>validity. With current implementation in S3 resume path, if mrc cache<br>data is invalid, the bootmode is not updated further and remains set at<br>FSP_BOOT_WITH_FULL_CONFIGURATION. This results in fsp-m to get incorrect<br>boot mode context and reinitialize memory in S3 resume path. In correct<br>follow fspm should have correct bootmode context i.e S3 resume and return<br>error in case mrc cache data is invalid or not found.<br><br>BUG=b:70973961<br>BRANCH=None<br>TEST=Verify correct bootmode is set on S3 resume, even when<br>     mrc cache data is invalid.<br><br>Change-Id: Idc0da6ffbfe5ce616d852908a9b0074dc8ce7cbe<br>Signed-off-by: Aamir Bohra <aamir.bohra@intel.com><br>---<br>M src/drivers/intel/fsp2_0/memory_init.c<br>1 file changed, 2 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/56/23156/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/drivers/intel/fsp2_0/memory_init.c b/src/drivers/intel/fsp2_0/memory_init.c</span><br><span>index ce1265a..9c540ef 100644</span><br><span>--- a/src/drivers/intel/fsp2_0/memory_init.c</span><br><span>+++ b/src/drivers/intel/fsp2_0/memory_init.c</span><br><span>@@ -278,7 +278,8 @@</span><br><span> </span><br><span>       arch_upd->StackBase = (void *)stack_begin;</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-       arch_upd->BootMode = FSP_BOOT_WITH_FULL_CONFIGURATION;</span><br><span style="color: hsl(120, 100%, 40%);">+     arch_upd->BootMode = s3wake ? FSP_BOOT_ON_S3_RESUME :</span><br><span style="color: hsl(120, 100%, 40%);">+              FSP_BOOT_WITH_FULL_CONFIGURATION;</span><br><span> </span><br><span>        /*</span><br><span>    * For S3 resume case, if valid mrc cache data is not found</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/23156">change 23156</a>. To unsubscribe, or for help writing mail filters, 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/23156"/><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: Idc0da6ffbfe5ce616d852908a9b0074dc8ce7cbe </div>
<div style="display:none"> Gerrit-Change-Number: 23156 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Aamir Bohra <aamir.bohra@intel.com> </div>