<p>Marc Jones has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/23818">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/amd/common: Save the UMA settings from AGESA<br><br>Save the UMA base and size settings returned by AGESA<br>in amdinitpost();<br><br>Change-Id: Id96cc65582118ad41d397b1a600cab1615676a55<br>Signed-off-by: Marc Jones <marcj303@gmail.com><br>---<br>M 3rdparty/blobs<br>M src/soc/amd/common/block/pi/agesawrapper.c<br>2 files changed, 7 insertions(+), 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/18/23818/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/3rdparty/blobs b/3rdparty/blobs</span><br><span>index 19dea8d..a5efee5 160000</span><br><span>--- a/3rdparty/blobs</span><br><span>+++ b/3rdparty/blobs</span><br><span>@@ -1 +1 @@</span><br><span style="color: hsl(0, 100%, 40%);">-Subproject commit 19dea8d171544f01f12ee6b78af0cc356ab994aa</span><br><span style="color: hsl(120, 100%, 40%);">+Subproject commit a5efee5fdea55f398decc9cabebce3744bbd8147</span><br><span>diff --git a/src/soc/amd/common/block/pi/agesawrapper.c b/src/soc/amd/common/block/pi/agesawrapper.c</span><br><span>index dca7222..cfa3760 100644</span><br><span>--- a/src/soc/amd/common/block/pi/agesawrapper.c</span><br><span>+++ b/src/soc/amd/common/block/pi/agesawrapper.c</span><br><span>@@ -26,6 +26,7 @@</span><br><span> #include <amdblocks/s3_resume.h></span><br><span> #include <amdblocks/agesawrapper.h></span><br><span> #include <amdblocks/BiosCallOuts.h></span><br><span style="color: hsl(120, 100%, 40%);">+#include <soc/southbridge.h></span><br><span> </span><br><span> void __attribute__((weak)) SetMemParams(AMD_POST_PARAMS *PostParams) {}</span><br><span> void __attribute__((weak)) OemPostParams(AMD_POST_PARAMS *PostParams) {}</span><br><span>@@ -207,12 +208,16 @@</span><br><span>     * higher than UmaBase. With UMA_NONE we see UmaBase==0.</span><br><span>      */</span><br><span>  uintptr_t top;</span><br><span style="color: hsl(0, 100%, 40%);">-  if (PostParams->MemConfig.UmaBase)</span><br><span style="color: hsl(120, 100%, 40%);">+ if (PostParams->MemConfig.UmaBase &&</span><br><span style="color: hsl(120, 100%, 40%);">+                       (PostParams->MemConfig.UmaBase < ((4ull * GiB) >> 16)))</span><br><span>          top = PostParams->MemConfig.UmaBase << 16;</span><br><span>  else</span><br><span>                 top = PostParams->MemConfig.Sub4GCacheTop;</span><br><span>        backup_top_of_low_cacheable(top);</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+ save_uma_size(PostParams->MemConfig.UmaSize * 64 * KiB);</span><br><span style="color: hsl(120, 100%, 40%);">+   save_uma_base((u64)PostParams->MemConfig.UmaBase * 64 * KiB);</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>   print_init_post_settings(PostParams);</span><br><span> </span><br><span>    if (status != AGESA_SUCCESS)</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/23818">change 23818</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/23818"/><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: Id96cc65582118ad41d397b1a600cab1615676a55 </div>
<div style="display:none"> Gerrit-Change-Number: 23818 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Marc Jones <marc@marcjonesconsulting.com> </div>