<p>Marshall Dawson has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21606">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">amd/stoneyridge: Make UMA memory cacheable<br><br>Use reserved_ram_resource to help ensure the UMA memory is typed<br>as WB.<br><br>BUG=b:65856868<br><br>Change-Id: Ifa54d9b1c206d2ee6dc4b8f90b445a6820ceb8fd<br>Signed-off-by: Marshall Dawson <marshalldawson3rd@gmail.com><br>---<br>M src/soc/amd/stoneyridge/northbridge.c<br>1 file changed, 8 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/06/21606/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/amd/stoneyridge/northbridge.c b/src/soc/amd/stoneyridge/northbridge.c<br>index 0a740cb..ec48a47 100644<br>--- a/src/soc/amd/stoneyridge/northbridge.c<br>+++ b/src/soc/amd/stoneyridge/northbridge.c<br>@@ -490,7 +490,14 @@<br>                            " limitk=%08llx\n", mmio_basek, basek, limitk);<br>     }<br> <br>- add_uma_resource_below_tolm(dev, 7);<br>+ /* UMA is not set up yet, but infer the base & size to make cacheable */<br>+ uint32_t uma_base = restore_top_of_low_cacheable();<br>+  if (uma_base != bsp_topmem()) {<br>+              uint32_t uma_size = bsp_topmem() - uma_base;<br>+         printk(BIOS_INFO, "%s: uma size 0x%08x, memory start 0x%08x\n",<br>+                            __func__, uma_size, uma_base);<br>+               reserved_ram_resource(dev, 7, uma_base / KiB, uma_size / KiB);<br>+       }<br> <br>  for (link = dev->link_list ; link ; link = link->next)<br>          if (link->children)<br></pre><p>To view, visit <a href="https://review.coreboot.org/21606">change 21606</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/21606"/><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: Ifa54d9b1c206d2ee6dc4b8f90b445a6820ceb8fd </div>
<div style="display:none"> Gerrit-Change-Number: 21606 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Marshall Dawson <marshalldawson3rd@gmail.com> </div>