summaryrefslogtreecommitdiff
path: root/ArmPkg
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
parentd51949806737c214f8df7dc6a70fedf9c2558dd5 (diff)
downloadedk2-platforms-b843b3730d69619e1e69b717ce97065a88b54611.tar.xz
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')
-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 1be90c515c..e6751b29d9 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);