summaryrefslogtreecommitdiff
path: root/ArmPkg/Library
diff options
context:
space:
mode:
authorSami Mujawar <sami.mujawar@arm.com>2016-03-03 11:55:20 +0000
committerArd Biesheuvel <ard.biesheuvel@linaro.org>2016-04-01 17:14:47 +0200
commitb843b3730d69619e1e69b717ce97065a88b54611 (patch)
tree0f597ba04ebb299ae838203cdc8bfef71d2d1581 /ArmPkg/Library
parentd51949806737c214f8df7dc6a70fedf9c2558dd5 (diff)
downloadedk2-b843b3730d69619e1e69b717ce97065a88b54611.zip
edk2-b843b3730d69619e1e69b717ce97065a88b54611.tar.gz
edk2-b843b3730d69619e1e69b717ce97065a88b54611.tar.bz2
ArmPkg/ArmArchTimerLib: fix unused variable in RELEASE builds
The TimerFreq variable in the TimerConstructor() is unused in RELEASE builds since ASSERTs are then disabled. The only use of the variable (in the ASSERT) is replaced by a direct invocation of the function previously used to set it. NOTE: The build tools suppress warnings of this using compiler options eg. -Wno-unused-but-set-variable for GCC toolchain or --diag_suppress=550 for RVCT toolchain. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Evan Lloyd <evan.lloyd@arm.com> Reviewed-by: Ryan Harkin <ryan.harkin@linaro.org> Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Diffstat (limited to 'ArmPkg/Library')
-rw-r--r--ArmPkg/Library/ArmArchTimerLib/ArmArchTimerLib.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/ArmPkg/Library/ArmArchTimerLib/ArmArchTimerLib.c b/ArmPkg/Library/ArmArchTimerLib/ArmArchTimerLib.c
index 1be90c5..e6751b2 100644
--- a/ArmPkg/Library/ArmArchTimerLib/ArmArchTimerLib.c
+++ b/ArmPkg/Library/ArmArchTimerLib/ArmArchTimerLib.c
@@ -42,7 +42,6 @@ TimerConstructor (
// Check if the ARM Generic Timer Extension is implemented.
//
if (ArmIsArchTimerImplemented ()) {
- UINTN TimerFreq;
//
// Check if Architectural Timer frequency is pre-determined by the platform
@@ -74,8 +73,8 @@ TimerConstructor (
// mode (if secure extension is supported).
// If the reset value (0) is returned, just ASSERT.
//
- TimerFreq = ArmGenericTimerGetTimerFreq ();
- ASSERT (TimerFreq != 0);
+ ASSERT (ArmGenericTimerGetTimerFreq () != 0);
+
} else {
DEBUG ((EFI_D_ERROR, "ARM Architectural Timer is not available in the CPU, hence this library can not be used.\n"));
ASSERT (0);