<p>Marshall Dawson has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/22053">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/amd/common: Revert PI blob search hack<br><br>Remove the check for CONFIG_VBOOT when finding the binaryPI blob and<br>rely on the cbfs search 100% of the time.  The change was initially<br>put in to avoid a hang when vboot presearched memory for the blob.<br>The implementation now supports early cbmem init and cbmem_top() is<br>careful to return 0 if DRAM has not yet been set up.  As a result the<br>hang no longer occurs and the hack may be removed safely.<br><br>BUG=b:67747902<br><br>Change-Id: I1f38709fcce250b0902a639ebf0554219bc47cf8<br>Signed-off-by: Marshall Dawson <marshalldawson3rd@gmail.com><br>---<br>M src/soc/amd/common/agesawrapper.c<br>1 file changed, 2 insertions(+), 8 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/53/22053/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/amd/common/agesawrapper.c b/src/soc/amd/common/agesawrapper.c<br>index a269130..3ef76f3 100644<br>--- a/src/soc/amd/common/agesawrapper.c<br>+++ b/src/soc/amd/common/agesawrapper.c<br>@@ -395,14 +395,8 @@<br>   const AMD_MODULE_HEADER* module;<br>      size_t file_size;<br> <br>- if (IS_ENABLED(CONFIG_VBOOT)) {<br>-              /* Use phys. location in flash and prevent vboot from searching cbmem */<br>-             agesa = (void *)CONFIG_AGESA_BINARY_PI_LOCATION;<br>-             file_size = 0x100000;<br>-        } else {<br>-             agesa = cbfs_boot_map_with_leak((const char *)CONFIG_AGESA_CBFS_NAME,<br>-                                        CBFS_TYPE_RAW, &file_size);<br>-      }<br>+    agesa = cbfs_boot_map_with_leak((const char *)CONFIG_AGESA_CBFS_NAME,<br>+                                                CBFS_TYPE_RAW, &file_size);<br> <br>    if (!agesa)<br>           return NULL;<br></pre><p>To view, visit <a href="https://review.coreboot.org/22053">change 22053</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/22053"/><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: I1f38709fcce250b0902a639ebf0554219bc47cf8 </div>
<div style="display:none"> Gerrit-Change-Number: 22053 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Marshall Dawson <marshalldawson3rd@gmail.com> </div>