<p>Marshall Dawson has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21850">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">amd/stoneyridge: Remove 16MB cbmem assert<br><br>Do not check for the top of memory being 16MB-aligned near the end of<br>romstage.  This is not the expected alignment using the default 8MB SMM<br>region size.<br><br>BUG=b:67320715<br><br>Change-Id: I6bf0b9141232dea1a3b02794fda7af08887df119<br>Signed-off-by: Marshall Dawson <marshalldawson3rd@gmail.com><br>---<br>M src/soc/amd/stoneyridge/romstage.c<br>1 file changed, 1 insertion(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/21850/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/amd/stoneyridge/romstage.c b/src/soc/amd/stoneyridge/romstage.c<br>index 8cd4d43..7b86a3d 100644<br>--- a/src/soc/amd/stoneyridge/romstage.c<br>+++ b/src/soc/amd/stoneyridge/romstage.c<br>@@ -88,8 +88,7 @@<br>    * 16 megs under cbmem top which is a safe bet to cover ramstage.<br>      */<br>   top_of_ram = (uintptr_t) cbmem_top();<br>-        /* cbmem_top() needs to be at least 16 MiB aligned */<br>-        assert(ALIGN_DOWN(top_of_ram, 16*MiB) == top_of_ram);<br>+<br>      postcar_frame_add_mtrr(&pcf, top_of_ram - 16*MiB, 16*MiB,<br>                 MTRR_TYPE_WRBACK);<br> <br></pre><p>To view, visit <a href="https://review.coreboot.org/21850">change 21850</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/21850"/><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: I6bf0b9141232dea1a3b02794fda7af08887df119 </div>
<div style="display:none"> Gerrit-Change-Number: 21850 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Marshall Dawson <marshalldawson3rd@gmail.com> </div>