diff options
author | Patrick Georgi <pgeorgi@chromium.org> | 2016-08-08 11:43:43 +0200 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2016-08-09 10:40:32 +0200 |
commit | 5c4748b342fafb84dfaf03c3e8187b8e5bf9e3e9 (patch) | |
tree | 6a002b55199e41e087117f777f4cdaec581cc430 | |
parent | 57603e2b8475b9f42466ec7dec8c0567ae6dae52 (diff) |
vendorcode/amd: Remove dead code
Change-Id: I6b21822d60d379cb8cd21b69c714a437bb7977ce
Signed-off-by: Patrick Georgi <pgeorgi@chromium.org>
Found-by: Coverity Scan #1254643 and others
Reviewed-on: https://review.coreboot.org/16112
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
-rw-r--r-- | src/vendorcode/amd/pi/00630F01/binaryPI/AGESA.c | 42 | ||||
-rw-r--r-- | src/vendorcode/amd/pi/00730F01/binaryPI/AGESA.c | 42 |
2 files changed, 0 insertions, 84 deletions
diff --git a/src/vendorcode/amd/pi/00630F01/binaryPI/AGESA.c b/src/vendorcode/amd/pi/00630F01/binaryPI/AGESA.c index 7a6851df65..c207d67506 100644 --- a/src/vendorcode/amd/pi/00630F01/binaryPI/AGESA.c +++ b/src/vendorcode/amd/pi/00630F01/binaryPI/AGESA.c @@ -438,13 +438,7 @@ WaitForEcLDN9MailboxCmdAck ( IN AMD_CONFIG_PARAMS *StdHeader ) { - MODULE_ENTRY Dispatcher = NULL; - const AMD_MODULE_HEADER* module = agesawrapper_locate_module(ModuleIdentifier); StdHeader->Func = 0; - return; - if (!module) return; - Dispatcher = module->ModuleDispatcher; - Dispatcher(StdHeader); } /** @@ -459,13 +453,7 @@ ImcSleep ( IN VOID *FchDataPtr ) { - MODULE_ENTRY Dispatcher = NULL; - const AMD_MODULE_HEADER* module = agesawrapper_locate_module(ModuleIdentifier); ((FCH_DATA_BLOCK*)FchDataPtr)->StdHeader->Func = 0; - return; - if (!module) return; - Dispatcher = module->ModuleDispatcher; - Dispatcher(FchDataPtr); } /** @@ -480,13 +468,7 @@ SoftwareDisableImc ( IN VOID *FchDataPtr ) { - MODULE_ENTRY Dispatcher = NULL; - const AMD_MODULE_HEADER* module = agesawrapper_locate_module(ModuleIdentifier); ((FCH_DATA_BLOCK*)FchDataPtr)->StdHeader->Func = 0; - return; - if (!module) return; - Dispatcher = module->ModuleDispatcher; - Dispatcher(FchDataPtr); } /** @@ -501,13 +483,7 @@ ImcEnableSurebootTimer ( IN VOID *FchDataPtr ) { - MODULE_ENTRY Dispatcher = NULL; - const AMD_MODULE_HEADER* module = agesawrapper_locate_module(ModuleIdentifier); ((FCH_DATA_BLOCK*)FchDataPtr)->StdHeader->Func = 0; - return; - if (!module) return; - Dispatcher = module->ModuleDispatcher; - Dispatcher(FchDataPtr); } /** @@ -522,13 +498,7 @@ ImcDisableSurebootTimer ( IN VOID *FchDataPtr ) { - MODULE_ENTRY Dispatcher = NULL; - const AMD_MODULE_HEADER* module = agesawrapper_locate_module(ModuleIdentifier); ((FCH_DATA_BLOCK*)FchDataPtr)->StdHeader->Func = 0; - return; - if (!module) return; - Dispatcher = module->ModuleDispatcher; - Dispatcher(FchDataPtr); } /** @@ -543,13 +513,7 @@ ImcWakeup ( IN VOID *FchDataPtr ) { - MODULE_ENTRY Dispatcher = NULL; - const AMD_MODULE_HEADER* module = agesawrapper_locate_module(ModuleIdentifier); ((FCH_DATA_BLOCK*)FchDataPtr)->StdHeader->Func = 0; - return; - if (!module) return; - Dispatcher = module->ModuleDispatcher; - Dispatcher(FchDataPtr); } /** @@ -564,13 +528,7 @@ ImcIdle ( IN VOID *FchDataPtr ) { - MODULE_ENTRY Dispatcher = NULL; - const AMD_MODULE_HEADER* module = agesawrapper_locate_module(ModuleIdentifier); ((FCH_DATA_BLOCK*)FchDataPtr)->StdHeader->Func = 0; - return; - if (!module) return; - Dispatcher = module->ModuleDispatcher; - Dispatcher(FchDataPtr); } // TODO This has to be removed diff --git a/src/vendorcode/amd/pi/00730F01/binaryPI/AGESA.c b/src/vendorcode/amd/pi/00730F01/binaryPI/AGESA.c index b58107ca78..82744dd0cf 100644 --- a/src/vendorcode/amd/pi/00730F01/binaryPI/AGESA.c +++ b/src/vendorcode/amd/pi/00730F01/binaryPI/AGESA.c @@ -436,13 +436,7 @@ WaitForEcLDN9MailboxCmdAck ( IN AMD_CONFIG_PARAMS *StdHeader ) { - MODULE_ENTRY Dispatcher = NULL; - const AMD_MODULE_HEADER* module = agesawrapper_locate_module(ModuleIdentifier); StdHeader->Func = 0; - return; - if (!module) return; - Dispatcher = module->ModuleDispatcher; - Dispatcher(StdHeader); } /** @@ -457,13 +451,7 @@ ImcSleep ( IN VOID *FchDataPtr ) { - MODULE_ENTRY Dispatcher = NULL; - const AMD_MODULE_HEADER* module = agesawrapper_locate_module(ModuleIdentifier); ((FCH_DATA_BLOCK*)FchDataPtr)->StdHeader->Func = 0; - return; - if (!module) return; - Dispatcher = module->ModuleDispatcher; - Dispatcher(FchDataPtr); } /** @@ -478,13 +466,7 @@ SoftwareDisableImc ( IN VOID *FchDataPtr ) { - MODULE_ENTRY Dispatcher = NULL; - const AMD_MODULE_HEADER* module = agesawrapper_locate_module(ModuleIdentifier); ((FCH_DATA_BLOCK*)FchDataPtr)->StdHeader->Func = 0; - return; - if (!module) return; - Dispatcher = module->ModuleDispatcher; - Dispatcher(FchDataPtr); } /** @@ -499,13 +481,7 @@ ImcEnableSurebootTimer ( IN VOID *FchDataPtr ) { - MODULE_ENTRY Dispatcher = NULL; - const AMD_MODULE_HEADER* module = agesawrapper_locate_module(ModuleIdentifier); ((FCH_DATA_BLOCK*)FchDataPtr)->StdHeader->Func = 0; - return; - if (!module) return; - Dispatcher = module->ModuleDispatcher; - Dispatcher(FchDataPtr); } /** @@ -520,13 +496,7 @@ ImcDisableSurebootTimer ( IN VOID *FchDataPtr ) { - MODULE_ENTRY Dispatcher = NULL; - const AMD_MODULE_HEADER* module = agesawrapper_locate_module(ModuleIdentifier); ((FCH_DATA_BLOCK*)FchDataPtr)->StdHeader->Func = 0; - return; - if (!module) return; - Dispatcher = module->ModuleDispatcher; - Dispatcher(FchDataPtr); } /** @@ -541,13 +511,7 @@ ImcWakeup ( IN VOID *FchDataPtr ) { - MODULE_ENTRY Dispatcher = NULL; - const AMD_MODULE_HEADER* module = agesawrapper_locate_module(ModuleIdentifier); ((FCH_DATA_BLOCK*)FchDataPtr)->StdHeader->Func = 0; - return; - if (!module) return; - Dispatcher = module->ModuleDispatcher; - Dispatcher(FchDataPtr); } /** @@ -562,13 +526,7 @@ ImcIdle ( IN VOID *FchDataPtr ) { - MODULE_ENTRY Dispatcher = NULL; - const AMD_MODULE_HEADER* module = agesawrapper_locate_module(ModuleIdentifier); ((FCH_DATA_BLOCK*)FchDataPtr)->StdHeader->Func = 0; - return; - if (!module) return; - Dispatcher = module->ModuleDispatcher; - Dispatcher(FchDataPtr); } // TODO This has to be removed |