<p>Marc Jones has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/22150">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/amd/common: Remove RunFcnOnAllAps() callback<br><br>RunFcnOnAllAps() is not a callback, but a AGESA call. This is a mistake<br>in the AGESA spec and the function is in the wrong section.<br><br>bug=b:66690176, b:67210418<br>branch=none<br>test=none<br><br>Change-Id: I900e7db13a58e73a7b054e06088bc77c89445876<br>Signed-off-by: Marc Jones <marcj303@gmail.com><br>---<br>M src/soc/amd/common/BiosCallOuts.h<br>M src/soc/amd/common/def_callouts.c<br>2 files changed, 0 insertions(+), 11 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/22150/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/amd/common/BiosCallOuts.h b/src/soc/amd/common/BiosCallOuts.h<br>index ef853fa..8ee7463 100644<br>--- a/src/soc/amd/common/BiosCallOuts.h<br>+++ b/src/soc/amd/common/BiosCallOuts.h<br>@@ -49,7 +49,6 @@<br>                                                        VOID *ConfigPrt);<br> <br> AGESA_STATUS agesa_ReadSpd(UINT32 Func, UINTN Data, VOID *ConfigPtr);<br>-AGESA_STATUS agesa_RunFcnOnAllAps(UINT32 Func, UINTN Data, VOID *ConfigPtr);<br> AGESA_STATUS agesa_LateRunApTask(UINT32 Func, UINTN Data, VOID *ConfigPtr);<br> AGESA_STATUS agesa_PcieSlotResetControl(UINT32 Func, UINTN Data,<br>                                                  VOID *ConfigPtr);<br>diff --git a/src/soc/amd/common/def_callouts.c b/src/soc/amd/common/def_callouts.c<br>index 6e46f69..fd3b5e9 100644<br>--- a/src/soc/amd/common/def_callouts.c<br>+++ b/src/soc/amd/common/def_callouts.c<br>@@ -39,7 +39,6 @@<br>     { AGESA_READ_SPD,                 agesa_ReadSpd },<br>    { AGESA_RUNFUNC_ONAP,             agesa_RunFuncOnAp },<br>        { AGESA_RUNFUNC_ON_ALL_APS,       agesa_RunFcnOnAllAps },<br>-    { AMD_LATE_RUN_AP_TASK,           agesa_LateRunApTask },<br>      { AGESA_GNB_PCIE_SLOT_RESET,      agesa_PcieSlotResetControl },<br>       { AGESA_WAIT_FOR_ALL_APS,         agesa_WaitForAllApsFinished },<br>      { AGESA_IDLE_AN_AP,               agesa_IdleAnAp },<br>@@ -162,15 +161,6 @@<br> <br>  return Status;<br> }<br>-<br>-AGESA_STATUS agesa_LateRunApTask(UINT32 Func, UINTN Data, VOID *ConfigPtr)<br>-{<br>-       printk(BIOS_WARNING, "Warning - Missing AGESA callout: %s\n", __func__);<br>-   AGESA_STATUS Status = AGESA_UNSUPPORTED;<br>-<br>-  return Status;<br>-}<br>-<br> <br> AGESA_STATUS agesa_PcieSlotResetControl(UINT32 Func, UINTN Data,<br>   VOID *ConfigPtr)<br></pre><p>To view, visit <a href="https://review.coreboot.org/22150">change 22150</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/22150"/><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: I900e7db13a58e73a7b054e06088bc77c89445876 </div>
<div style="display:none"> Gerrit-Change-Number: 22150 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Marc Jones <marc@marcjonesconsulting.com> </div>