<p>Marshall Dawson has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21491">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">amd/stoneyridge: Make cbmem_top SMM aware<br><br>Make the default for SMM_TSEG_SIZE depend on SMM_TSEG in addition to<br>HAVE_SMI_HANDLER.<br><br>Change the value returned by cbmem_top() to carve out a range to be<br>used by TSEG.  Align the value to 128KB to match the granularity of<br>the TSegBase field.<br><br>Change-Id: I54ffc10108862b7d022fbbd92bf97525b349df27<br>Signed-off-by: Marshall Dawson <marshalldawson3rd@gmail.com><br>---<br>M src/soc/amd/stoneyridge/Kconfig<br>M src/soc/amd/stoneyridge/ramtop.c<br>2 files changed, 3 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/91/21491/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/amd/stoneyridge/Kconfig b/src/soc/amd/stoneyridge/Kconfig<br>index 08613fe..f8768e7 100644<br>--- a/src/soc/amd/stoneyridge/Kconfig<br>+++ b/src/soc/amd/stoneyridge/Kconfig<br>@@ -289,7 +289,7 @@<br> <br> config SMM_TSEG_SIZE<br>      hex<br>-  default 0x800000 if HAVE_SMI_HANDLER<br>+ default 0x800000 if SMM_TSEG && HAVE_SMI_HANDLER<br>      default 0x0<br> <br> config ACPI_CPU_STRING<br>diff --git a/src/soc/amd/stoneyridge/ramtop.c b/src/soc/amd/stoneyridge/ramtop.c<br>index c81e73b..bbe11d2 100644<br>--- a/src/soc/amd/stoneyridge/ramtop.c<br>+++ b/src/soc/amd/stoneyridge/ramtop.c<br>@@ -41,5 +41,6 @@<br>   if (!tom.lo)<br>          return 0;<br>     else<br>-         return (void *)restore_top_of_low_cacheable();<br>+               return (void *)ALIGN_DOWN(restore_top_of_low_cacheable()<br>+                             - CONFIG_SMM_TSEG_SIZE, 128*KiB);<br> }<br></pre><p>To view, visit <a href="https://review.coreboot.org/21491">change 21491</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/21491"/><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: I54ffc10108862b7d022fbbd92bf97525b349df27 </div>
<div style="display:none"> Gerrit-Change-Number: 21491 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Marshall Dawson <marshalldawson3rd@gmail.com> </div>