<p>Martin Roth has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/22060">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/amd/common: Set AltImageBasePtr to 0<br><br>In the original AGESA headers, AltImageBasePtr is a UINT32, so don't<br>set it to VOID.  0 workes for either UINT32 or VOID *, as demonstrated<br>by the other 7 places in this file where it's already set to 0 instead<br>of NULL.<br><br>Change this location to 0 to support either version of the headers.<br><br>BUG=b:64766233<br>TEST=Build in cros tree and upstream coreboot, with old headers<br>and updated headers.<br><br>Change-Id: Ib6f3883e08231a6ca896c2ee2ef631c77feafedd<br>Signed-off-by: Martin Roth <martinroth@google.com><br>---<br>M src/soc/amd/common/agesawrapper.c<br>1 file changed, 1 insertion(+), 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/60/22060/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..38ca3a8 100644<br>--- a/src/soc/amd/common/agesawrapper.c<br>+++ b/src/soc/amd/common/agesawrapper.c<br>@@ -122,7 +122,7 @@<br> <br>   AmdParamStruct.AgesaFunctionName = AMD_INIT_POST;<br>     AmdParamStruct.AllocationMethod = PreMemHeap;<br>-        AmdParamStruct.StdHeader.AltImageBasePtr = NULL;<br>+     AmdParamStruct.StdHeader.AltImageBasePtr = 0;<br>         AmdParamStruct.StdHeader.CalloutPtr = &GetBiosCallout;<br>    AmdParamStruct.StdHeader.Func = 0;<br>    AmdParamStruct.StdHeader.ImageBasePtr = 0;<br></pre><p>To view, visit <a href="https://review.coreboot.org/22060">change 22060</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/22060"/><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: Ib6f3883e08231a6ca896c2ee2ef631c77feafedd </div>
<div style="display:none"> Gerrit-Change-Number: 22060 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Martin Roth <martinroth@google.com> </div>