From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web12.9497.1614086062910463315 for ; Tue, 23 Feb 2021 05:14:23 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: pierre.gondois@arm.com) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 85B8A101E; Tue, 23 Feb 2021 05:14:22 -0800 (PST) Received: from e120189.arm.com (unknown [10.57.9.161]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id C33EF3F70D; Tue, 23 Feb 2021 05:14:21 -0800 (PST) From: "PierreGondois" To: devel@edk2.groups.io, ardb+tianocore@kernel.org, leif@nuviainc.com, sami.mujawar@arm.com Subject: [PATCH v3 03/26] ArmPkg: Fix Ecc error 8001 in ArmArchTimerLib Date: Tue, 23 Feb 2021 13:13:43 +0000 Message-Id: <20210223131406.3484-4-Pierre.Gondois@arm.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20210223131406.3484-1-Pierre.Gondois@arm.com> References: <20210223131406.3484-1-Pierre.Gondois@arm.com> From: Pierre Gondois This patch fixes the following Ecc reported error: Only capital letters are allowed to be used for #define declarations Signed-off-by: Pierre Gondois Reviewed-by: Sami Mujawar --- The changes can be seen at: https://github.com/PierreARM/edk2/tree/1552_Ecc_ArmPkg_BIS_v3 ArmPkg/Library/ArmArchTimerLib/ArmArchTimerLib.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/ArmPkg/Library/ArmArchTimerLib/ArmArchTimerLib.c b/ArmPkg/Library/ArmArchTimerLib/ArmArchTimerLib.c index 24d9dae4e660..875c835706d6 100644 --- a/ArmPkg/Library/ArmArchTimerLib/ArmArchTimerLib.c +++ b/ArmPkg/Library/ArmArchTimerLib/ArmArchTimerLib.c @@ -1,7 +1,7 @@ /** @file Generic ARM implementation of TimerLib.h - Copyright (c) 2011-2016, ARM Limited. All rights reserved. + Copyright (c) 2011 - 2021, Arm Limited. All rights reserved.
SPDX-License-Identifier: BSD-2-Clause-Patent @@ -20,9 +20,9 @@ // Select appropriate multiply function for platform architecture. #ifdef MDE_CPU_ARM -#define MultU64xN MultU64x32 +#define MULT_U64_X_N MultU64x32 #else -#define MultU64xN MultU64x64 +#define MULT_U64_X_N MultU64x64 #endif @@ -121,7 +121,7 @@ MicroSecondDelay ( // = MicroSeconds x TICKS_PER_MICRO_SEC // = MicroSeconds x Frequency.10^-6 TimerTicks64 = DivU64x32 ( - MultU64xN ( + MULT_U64_X_N ( MicroSeconds, GetPlatformTimerFreq () ), @@ -263,7 +263,7 @@ GetTimeInNanoSecond ( // Time = --------- x 1,000,000,000 // Frequency // - NanoSeconds = MultU64xN ( + NanoSeconds = MULT_U64_X_N ( DivU64x32Remainder ( Ticks, TimerFreq, @@ -276,7 +276,7 @@ GetTimeInNanoSecond ( // will not overflow 64-bit. // NanoSeconds += DivU64x32 ( - MultU64xN ( + MULT_U64_X_N ( (UINT64) Remainder, 1000000000U), TimerFreq -- 2.17.1