<p>Martin Roth has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21848">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">vc/amd/pi/00670F00: Remove HeapXXBuffer functions<br><br>The HeapAllocateBuffer and HEAPDeallocateBuffer functions are not used<br>in Stoney Ridge, so get rid of them.<br><br>Change-Id: I716d5c8957ced52c25fd501697111b1b0b263467<br>Signed-off-by: Martin Roth <martinroth@google.com><br>---<br>M src/vendorcode/amd/pi/00670F00/binaryPI/AGESA.c<br>1 file changed, 0 insertions(+), 60 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/48/21848/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/vendorcode/amd/pi/00670F00/binaryPI/AGESA.c b/src/vendorcode/amd/pi/00670F00/binaryPI/AGESA.c<br>index 046dcb4..a4eef5a 100644<br>--- a/src/vendorcode/amd/pi/00670F00/binaryPI/AGESA.c<br>+++ b/src/vendorcode/amd/pi/00670F00/binaryPI/AGESA.c<br>@@ -51,19 +51,6 @@<br> #include <console/console.h><br> #include <commonlib/loglevel.h><br> <br>-// TODO These need to be replaced with calls to CreateStruct()<br>-AGESA_STATUS<br>-HeapAllocateBuffer (<br>-  IN OUT   VOID *AllocateHeapParams,<br>-  IN OUT   VOID *StdHeader<br>-  );<br>-<br>-AGESA_STATUS<br>-HeapDeallocateBuffer (<br>-  IN       UINT32 BufferHandle,<br>-  IN       VOID *StdHeader<br>-  );<br>-<br> CONST UINT32 ImageSignature = IMAGE_SIGNATURE;<br> CONST UINT32 ModuleSignature = MODULE_SIGNATURE;<br> CONST CHAR8 ModuleIdentifier[] = AGESA_ID;<br>@@ -412,51 +399,4 @@<br>  if (!module) return;<br>  Dispatcher = module->ModuleDispatcher;<br>     Dispatcher(FchDataPtr);<br>-}<br>-<br>-// TODO This has to be removed<br>-AGESA_STATUS<br>-HeapAllocateBuffer (<br>-  IN OUT   VOID *AllocateHeapParams,<br>-  IN OUT   VOID *StdHeader<br>-  )<br>-{<br>-  MODULE_ENTRY Dispatcher = NULL;<br>-      const AMD_MODULE_HEADER* module = agesawrapper_locate_module(ModuleIdentifier);<br>-<br>-   AMD_INTERFACE_PARAMS InterfaceParams = {};<br>-<br>-        if (!module) return AGESA_UNSUPPORTED;<br>-       Dispatcher = module->ModuleDispatcher;<br>-<br>- InterfaceParams.StdHeader = *(AMD_CONFIG_PARAMS*)StdHeader;<br>-  InterfaceParams.StdHeader.Func = AMD_HEAP_ALLOCATE_BUFFER;<br>-<br>-        InterfaceParams.AllocationMethod = PreMemHeap;<br>-       InterfaceParams.NewStructPtr = AllocateHeapParams;<br>-<br>-        return Dispatcher(&InterfaceParams);<br>-}<br>-<br>-// TODO This has to be removed<br>-AGESA_STATUS<br>-HeapDeallocateBuffer (<br>-  IN       UINT32 BufferHandle,<br>-  IN       VOID *StdHeader<br>-  )<br>-{<br>-    MODULE_ENTRY Dispatcher = NULL;<br>-      const AMD_MODULE_HEADER* module = agesawrapper_locate_module(ModuleIdentifier);<br>-<br>-   AMD_INTERFACE_PARAMS InterfaceParams = {};<br>-<br>-        if (!module) return AGESA_UNSUPPORTED;<br>-       Dispatcher = module->ModuleDispatcher;<br>-<br>- InterfaceParams.StdHeader = *(AMD_CONFIG_PARAMS*)StdHeader;<br>-  InterfaceParams.StdHeader.Func = AMD_HEAP_DEALLOCATE_BUFFER;<br>-<br>-      InterfaceParams.NewStructPtr = &BufferHandle;<br>-<br>- return Dispatcher(&InterfaceParams);<br> }<br></pre><p>To view, visit <a href="https://review.coreboot.org/21848">change 21848</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/21848"/><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: I716d5c8957ced52c25fd501697111b1b0b263467 </div>
<div style="display:none"> Gerrit-Change-Number: 21848 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Martin Roth <martinroth@google.com> </div>