summaryrefslogtreecommitdiff
path: root/src/soc/amd/common/block
diff options
context:
space:
mode:
authorRichard Spiegel <richard.spiegel@amd.corp-partner.google.com>2018-11-30 10:53:40 -0700
committerPatrick Georgi <pgeorgi@google.com>2018-12-05 14:05:23 +0000
commit6a9e6cd89e2be6b35675ee81c2272a5ea5f2db75 (patch)
tree4bcf764753debbdcde977c3635761f5fcd2607ac /src/soc/amd/common/block
parentc3a27dffe90b1f3021ed9f8beed531ad224dbc4f (diff)
downloadcoreboot-6a9e6cd89e2be6b35675ee81c2272a5ea5f2db75.tar.xz
src/(device/lib/soc): Remove unused variables
When building grunt with flags set to detect variables that get a value but then are unused, there are 5 instances that causes error (unused variable). In most cases it's enough to simply remove the variable. Other instances, is better to simply use the variables (one instance it's a return value, on the other instance using the variables makes code more readable). BUG=b:120260448 TEST=Build and boot grunt. Change-Id: I0d00fb6a42db20afb34c76b9445a741a57096ead Signed-off-by: Richard Spiegel <richard.spiegel@silverbackltd.com> Reviewed-on: https://review.coreboot.org/c/29985 Reviewed-by: HAOUAS Elyes <ehaouas@noos.fr> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/soc/amd/common/block')
-rw-r--r--src/soc/amd/common/block/pi/def_callouts.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/src/soc/amd/common/block/pi/def_callouts.c b/src/soc/amd/common/block/pi/def_callouts.c
index 0fffaf345b..693c7c58e3 100644
--- a/src/soc/amd/common/block/pi/def_callouts.c
+++ b/src/soc/amd/common/block/pi/def_callouts.c
@@ -116,10 +116,8 @@ AGESA_STATUS agesa_Reset(uint32_t Func, uintptr_t Data, void *ConfigPtr)
{
AGESA_STATUS Status;
uintptr_t ResetType;
- AMD_CONFIG_PARAMS *StdHeader;
ResetType = Data;
- StdHeader = ConfigPtr;
/*
* This should perform the RESET based upon the ResetType, but coreboot