<p>Martin Roth has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/22061">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">vendorcode/amd/pi/00670F00: Remove S3 restore functions<br><br>These functions are not currently used, and were not in the original<br>AGESA source code drop.  If we end up needing them for S3 resume, we<br>can add them back at that point.<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: Iec346205470150257fd9d09131d54231b321740b<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(+), 33 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/61/22061/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 a4eef5a..5a41897 100644<br>--- a/src/vendorcode/amd/pi/00670F00/binaryPI/AGESA.c<br>+++ b/src/vendorcode/amd/pi/00670F00/binaryPI/AGESA.c<br>@@ -367,36 +367,3 @@<br>      Dispatcher = module->ModuleDispatcher;<br>     return Dispatcher(AmdGetDataEye);<br> }<br>-<br>-/**********************************************************************<br>- * FCH Functions<br>- **********************************************************************/<br>-<br>-VOID FchInitS3LateRestore (IN FCH_DATA_BLOCK *FchDataPtr);<br>-VOID FchInitS3EarlyRestore (IN FCH_DATA_BLOCK *FchDataPtr);<br>-<br>-VOID<br>-FchInitS3EarlyRestore (<br>-  IN      FCH_DATA_BLOCK     *FchDataPtr<br>-  )<br>-{<br>-      MODULE_ENTRY Dispatcher = NULL;<br>-      const AMD_MODULE_HEADER* module = agesawrapper_locate_module(ModuleIdentifier);<br>-      FchDataPtr->StdHeader->Func = FCH_INIT_S3_EARLY_RESTORE;<br>-       if (!module) return;<br>- Dispatcher = module->ModuleDispatcher;<br>-    Dispatcher(FchDataPtr);<br>-}<br>-<br>-VOID<br>-FchInitS3LateRestore (<br>-  IN      FCH_DATA_BLOCK     *FchDataPtr<br>-  )<br>-{<br>-  MODULE_ENTRY Dispatcher = NULL;<br>-      const AMD_MODULE_HEADER* module = agesawrapper_locate_module(ModuleIdentifier);<br>-      FchDataPtr->StdHeader->Func = FCH_INIT_S3_LATE_RESTORE;<br>-        if (!module) return;<br>- Dispatcher = module->ModuleDispatcher;<br>-    Dispatcher(FchDataPtr);<br>-}<br></pre><p>To view, visit <a href="https://review.coreboot.org/22061">change 22061</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/22061"/><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: Iec346205470150257fd9d09131d54231b321740b </div>
<div style="display:none"> Gerrit-Change-Number: 22061 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Martin Roth <martinroth@google.com> </div>