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.web11.6571.1618564447260706290 for ; Fri, 16 Apr 2021 02:14:07 -0700 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 810C0106F; Fri, 16 Apr 2021 02:14:06 -0700 (PDT) Received: from e120189.arm.com (unknown [10.57.1.138]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id C0B633FA35; Fri, 16 Apr 2021 02:14:05 -0700 (PDT) From: "PierreGondois" To: devel@edk2.groups.io, Sami.Mujawar@arm.com Subject: [PATCH RESEND 1/1] Platform/Arm: Fix Ecc error 7008 for SCMI_CLOCK_RATE Date: Fri, 16 Apr 2021 10:14:02 +0100 Message-Id: <20210416091402.16525-1-Pierre.Gondois@arm.com> X-Mailer: git-send-email 2.17.1 From: Pierre Gondois Following the Ecc reported error in the edk2 repository, the SCMI_CLOCK_RATE structure has been modified in: an enum and its elements have been renamed in: ArmPkg/Include/Protocol/ArmScmiClockProtocol.h This patch is a follow-up and fixes the following Ecc reported error: Complex types should be typedef-ed The error is due to the a nested structure declaration. The patch also re-formats the debug messages to fit in a 80 chars line. Signed-off-by: Pierre Gondois --- The changes can be seen at: https://github.com/PierreARM/edk2-platforms/tree/1552_Ecc_ArmPkg_BIS_v1 .../Library/HdLcdArmJunoLib/HdLcdArmJuno.c | 33 +++++++++++++++---- 1 file changed, 26 insertions(+), 7 deletions(-) diff --git a/Platform/ARM/JunoPkg/Library/HdLcdArmJunoLib/HdLcdArmJuno.c b/Platform/ARM/JunoPkg/Library/HdLcdArmJunoLib/HdLcdArmJuno.c index 30558878f068..67c9d43cf2b7 100644 --- a/Platform/ARM/JunoPkg/Library/HdLcdArmJunoLib/HdLcdArmJuno.c +++ b/Platform/ARM/JunoPkg/Library/HdLcdArmJunoLib/HdLcdArmJuno.c @@ -1,6 +1,6 @@ /** @file - Copyright (c) 2013 - 2020, Arm Limited. All rights reserved.
+ Copyright (c) 2013 - 2021, Arm Limited. All rights reserved.
SPDX-License-Identifier: BSD-2-Clause-Patent @@ -352,12 +352,31 @@ ProbeHdLcdClock ( return Status; } - DEBUG ((DEBUG_ERROR, "Clock ID = %d Clock name = %a\n", *ClockId, ClockName)); - DEBUG ((DEBUG_ERROR, "Minimum frequency = %uHz\n", ClockRate.Min)); - DEBUG ((DEBUG_ERROR, "Maximum frequency = %uHz\n", ClockRate.Max)); - DEBUG ((DEBUG_ERROR, "Clock rate step = %uHz\n", ClockRate.Step)); - - DEBUG ((DEBUG_ERROR, "HDLCD Current frequency = %uHz\n", CurrentHdLcdFreq)); + DEBUG (( + DEBUG_ERROR, + "Clock ID = %d Clock name = %a\n", + *ClockId, ClockName + )); + DEBUG (( + DEBUG_ERROR, + "Minimum frequency = %uHz\n", + ClockRate.ContinuousRate.Min + )); + DEBUG (( + DEBUG_ERROR, + "Maximum frequency = %uHz\n", + ClockRate.ContinuousRate.Max + )); + DEBUG (( + DEBUG_ERROR, + "Clock rate step = %uHz\n", + ClockRate.ContinuousRate.Step + )); + DEBUG (( + DEBUG_ERROR, + "HDLCD Current frequency = %uHz\n", + CurrentHdLcdFreq + )); return EFI_SUCCESS; } -- 2.17.1