From 6a9e6cd89e2be6b35675ee81c2272a5ea5f2db75 Mon Sep 17 00:00:00 2001 From: Richard Spiegel Date: Fri, 30 Nov 2018 10:53:40 -0700 Subject: 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 Reviewed-on: https://review.coreboot.org/c/29985 Reviewed-by: HAOUAS Elyes Tested-by: build bot (Jenkins) --- src/soc/amd/common/block/pi/def_callouts.c | 2 -- 1 file changed, 2 deletions(-) (limited to 'src/soc/amd/common/block/pi') 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 -- cgit v1.2.3