From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from NAM04-DM6-obe.outbound.protection.outlook.com (NAM04-DM6-obe.outbound.protection.outlook.com [40.107.102.83]) by mx.groups.io with SMTP id smtpd.web11.4001.1685428421961531857 for ; Mon, 29 May 2023 23:33:42 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="body hash did not verify" header.i=@amd.com header.s=selector1 header.b=lAA96pCY; spf=permerror, err=parse error for token &{10 18 %{i}._ip.%{h}._ehlo.%{d}._spf.vali.email}: invalid domain name (domain: amd.com, ip: 40.107.102.83, mailfrom: abner.chang@amd.com) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=fBcGFS+2pkrARJ9Q7gv1vrBsFJQm6zdQt6U8XziEOAGQ4ihkOmQR6tPicOGxuQLnaQOHVGpaGJeHl2VbDbiO6ZF50Lqku3RziGGtBEQUkjofuhLEMQhRrr1evNr+TCHZHFpLJFZqQp1/6gAzEq5VRbPy2Ksy4hswy7I7fBYtOcJgHUtU3+zY3Bgp4bA5pZgQWewyiF1Bhmh96tEG+c5sAkJyRAmsvpPLC96ZD4xcnX6yHjbqPb8tJylcIuro/7IpuJ3bMbu7H6HXvGB5Tj/rN6PtH0RulMS+oCXv8wUvci4L2ccBC8ykBUfBUCn57lwyzXDlxv68GPeAvhKD1J70yg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=IAWB12bhhMPdB051n229nAmMdjiljsMl/60psIpHqzA=; b=LoEvO4oDDvCatd3bm5BA7fwp/ZPwmoGocqG2OiC23FB3ldfVtRreqKiPxvkEys28tmWeLsIg2OAH4B4mAy57qjjdjEnZ2r6FC25QGwls/ms8AQxEcRN8hn1yYJYDHtGudDh5Q5X1rp6EaTL00lmjcMNkR7ZC9BsU6CV+DPUC98CfRVsge5JtkEK34yTKI79PZX6NMTD66/yGJUaYCIf9Ktq1Y3W6dNrSe8x8jHYTCIlBjWcYRIXPO2hP7ZV3Z3s9afqS9XQEeAG2VumV7IuKdD3NTrbJOcU0EYhaghS5C+J8Gcj0Gh6UiU0qPCqEBu3ivSxhiQ0Z1vRT6o2nLbQhtw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=edk2.groups.io smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=IAWB12bhhMPdB051n229nAmMdjiljsMl/60psIpHqzA=; b=lAA96pCYsSsSVAQqovMc3amf8abqwov39Pg84K56Hsk9iTMEAvT041bz0URIOLVz726oJqaVtuGLNOrocMI5kmAW1OHu7kltyY5wfEnDT62dvHm4aJFWiTEU1zEOQ+XZHGNxycfiif+KT1wwmR9Tkres/2Cys57xQD5S6v7DqUc= Received: from BN0PR04CA0196.namprd04.prod.outlook.com (2603:10b6:408:e9::21) by PH0PR12MB7486.namprd12.prod.outlook.com (2603:10b6:510:1e9::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6433.23; Tue, 30 May 2023 06:33:35 +0000 Received: from BN8NAM11FT050.eop-nam11.prod.protection.outlook.com (2603:10b6:408:e9:cafe::57) by BN0PR04CA0196.outlook.office365.com (2603:10b6:408:e9::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6433.23 via Frontend Transport; Tue, 30 May 2023 06:33:35 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by BN8NAM11FT050.mail.protection.outlook.com (10.13.177.5) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.6455.22 via Frontend Transport; Tue, 30 May 2023 06:33:35 +0000 Received: from TPE-L1-ABNCHANG.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.34; Tue, 30 May 2023 01:33:31 -0500 From: "Chang, Abner" To: CC: Isaac Oram , Abdul Lateef Attar , Nickle Wang , Tinh Nguyen Subject: [edk2-platforms][PATCH 1/2] ManageabilityPkg: Replace __FUNCTION__ with __func__ Date: Tue, 30 May 2023 14:33:15 +0800 Message-ID: <20230530063316.46-1-abner.chang@amd.com> X-Mailer: git-send-email 2.37.1.windows.1 MIME-Version: 1.0 Return-Path: Abner.Chang@amd.com X-Originating-IP: [10.180.168.240] X-ClientProxiedBy: SATLEXMB04.amd.com (10.181.40.145) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BN8NAM11FT050:EE_|PH0PR12MB7486:EE_ X-MS-Office365-Filtering-Correlation-Id: defcaf19-8e4b-42d6-01e6-08db60d7cb7e X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: NBEjckpQm8OD+ZwN0Yj7D7aBBNAmJo3hC4yky9aCHkmhXkhz9EEvW1QxW0n0gVwdQMU2OTcMITY6WjGFyIHAhf9NGhyD+Bt9zmfCyzTju0u9+5q79J8LO7fr7OuJMl1hof7PpAdpzJy9S4U4OJxWs0iCJ0HMubLqrhvOQoK1GYbfBfdUGTHcfaZl6Hh/0ESV/vQzoR8AiE1Rusy59TfIt+YJ57d+8kfPkBHEJk/TlC/qylLHeUMycfud2N3Vj7SnsELizY7MQVCa+2B9YhOpJwyV7GvrQ2txT8zsZFjqmfOCVEzOb/bMyESC2oBqBJJebrDoCPZ6K7LAq+RoBTg+SCg7SGewdt8A34FzTcM38SvSmEYJDigE0UOGMDlTig+tEMWurya0KE2DsPrxsNxYAneoYPI7F6ZJ64Zy786FmLO6JDQVz6MyczADELftxOr+YB4eDf4g14lU9uDRB891FJzeTnzsMWU2r1c7sVU0+r86zuRhW0RgN4+xJcnD4hUX1M80A2LVXlfAASU6qDJW83L4xDZgf+Pj80/uZjupksdwWB8Ixpf820iUpDAFtwwweiV1p1IroMEZ3pSqCWV14vQb1AIOWx+/NvLA+zfj9KxGh1fJ0VbfoHtRmqAETquJc0AkWqao7XvyIjeKVXCgyR0yuco8Fs0lxsX2DTeBdzOSgGHd4kweWoHyHdYsyRKpJtwKPsDZuQhFAlMFGxv07OVOcp4xW2tRU0pbgcPNZmi5vq/mEzbREni6ON4q+ESUh7P4vc7Vv8s6hI+DVqQZ5suJTWG9pebJmuiLbqtKHXo= X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230028)(4636009)(346002)(376002)(396003)(136003)(39860400002)(451199021)(36840700001)(40470700004)(46966006)(356005)(82740400003)(81166007)(478600001)(36860700001)(19627235002)(47076005)(2616005)(83380400001)(70586007)(70206006)(426003)(336012)(54906003)(86362001)(4326008)(6916009)(82310400005)(40480700001)(6666004)(2906002)(30864003)(16526019)(316002)(186003)(7696005)(1076003)(26005)(2876002)(41300700001)(36756003)(5660300002)(8676002)(8936002)(40460700003)(36900700001)(559001)(579004)(44824005);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 May 2023 06:33:35.0856 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: defcaf19-8e4b-42d6-01e6-08db60d7cb7e X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: BN8NAM11FT050.eop-nam11.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR12MB7486 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain From: Abner Chang Signed-off-by: Abner Chang Cc: Isaac Oram Cc: Abdul Lateef Attar Cc: Nickle Wang Cc: Tinh Nguyen --- .../BaseManageabilityTransportHelper.c | 36 +++++++++---------- .../Common/KcsCommon.c | 12 +++---- .../Dxe/ManageabilityTransportKcs.c | 28 +++++++-------- .../Dxe/ManageabilityTransportMctp.c | 22 ++++++------ .../PldmProtocolLibrary/Dxe/PldmProtocolLib.c | 4 +-- .../IpmiProtocol/Common/IpmiProtocolCommon.c | 14 ++++---- .../Universal/IpmiProtocol/Dxe/IpmiProtocol.c | 14 ++++---- .../Universal/IpmiProtocol/Pei/IpmiPpi.c | 18 +++++----- .../Universal/IpmiProtocol/Smm/IpmiProtocol.c | 14 ++++---- .../MctpProtocol/Common/MctpProtocolCommon.c | 30 ++++++++-------- .../Universal/MctpProtocol/Dxe/MctpProtocol.c | 18 +++++----- .../PldmProtocol/Common/PldmProtocolCommon.c | 20 +++++------ .../Universal/PldmProtocol/Dxe/PldmProtocol.c | 14 ++++---- .../PldmSmbiosTransferDxe.c | 22 ++++++------ 14 files changed, 133 insertions(+), 133 deletions(-) diff --git a/Features/ManageabilityPkg/Library/BaseManageabilityTransportHe= lperLib/BaseManageabilityTransportHelper.c b/Features/ManageabilityPkg/Libr= ary/BaseManageabilityTransportHelperLib/BaseManageabilityTransportHelper.c index ce68f89531..f72957ea7f 100644 --- a/Features/ManageabilityPkg/Library/BaseManageabilityTransportHelperLib= /BaseManageabilityTransportHelper.c +++ b/Features/ManageabilityPkg/Library/BaseManageabilityTransportHelperLib= /BaseManageabilityTransportHelper.c @@ -49,7 +49,7 @@ HelperManageabilitySpecName ( } =20 if ((SpecificationGuid =3D=3D NULL) || IsZeroGuid (SpecificationGuid)) { - DEBUG ((DEBUG_ERROR, "%a: Improper input GUIDs, could be NULL or zero = GUID.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Improper input GUIDs, could be NULL or zero = GUID.\n", __func__)); return NULL; } =20 @@ -106,7 +106,7 @@ HelperManageabilityCheckSupportedSpec ( IsZeroGuid (ManageabilityProtocolToCheck) ) { - DEBUG ((DEBUG_ERROR, "%a: Improper input GUIDs, could be NULL or zero = GUID.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Improper input GUIDs, could be NULL or zero = GUID.\n", __func__)); return EFI_INVALID_PARAMETER; } =20 @@ -120,7 +120,7 @@ HelperManageabilityCheckSupportedSpec ( DEBUG (( DEBUG_MANAGEABILITY_INFO, "%a: Transport interface %s supports %s manageability specificatio= n.\n", - __FUNCTION__, + __func__, HelperManageabilitySpecName (TransportGuid), HelperManageabilitySpecName (ManageabilityProtocolToCheck) )); @@ -133,7 +133,7 @@ HelperManageabilityCheckSupportedSpec ( DEBUG (( DEBUG_ERROR, "%a: Transport interface %s doesn't support %s manageability specifica= tion.\n", - __FUNCTION__, + __func__, HelperManageabilitySpecName (TransportGuid), HelperManageabilitySpecName (ManageabilityProtocolToCheck) )); @@ -163,16 +163,16 @@ HelperAcquireManageabilityTransport ( CHAR16 *ManageabilityProtocolName; CHAR16 *ManageabilityTransportName; =20 - DEBUG ((DEBUG_INFO, "%a: Entry\n", __FUNCTION__)); + DEBUG ((DEBUG_INFO, "%a: Entry\n", __func__)); if ((TransportToken =3D=3D NULL) || (ManageabilityProtocolSpec =3D=3D NU= LL)) { - DEBUG ((DEBUG_ERROR, "%a: One of the required input parameters is NULL= .\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: One of the required input parameters is NULL= .\n", __func__)); return EFI_INVALID_PARAMETER; } =20 *TransportToken =3D NULL; ManageabilityProtocolName =3D HelperManageabilitySpecName (Manageability= ProtocolSpec); if (ManageabilityProtocolName =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: Unsupported Manageability Protocol Specifica= tion.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Unsupported Manageability Protocol Specifica= tion.\n", __func__)); return EFI_UNSUPPORTED; } =20 @@ -180,7 +180,7 @@ HelperAcquireManageabilityTransport ( =20 Status =3D AcquireTransportSession (ManageabilityProtocolSpec, Transport= Token); if (Status =3D=3D EFI_UNSUPPORTED) { - DEBUG ((DEBUG_ERROR, "%a: No supported transport interface for %s pack= et.\n", __FUNCTION__, ManageabilityProtocolName)); + DEBUG ((DEBUG_ERROR, "%a: No supported transport interface for %s pack= et.\n", __func__, ManageabilityProtocolName)); return Status; } =20 @@ -188,7 +188,7 @@ HelperAcquireManageabilityTransport ( DEBUG (( DEBUG_ERROR, "%a: Fail to acquire Manageability transport token for %s (%r).\n", - __FUNCTION__, + __func__, ManageabilityProtocolName, Status )); @@ -197,11 +197,11 @@ HelperAcquireManageabilityTransport ( =20 ManageabilityTransportName =3D HelperManageabilitySpecName ((*TransportT= oken)->Transport->ManageabilityTransportSpecification); if (ManageabilityTransportName =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: Unsupported Manageability Transport Interfac= e Specification\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Unsupported Manageability Transport Interfac= e Specification\n", __func__)); return EFI_UNSUPPORTED; } =20 - DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: This is the transfer session for = %s over %s\n", __FUNCTION__, ManageabilityProtocolName, ManageabilityTransp= ortName)); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: This is the transfer session for = %s over %s\n", __func__, ManageabilityProtocolName, ManageabilityTransportN= ame)); return Status; } =20 @@ -227,7 +227,7 @@ HelperInitManageabilityTransport ( EFI_STATUS Status; =20 if (TransportToken =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: TransportToken is invalid.\n", __FUNCTION__)= ); + DEBUG ((DEBUG_ERROR, "%a: TransportToken is invalid.\n", __func__)); return EFI_INVALID_PARAMETER; } =20 @@ -242,20 +242,20 @@ HelperInitManageabilityTransport ( ); if (EFI_ERROR (Status)) { if (Status =3D=3D EFI_UNSUPPORTED) { - DEBUG ((DEBUG_ERROR, "%a: Transport interface doesn't have reset= capability.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Transport interface doesn't have reset= capability.\n", __func__)); } else { - DEBUG ((DEBUG_ERROR, "%a: Fail to reset transport interface (%r)= .\n", __FUNCTION__, Status)); + DEBUG ((DEBUG_ERROR, "%a: Fail to reset transport interface (%r)= .\n", __func__, Status)); } =20 Status =3D EFI_DEVICE_ERROR; } else { Status =3D TransportToken->Transport->Function.Version1_0->Transpo= rtInit (TransportToken, HardwareInfo); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Transport interface is not able to use= after the reset (%r).\n", __FUNCTION__, Status)); + DEBUG ((DEBUG_ERROR, "%a: Transport interface is not able to use= after the reset (%r).\n", __func__, Status)); } } } else { - DEBUG ((DEBUG_ERROR, "%a: Transport interface is not able to use (%r= ).\n", __FUNCTION__, Status)); + DEBUG ((DEBUG_ERROR, "%a: Transport interface is not able to use (%r= ).\n", __func__, Status)); } } =20 @@ -343,7 +343,7 @@ HelperManageabilitySplitPayload ( DEBUG (( DEBUG_ERROR, "%a: (Preamble 0x%x + PostambleSize 0x%x) is greater than MaximumTra= nsferUnit 0x%x.\n", - __FUNCTION__, + __func__, PreambleSize, PostambleSize, MaximumTransferUnit @@ -373,7 +373,7 @@ HelperManageabilitySplitPayload ( } =20 if (TotalPayloadRemaining !=3D 0) { - DEBUG ((DEBUG_ERROR, "%a: Error processing multiple packages (TotalPay= loadRemaining !=3D 0)\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Error processing multiple packages (TotalPay= loadRemaining !=3D 0)\n", __func__)); FreePool (ThisMultiplePackages); return EFI_INVALID_PARAMETER; } diff --git a/Features/ManageabilityPkg/Library/ManageabilityTransportKcsLib= /Common/KcsCommon.c b/Features/ManageabilityPkg/Library/ManageabilityTransp= ortKcsLib/Common/KcsCommon.c index 4693e7f06f..d5b54c04be 100644 --- a/Features/ManageabilityPkg/Library/ManageabilityTransportKcsLib/Common= /KcsCommon.c +++ b/Features/ManageabilityPkg/Library/ManageabilityTransportKcsLib/Common= /KcsCommon.c @@ -143,7 +143,7 @@ KcsTransportWrite ( ((RequestData !=3D NULL) && (RequestDataSize =3D=3D 0)) ) { - DEBUG ((DEBUG_ERROR, "%a: Mismatched values of RequestData or RequestD= ataSize.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Mismatched values of RequestData or RequestD= ataSize.\n", __func__)); return EFI_INVALID_PARAMETER; } =20 @@ -152,7 +152,7 @@ KcsTransportWrite ( ((TransmitHeader !=3D NULL) && (TransmitHeaderSize =3D=3D 0)) ) { - DEBUG ((DEBUG_ERROR, "%a: Mismatched values of TransmitHeader or Trans= mitHeaderSize.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Mismatched values of TransmitHeader or Trans= mitHeaderSize.\n", __func__)); return EFI_INVALID_PARAMETER; } =20 @@ -161,7 +161,7 @@ KcsTransportWrite ( ((TransmitTrailer !=3D NULL) && (TransmitTrailerSize =3D=3D 0)) ) { - DEBUG ((DEBUG_ERROR, "%a: Mismatched values of TransmitTrailer or Tran= smitTrailerSize.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Mismatched values of TransmitTrailer or Tran= smitTrailerSize.\n", __func__)); return EFI_INVALID_PARAMETER; } =20 @@ -325,7 +325,7 @@ KcsTransportRead ( UINT32 ReadLength; =20 if ((DataByte =3D=3D NULL) || (*Length =3D=3D 0)) { - DEBUG ((DEBUG_ERROR, "%a: Either DataByte is NULL or Length is 0.\n", = __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Either DataByte is NULL or Length is 0.\n", = __func__)); return EFI_INVALID_PARAMETER; } =20 @@ -430,12 +430,12 @@ KcsTransportSendCommand ( CHAR16 *CompletionCodeStr; =20 if ((RequestData !=3D NULL) && (RequestDataSize =3D=3D 0)) { - DEBUG ((DEBUG_ERROR, "%a: Mismatched values of RequestData and Request= DataSize\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Mismatched values of RequestData and Request= DataSize\n", __func__)); return EFI_INVALID_PARAMETER; } =20 if ((ResponseData !=3D NULL) && ((ResponseDataSize !=3D NULL) && (*Respo= nseDataSize =3D=3D 0))) { - DEBUG ((DEBUG_ERROR, "%a: Mismatched values of ResponseData and Respon= seDataSize\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Mismatched values of ResponseData and Respon= seDataSize\n", __func__)); return EFI_INVALID_PARAMETER; } =20 diff --git a/Features/ManageabilityPkg/Library/ManageabilityTransportKcsLib= /Dxe/ManageabilityTransportKcs.c b/Features/ManageabilityPkg/Library/Manage= abilityTransportKcsLib/Dxe/ManageabilityTransportKcs.c index c690bea74e..9bd8ff4c0a 100644 --- a/Features/ManageabilityPkg/Library/ManageabilityTransportKcsLib/Dxe/Ma= nageabilityTransportKcs.c +++ b/Features/ManageabilityPkg/Library/ManageabilityTransportKcsLib/Dxe/Ma= nageabilityTransportKcs.c @@ -58,12 +58,12 @@ KcsTransportInit ( CHAR16 *ManageabilityProtocolName; =20 if (TransportToken =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: Invalid transport token.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Invalid transport token.\n", __func__)); return EFI_INVALID_PARAMETER; } =20 if (HardwareInfo.Kcs =3D=3D NULL) { - DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Hardware information is not pro= vided, use dfault settings.\n", __FUNCTION__)); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Hardware information is not pro= vided, use dfault settings.\n", __func__)); mKcsHardwareInfo.MemoryMap =3D MANAGEABILITY_TRANSP= ORT_KCS_IO_MAP_IO; mKcsHardwareInfo.IoBaseAddress.IoAddress16 =3D PcdGet16 (PcdIpmiKcs= IoBaseAddress); mKcsHardwareInfo.IoDataInAddress.IoAddress16 =3D mKcsHardwareInfo.IoB= aseAddress.IoAddress16 + IPMI_KCS_DATA_IN_REGISTER_OFFSET; @@ -82,9 +82,9 @@ KcsTransportInit ( // Get protocol specification name. ManageabilityProtocolName =3D HelperManageabilitySpecName (TransportToke= n->ManageabilityProtocolSpecification); =20 - DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: KCS transport hardware for %s is:= \n", __FUNCTION__, ManageabilityProtocolName)); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: KCS transport hardware for %s is:= \n", __func__, ManageabilityProtocolName)); if (mKcsHardwareInfo.MemoryMap) { - DEBUG ((DEBUG_MANAGEABILITY_INFO, "Memory Map I/O\n", __FUNCTION__)); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "Memory Map I/O\n", __func__)); DEBUG ((DEBUG_MANAGEABILITY_INFO, "Base Memory Address : 0x%08x\n", mK= csHardwareInfo.IoBaseAddress.IoAddress32)); DEBUG ((DEBUG_MANAGEABILITY_INFO, "Data in Address : 0x%08x\n", mK= csHardwareInfo.IoDataInAddress.IoAddress32)); DEBUG ((DEBUG_MANAGEABILITY_INFO, "Data out Address : 0x%08x\n", mK= csHardwareInfo.IoDataOutAddress.IoAddress32)); @@ -134,7 +134,7 @@ KcsTransportStatus ( UINT8 TransportStatus; =20 if (TransportToken =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: Invalid transport token.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Invalid transport token.\n", __func__)); return EFI_INVALID_PARAMETER; } =20 @@ -223,7 +223,7 @@ KcsTransportTransmitReceive ( MANAGEABILITY_TRANSPORT_ADDITIONAL_STATUS AdditionalStatus; =20 if ((TransportToken =3D=3D NULL) || (TransferToken =3D=3D NULL)) { - DEBUG ((DEBUG_ERROR, "%a: Invalid transport token or transfer token.\n= ", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Invalid transport token or transfer token.\n= ", __func__)); return; } =20 @@ -267,12 +267,12 @@ AcquireTransportSession ( MANAGEABILITY_TRANSPORT_KCS *KcsTransportToken; =20 if (ManageabilityProtocolSpec =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: No Manageability protocol specification spec= ified.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: No Manageability protocol specification spec= ified.\n", __func__)); return EFI_INVALID_PARAMETER; } =20 if (TransportToken =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: TransportToken is NULL.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: TransportToken is NULL.\n", __func__)); return EFI_INVALID_PARAMETER; } =20 @@ -283,25 +283,25 @@ AcquireTransportSession ( ManageabilityProtocolSpec ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Protocol is not supported on this transport = interface.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Protocol is not supported on this transport = interface.\n", __func__)); return EFI_UNSUPPORTED; } =20 if (mSingleSessionToken !=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: This manageability transport library only su= pports one session transport token.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: This manageability transport library only su= pports one session transport token.\n", __func__)); return EFI_OUT_OF_RESOURCES; } =20 KcsTransportToken =3D AllocateZeroPool (sizeof (MANAGEABILITY_TRANSPORT_= KCS)); if (KcsTransportToken =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: Fail to allocate memory for MANAGEABILITY_TR= ANSPORT_KCS\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Fail to allocate memory for MANAGEABILITY_TR= ANSPORT_KCS\n", __func__)); return EFI_OUT_OF_RESOURCES; } =20 KcsTransportToken->Token.Transport =3D AllocateZeroPool (sizeof (MANAGEA= BILITY_TRANSPORT)); if (KcsTransportToken->Token.Transport =3D=3D NULL) { FreePool (KcsTransportToken); - DEBUG ((DEBUG_ERROR, "%a: Fail to allocate memory for MANAGEABILITY_TR= ANSPORT\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Fail to allocate memory for MANAGEABILITY_TR= ANSPORT\n", __func__)); return EFI_OUT_OF_RESOURCES; } =20 @@ -312,7 +312,7 @@ AcquireTransportSession ( KcsTransportToken->Token.Transport->TransportName = =3D L"KCS"; KcsTransportToken->Token.Transport->Function.Version1_0 = =3D AllocateZeroPool (sizeof (MANAGEABILITY_TRANSPORT_FUNCTION_V1_0)); if (KcsTransportToken->Token.Transport->Function.Version1_0 =3D=3D NULL)= { - DEBUG ((DEBUG_ERROR, "%a: Fail to allocate memory for MANAGEABILITY_TR= ANSPORT_FUNCTION_V1_0\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Fail to allocate memory for MANAGEABILITY_TR= ANSPORT_FUNCTION_V1_0\n", __func__)); FreePool (KcsTransportToken); FreePool (KcsTransportToken->Token.Transport); return EFI_OUT_OF_RESOURCES; @@ -406,7 +406,7 @@ ReleaseTransportSession ( } =20 if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Fail to release KCS transport token (%r).\n"= , __FUNCTION__, Status)); + DEBUG ((DEBUG_ERROR, "%a: Fail to release KCS transport token (%r).\n"= , __func__, Status)); } =20 return Status; diff --git a/Features/ManageabilityPkg/Library/ManageabilityTransportMctpLi= b/Dxe/ManageabilityTransportMctp.c b/Features/ManageabilityPkg/Library/Mana= geabilityTransportMctpLib/Dxe/ManageabilityTransportMctp.c index eaf02bbb0a..c520e2302d 100644 --- a/Features/ManageabilityPkg/Library/ManageabilityTransportMctpLib/Dxe/M= anageabilityTransportMctp.c +++ b/Features/ManageabilityPkg/Library/ManageabilityTransportMctpLib/Dxe/M= anageabilityTransportMctp.c @@ -86,7 +86,7 @@ MctpTransportStatus ( ) { if (TransportToken =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: Invalid transport token.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Invalid transport token.\n", __func__)); return EFI_INVALID_PARAMETER; } =20 @@ -161,7 +161,7 @@ MctpTransportTransmitReceive ( MANAGEABILITY_MCTP_TRANSPORT_HEADER *TransmitHeader; =20 if (TransportToken =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: Invalid transport token.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Invalid transport token.\n", __func__)); TransferToken->TransportAdditionalStatus =3D MANAGEABILITY_TRANSPORT_A= DDITIONAL_STATUS_NOT_AVAILABLE; return; } @@ -183,7 +183,7 @@ MctpTransportTransmitReceive ( // // Dxe MCTP Protocol is not installed. // - DEBUG ((DEBUG_ERROR, "%a: EDKII MCTP protocol is not found - %r\n", = __FUNCTION__, Status)); + DEBUG ((DEBUG_ERROR, "%a: EDKII MCTP protocol is not found - %r\n", = __func__, Status)); return; } } @@ -191,7 +191,7 @@ MctpTransportTransmitReceive ( DEBUG (( DEBUG_MANAGEABILITY_INFO, "%a: MCTP message type: 0x%x, SourceEndpointId: 0x%x, DestinationEndpo= intId: 0x%x\n", - __FUNCTION__, + __func__, TransmitHeader->MessageHeader.MessageType, TransmitHeader->SourceEndpointId, TransmitHeader->DestinationEndpointId @@ -242,7 +242,7 @@ AcquireTransportSession ( MANAGEABILITY_TRANSPORT_MCTP *MctpTransportToken; =20 if (ManageabilityProtocolSpec =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: No Manageability protocol specification spec= ified.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: No Manageability protocol specification spec= ified.\n", __func__)); return EFI_INVALID_PARAMETER; } =20 @@ -253,25 +253,25 @@ AcquireTransportSession ( ManageabilityProtocolSpec ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Protocol is not supported on this transport = interface.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Protocol is not supported on this transport = interface.\n", __func__)); return EFI_UNSUPPORTED; } =20 if (mSingleSessionToken !=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: This manageability transport library only su= pports one session transport token.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: This manageability transport library only su= pports one session transport token.\n", __func__)); return EFI_OUT_OF_RESOURCES; } =20 MctpTransportToken =3D (MANAGEABILITY_TRANSPORT_MCTP *)AllocateZeroPool = (sizeof (MANAGEABILITY_TRANSPORT_MCTP)); if (MctpTransportToken =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: Fail to allocate memory for MANAGEABILITY_TR= ANSPORT_MCTP\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Fail to allocate memory for MANAGEABILITY_TR= ANSPORT_MCTP\n", __func__)); return EFI_OUT_OF_RESOURCES; } =20 MctpTransportToken->Token.Transport =3D AllocateZeroPool (sizeof (MANAGE= ABILITY_TRANSPORT)); if (MctpTransportToken->Token.Transport =3D=3D NULL) { FreePool (MctpTransportToken); - DEBUG ((DEBUG_ERROR, "%a: Fail to allocate memory for MANAGEABILITY_TR= ANSPORT\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Fail to allocate memory for MANAGEABILITY_TR= ANSPORT\n", __func__)); return EFI_OUT_OF_RESOURCES; } =20 @@ -282,7 +282,7 @@ AcquireTransportSession ( MctpTransportToken->Token.Transport->TransportName = =3D L"MCTP"; MctpTransportToken->Token.Transport->Function.Version1_0 = =3D AllocateZeroPool (sizeof (MANAGEABILITY_TRANSPORT_FUNCTION_V1_0)); if (MctpTransportToken->Token.Transport->Function.Version1_0 =3D=3D NULL= ) { - DEBUG ((DEBUG_ERROR, "%a: Fail to allocate memory for MANAGEABILITY_TR= ANSPORT_FUNCTION_V1_0\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Fail to allocate memory for MANAGEABILITY_TR= ANSPORT_FUNCTION_V1_0\n", __func__)); FreePool (MctpTransportToken); FreePool (MctpTransportToken->Token.Transport); return EFI_OUT_OF_RESOURCES; @@ -360,7 +360,7 @@ ReleaseTransportSession ( } =20 if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Fail to release MCTP transport token (%r).\n= ", __FUNCTION__, Status)); + DEBUG ((DEBUG_ERROR, "%a: Fail to release MCTP transport token (%r).\n= ", __func__, Status)); } =20 return Status; diff --git a/Features/ManageabilityPkg/Library/PldmProtocolLibrary/Dxe/Pldm= ProtocolLib.c b/Features/ManageabilityPkg/Library/PldmProtocolLibrary/Dxe/P= ldmProtocolLib.c index 96727054ce..267bd8fbc1 100644 --- a/Features/ManageabilityPkg/Library/PldmProtocolLibrary/Dxe/PldmProtoco= lLib.c +++ b/Features/ManageabilityPkg/Library/PldmProtocolLibrary/Dxe/PldmProtoco= lLib.c @@ -55,12 +55,12 @@ PldmSubmitCommand ( // // Dxe PLDM Protocol is not installed. So, PLDM device is not presen= t. // - DEBUG ((DEBUG_ERROR, "%a: EDKII PLDM protocol is not found - %r\n", = __FUNCTION__, Status)); + DEBUG ((DEBUG_ERROR, "%a: EDKII PLDM protocol is not found - %r\n", = __func__, Status)); return EFI_NOT_FOUND; } } =20 - DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: PLDM Type: 0x%x, Command: 0x%x\n"= , __FUNCTION__, PldmType, Command)); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: PLDM Type: 0x%x, Command: 0x%x\n"= , __func__, PldmType, Command)); if ((RequestData !=3D NULL) && (RequestDataSize !=3D 0)) { HelperManageabilityDebugPrint ((VOID *)RequestData, RequestDataSize, "= PLDM PLDM application layer Type/Command specific request payload\n"); } diff --git a/Features/ManageabilityPkg/Universal/IpmiProtocol/Common/IpmiPr= otocolCommon.c b/Features/ManageabilityPkg/Universal/IpmiProtocol/Common/Ip= miProtocolCommon.c index b055bad7da..6a09280bcc 100644 --- a/Features/ManageabilityPkg/Universal/IpmiProtocol/Common/IpmiProtocolC= ommon.c +++ b/Features/ManageabilityPkg/Universal/IpmiProtocol/Common/IpmiProtocolC= ommon.c @@ -39,7 +39,7 @@ SetupIpmiTransportHardwareInformation ( =20 KcsHardwareInfo =3D AllocatePool (sizeof (MANAGEABILITY_TRANSPORT_KCS_HA= RDWARE_INFO)); if (KcsHardwareInfo =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: Not enough memory for MANAGEABILITY_TRANSPOR= T_KCS_HARDWARE_INFO.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Not enough memory for MANAGEABILITY_TRANSPOR= T_KCS_HARDWARE_INFO.\n", __func__)); return EFI_OUT_OF_RESOURCES; } =20 @@ -54,7 +54,7 @@ SetupIpmiTransportHardwareInformation ( HardwareInformation->Kcs =3D KcsHardwareInfo; return EFI_SUCCESS; } else { - DEBUG ((DEBUG_ERROR, "%a: No implementation of setting hardware inform= ation.", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: No implementation of setting hardware inform= ation.", __func__)); ASSERT (FALSE); } =20 @@ -131,7 +131,7 @@ SetupIpmiRequestTransportPacket ( *PacketBodySize =3D 0; } } else { - DEBUG ((DEBUG_ERROR, "%a: No implementation of building up packet.", _= _FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: No implementation of building up packet.", _= _func__)); ASSERT (FALSE); } =20 @@ -179,7 +179,7 @@ CommonIpmiSubmitCommand ( UINT16 TrailerSize; =20 if (TransportToken =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: No transport toke for IPMI\n", __FUNCTION__)= ); + DEBUG ((DEBUG_ERROR, "%a: No transport toke for IPMI\n", __func__)); return EFI_UNSUPPORTED; } =20 @@ -188,7 +188,7 @@ CommonIpmiSubmitCommand ( &TransportAdd= itionalStatus ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Transport for IPMI has problem - (%r)\n", __= FUNCTION__, Status)); + DEBUG ((DEBUG_ERROR, "%a: Transport for IPMI has problem - (%r)\n", __= func__, Status)); return Status; } =20 @@ -208,7 +208,7 @@ CommonIpmiSubmitCommand ( &TrailerSize ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Fail to build packets - (%r)\n", __FUNCTION_= _, Status)); + DEBUG ((DEBUG_ERROR, "%a: Fail to build packets - (%r)\n", __func__, S= tatus)); return Status; } =20 @@ -256,7 +256,7 @@ CommonIpmiSubmitCommand ( Status =3D TransferToken.TransferStatus; TransportAdditionalStatus =3D TransferToken.TransportAdditionalStatus; if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Failed to send IPMI command.\n", __FUNCTION_= _)); + DEBUG ((DEBUG_ERROR, "%a: Failed to send IPMI command.\n", __func__)); return Status; } =20 diff --git a/Features/ManageabilityPkg/Universal/IpmiProtocol/Dxe/IpmiProto= col.c b/Features/ManageabilityPkg/Universal/IpmiProtocol/Dxe/IpmiProtocol.c index 51d5c7f0ba..1660cfbbd5 100644 --- a/Features/ManageabilityPkg/Universal/IpmiProtocol/Dxe/IpmiProtocol.c +++ b/Features/ManageabilityPkg/Universal/IpmiProtocol/Dxe/IpmiProtocol.c @@ -97,26 +97,26 @@ DxeIpmiEntry ( &mTransportToken ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Failed to acquire transport interface for IP= MI protocol - %r\n", __FUNCTION__, Status)); + DEBUG ((DEBUG_ERROR, "%a: Failed to acquire transport interface for IP= MI protocol - %r\n", __func__, Status)); return Status; } =20 Status =3D GetTransportCapability (mTransportToken, &TransportCapability= ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Failed to GetTransportCapability().\n", __FU= NCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Failed to GetTransportCapability().\n", __fu= nc__)); return Status; } =20 TransportMaximumPayload =3D MANAGEABILITY_TRANSPORT_PAYLOAD_SIZE_FROM_CA= PABILITY (TransportCapability); if (TransportMaximumPayload =3D=3D (1 << MANAGEABILITY_TRANSPORT_CAPABIL= ITY_MAXIMUM_PAYLOAD_NOT_AVAILABLE)) { - DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface maximum pay= load is undefined.\n", __FUNCTION__)); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface maximum pay= load is undefined.\n", __func__)); } else { TransportMaximumPayload -=3D 1; - DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface for IPMI pr= otocol has maximum payload %x.\n", __FUNCTION__, TransportMaximumPayload)); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface for IPMI pr= otocol has maximum payload %x.\n", __func__, TransportMaximumPayload)); } =20 mTransportName =3D HelperManageabilitySpecName (mTransportToken->Transpo= rt->ManageabilityTransportSpecification); - DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: IPMI protocol over %s.\n", __FUNC= TION__, mTransportName)); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: IPMI protocol over %s.\n", __func= __, mTransportName)); =20 // // Setup hardware information according to the transport interface. @@ -126,7 +126,7 @@ DxeIpmiEntry ( ); if (EFI_ERROR (Status)) { if (Status =3D=3D EFI_UNSUPPORTED) { - DEBUG ((DEBUG_ERROR, "%a: No hardware information of %s transport in= terface.\n", __FUNCTION__, mTransportName)); + DEBUG ((DEBUG_ERROR, "%a: No hardware information of %s transport in= terface.\n", __func__, mTransportName)); } =20 return Status; @@ -151,7 +151,7 @@ DxeIpmiEntry ( (VOID **)&mIpmiProtocol ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Failed to install IPMI protocol - %r\n", __F= UNCTION__, Status)); + DEBUG ((DEBUG_ERROR, "%a: Failed to install IPMI protocol - %r\n", __f= unc__, Status)); } =20 return Status; diff --git a/Features/ManageabilityPkg/Universal/IpmiProtocol/Pei/IpmiPpi.c= b/Features/ManageabilityPkg/Universal/IpmiProtocol/Pei/IpmiPpi.c index 4bbdbf78fe..a6bdb27464 100644 --- a/Features/ManageabilityPkg/Universal/IpmiProtocol/Pei/IpmiPpi.c +++ b/Features/ManageabilityPkg/Universal/IpmiProtocol/Pei/IpmiPpi.c @@ -96,13 +96,13 @@ PeiIpmiEntry ( // PeiIpmiPpiinternal =3D (PEI_IPMI_PPI_INTERNAL *)AllocateZeroPool (sizeof= (PEI_IPMI_PPI_INTERNAL)); if (PeiIpmiPpiinternal =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: Not enough memory for PEI_IPMI_PPI_INTERNAL.= \n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Not enough memory for PEI_IPMI_PPI_INTERNAL.= \n", __func__)); return EFI_OUT_OF_RESOURCES; } =20 PpiDescriptor =3D (EFI_PEI_PPI_DESCRIPTOR *)AllocateZeroPool (sizeof (EF= I_PEI_PPI_DESCRIPTOR)); if (PpiDescriptor =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: Not enough memory for EFI_PEI_PPI_DESCRIPTOR= .\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Not enough memory for EFI_PEI_PPI_DESCRIPTOR= .\n", __func__)); return EFI_OUT_OF_RESOURCES; } =20 @@ -118,26 +118,26 @@ PeiIpmiEntry ( &PeiIpmiPpiinternal->TransportToken ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Failed to acquire transport interface for IP= MI protocol - %r\n", __FUNCTION__, Status)); + DEBUG ((DEBUG_ERROR, "%a: Failed to acquire transport interface for IP= MI protocol - %r\n", __func__, Status)); return Status; } =20 Status =3D GetTransportCapability (PeiIpmiPpiinternal->TransportToken, &= PeiIpmiPpiinternal->TransportCapability); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Failed to GetTransportCapability().\n", __FU= NCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Failed to GetTransportCapability().\n", __fu= nc__)); return Status; } =20 PeiIpmiPpiinternal->TransportMaximumPayload =3D MANAGEABILITY_TRANSPORT_= PAYLOAD_SIZE_FROM_CAPABILITY (PeiIpmiPpiinternal->TransportCapability); if (PeiIpmiPpiinternal->TransportMaximumPayload =3D=3D (1 << MANAGEABIL= ITY_TRANSPORT_CAPABILITY_MAXIMUM_PAYLOAD_NOT_AVAILABLE)) { - DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface maximum pay= load is undefined.\n", __FUNCTION__)); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface maximum pay= load is undefined.\n", __func__)); } else { PeiIpmiPpiinternal->TransportMaximumPayload -=3D 1; - DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface for IPMI pr= otocol has maximum payload 0x%x.\n", __FUNCTION__, PeiIpmiPpiinternal->Tran= sportMaximumPayload)); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface for IPMI pr= otocol has maximum payload 0x%x.\n", __func__, PeiIpmiPpiinternal->Transpor= tMaximumPayload)); } =20 TransportName =3D HelperManageabilitySpecName (PeiIpmiPpiinternal->Trans= portToken->Transport->ManageabilityTransportSpecification); - DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: IPMI protocol over %s.\n", __FUNC= TION__, TransportName)); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: IPMI protocol over %s.\n", __func= __, TransportName)); =20 // // Setup hardware information according to the transport interface. @@ -147,7 +147,7 @@ PeiIpmiEntry ( ); if (EFI_ERROR (Status)) { if (Status =3D=3D EFI_UNSUPPORTED) { - DEBUG ((DEBUG_ERROR, "%a: No hardware information of %s transport in= terface.\n", __FUNCTION__, TransportName)); + DEBUG ((DEBUG_ERROR, "%a: No hardware information of %s transport in= terface.\n", __func__, TransportName)); } =20 return Status; @@ -169,7 +169,7 @@ PeiIpmiEntry ( // Status =3D PeiServicesInstallPpi (PpiDescriptor); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Failed to install IPMI PPI - %r\n", __FUNCTI= ON__, Status)); + DEBUG ((DEBUG_ERROR, "%a: Failed to install IPMI PPI - %r\n", __func__= , Status)); } =20 return Status; diff --git a/Features/ManageabilityPkg/Universal/IpmiProtocol/Smm/IpmiProto= col.c b/Features/ManageabilityPkg/Universal/IpmiProtocol/Smm/IpmiProtocol.c index e4cd166b7a..aaa48b24e2 100644 --- a/Features/ManageabilityPkg/Universal/IpmiProtocol/Smm/IpmiProtocol.c +++ b/Features/ManageabilityPkg/Universal/IpmiProtocol/Smm/IpmiProtocol.c @@ -98,26 +98,26 @@ SmmIpmiEntry ( &mTransportToken ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Failed to acquire transport interface for IP= MI protocol - %r\n", __FUNCTION__, Status)); + DEBUG ((DEBUG_ERROR, "%a: Failed to acquire transport interface for IP= MI protocol - %r\n", __func__, Status)); return Status; } =20 Status =3D GetTransportCapability (mTransportToken, &TransportCapability= ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Failed to GetTransportCapability().\n", __FU= NCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Failed to GetTransportCapability().\n", __fu= nc__)); return Status; } =20 TransportMaximumPayload =3D MANAGEABILITY_TRANSPORT_PAYLOAD_SIZE_FROM_CA= PABILITY (TransportCapability); if (TransportMaximumPayload =3D=3D (1 << MANAGEABILITY_TRANSPORT_CAPABIL= ITY_MAXIMUM_PAYLOAD_NOT_AVAILABLE)) { - DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface maximum pay= load is undefined.\n", __FUNCTION__)); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface maximum pay= load is undefined.\n", __func__)); } else { TransportMaximumPayload -=3D 1; - DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface for IPMI pr= otocol has maximum payload 0x%x.\n", __FUNCTION__, TransportMaximumPayload)= ); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface for IPMI pr= otocol has maximum payload 0x%x.\n", __func__, TransportMaximumPayload)); } =20 mTransportName =3D HelperManageabilitySpecName (mTransportToken->Transpo= rt->ManageabilityTransportSpecification); - DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: IPMI protocol over %s.\n", __FUNC= TION__, mTransportName)); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: IPMI protocol over %s.\n", __func= __, mTransportName)); =20 // // Setup hardware information according to the transport interface. @@ -127,7 +127,7 @@ SmmIpmiEntry ( ); if (EFI_ERROR (Status)) { if (Status =3D=3D EFI_UNSUPPORTED) { - DEBUG ((DEBUG_ERROR, "%a: No hardware information of %s transport in= terface.\n", __FUNCTION__, mTransportName)); + DEBUG ((DEBUG_ERROR, "%a: No hardware information of %s transport in= terface.\n", __func__, mTransportName)); } =20 return Status; @@ -152,7 +152,7 @@ SmmIpmiEntry ( (VOID **)&mIpmiProtocol ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Failed to install IPMI SMM protocol - %r\n",= __FUNCTION__, Status)); + DEBUG ((DEBUG_ERROR, "%a: Failed to install IPMI SMM protocol - %r\n",= __func__, Status)); } =20 return Status; diff --git a/Features/ManageabilityPkg/Universal/MctpProtocol/Common/MctpPr= otocolCommon.c b/Features/ManageabilityPkg/Universal/MctpProtocol/Common/Mc= tpProtocolCommon.c index db572ec85a..1ad48efdc7 100644 --- a/Features/ManageabilityPkg/Universal/MctpProtocol/Common/MctpProtocolC= ommon.c +++ b/Features/ManageabilityPkg/Universal/MctpProtocol/Common/MctpProtocolC= ommon.c @@ -52,7 +52,7 @@ SetupMctpTransportHardwareInformation ( if (CompareGuid (&gManageabilityTransportKcsGuid, TransportToken->Transp= ort->ManageabilityTransportSpecification)) { KcsHardwareInfo =3D AllocatePool (sizeof (MANAGEABILITY_TRANSPORT_KCS_= HARDWARE_INFO)); if (KcsHardwareInfo =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: Not enough memory for MANAGEABILITY_TRANSP= ORT_KCS_HARDWARE_INFO.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Not enough memory for MANAGEABILITY_TRANSP= ORT_KCS_HARDWARE_INFO.\n", __func__)); return EFI_OUT_OF_RESOURCES; } =20 @@ -76,7 +76,7 @@ SetupMctpTransportHardwareInformation ( HardwareInformation->Kcs =3D KcsHardwareInfo; return EFI_SUCCESS; } else { - DEBUG ((DEBUG_ERROR, "%a: No implementation of setting hardware inform= ation.", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: No implementation of setting hardware inform= ation.", __func__)); ASSERT (FALSE); } =20 @@ -140,20 +140,20 @@ SetupMctpRequestTransportPacket ( (PacketTrailer =3D=3D NULL) || (PacketTrailerSize =3D=3D NULL) ) { - DEBUG ((DEBUG_ERROR, "%a: One or more than one of the input parameter = is invalid.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: One or more than one of the input parameter = is invalid.\n", __func__)); return EFI_INVALID_PARAMETER; } =20 if (CompareGuid (&gManageabilityTransportKcsGuid, TransportToken->Transp= ort->ManageabilityTransportSpecification)) { MctpKcsHeader =3D (MANAGEABILITY_MCTP_KCS_HEADER *)AllocateZeroPool (s= izeof (MANAGEABILITY_MCTP_KCS_HEADER)); if (MctpKcsHeader =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: Not enough resource for MANAGEABILITY_MCTP= _KCS_HEADER.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Not enough resource for MANAGEABILITY_MCTP= _KCS_HEADER.\n", __func__)); return EFI_OUT_OF_RESOURCES; } =20 Pec =3D (UINT8 *)AllocateZeroPool (sizeof (UINT8)); if (Pec =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: Not enough resource for PEC.\n", __FUNCTIO= N__)); + DEBUG ((DEBUG_ERROR, "%a: Not enough resource for PEC.\n", __func__)= ); FreePool (MctpKcsHeader); return EFI_OUT_OF_RESOURCES; } @@ -165,7 +165,7 @@ SetupMctpRequestTransportPacket ( =20 ThisPackage =3D (UINT8 *)AllocateZeroPool (MctpKcsHeader->ByteCount); if (ThisPackage =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: Not enough resource for package.\n", __FUN= CTION__)); + DEBUG ((DEBUG_ERROR, "%a: Not enough resource for package.\n", __fun= c__)); FreePool (MctpKcsHeader); FreePool (Pec); return EFI_OUT_OF_RESOURCES; @@ -203,7 +203,7 @@ SetupMctpRequestTransportPacket ( *PacketTrailerSize =3D 1; return EFI_SUCCESS; } else { - DEBUG ((DEBUG_ERROR, "%a: No implementation of building up packet.", _= _FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: No implementation of building up packet.", _= _func__)); ASSERT (FALSE); } =20 @@ -269,7 +269,7 @@ CommonMctpSubmitMessage ( MANAGEABILITY_TRANSMISSION_PACKAGE_ATTR *ThisPackage; =20 if (TransportToken =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: No transport toke for MCTP\n", __FUNCTION__)= ); + DEBUG ((DEBUG_ERROR, "%a: No transport toke for MCTP\n", __func__)); return EFI_UNSUPPORTED; } =20 @@ -278,7 +278,7 @@ CommonMctpSubmitMessage ( AdditionalTra= nsferError ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Transport %s for MCTP has problem - (%r)\n",= __FUNCTION__, mTransportName, Status)); + DEBUG ((DEBUG_ERROR, "%a: Transport %s for MCTP has problem - (%r)\n",= __func__, mTransportName, Status)); return Status; } =20 @@ -292,7 +292,7 @@ CommonMctpSubmitMessage ( &MultiPackages ); if (EFI_ERROR (Status) || (MultiPackages =3D=3D NULL)) { - DEBUG ((DEBUG_ERROR, "%a: Fails to split payload into multiple package= s - (%r)\n", __FUNCTION__, mTransportName, Status)); + DEBUG ((DEBUG_ERROR, "%a: Fails to split payload into multiple package= s - (%r)\n", __func__, mTransportName, Status)); return Status; } =20 @@ -342,7 +342,7 @@ CommonMctpSubmitMessage ( &MctpTransportTrailerSize ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Fail to build packets - (%r)\n", __FUNCTIO= N__, Status)); + DEBUG ((DEBUG_ERROR, "%a: Fail to build packets - (%r)\n", __func__,= Status)); return Status; } =20 @@ -373,7 +373,7 @@ CommonMctpSubmitMessage ( DEBUG (( DEBUG_MANAGEABILITY_INFO, "%a: Send MCTP message type: 0x%x, from source endpoint ID: 0x%x to = destination ID 0x%x: Request size: 0x%x, Response size: 0x%x\n", - __FUNCTION__, + __func__, MctpType, MctpSourceEndpointId, MctpDestinationEndpointId, @@ -426,7 +426,7 @@ CommonMctpSubmitMessage ( Status =3D TransferToken.TransferStatus; *AdditionalTransferError =3D TransferToken.TransportAdditionalStatus; if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Failed to send MCTP command over %s\n", __= FUNCTION__, mTransportName)); + DEBUG ((DEBUG_ERROR, "%a: Failed to send MCTP command over %s\n", __= func__, mTransportName)); FreePool (MultiPackages); return Status; } @@ -449,7 +449,7 @@ CommonMctpSubmitMessage ( DEBUG (( DEBUG_MANAGEABILITY_INFO, "%a: Retrieve MCTP message Response size: 0x%x\n", - __FUNCTION__, + __func__, TransferToken.ReceivePackage.ReceiveSizeInByte )); TransportToken->Transport->Function.Version1_0->TransportTransmitReceive= ( @@ -464,7 +464,7 @@ CommonMctpSubmitMessage ( *ResponseDataSize =3D TransferToken.ReceivePackage.ReceiveSizeInB= yte; Status =3D TransferToken.TransferStatus; if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Failed to send MCTP command over %s: %r\n", = __FUNCTION__, mTransportName, Status)); + DEBUG ((DEBUG_ERROR, "%a: Failed to send MCTP command over %s: %r\n", = __func__, mTransportName, Status)); return Status; } =20 diff --git a/Features/ManageabilityPkg/Universal/MctpProtocol/Dxe/MctpProto= col.c b/Features/ManageabilityPkg/Universal/MctpProtocol/Dxe/MctpProtocol.c index a1cac61e27..88bfd9b7e7 100644 --- a/Features/ManageabilityPkg/Universal/MctpProtocol/Dxe/MctpProtocol.c +++ b/Features/ManageabilityPkg/Universal/MctpProtocol/Dxe/MctpProtocol.c @@ -74,7 +74,7 @@ MctpSubmitMessage ( EFI_STATUS Status; =20 if ((RequestData =3D=3D NULL) && (ResponseData =3D=3D NULL)) { - DEBUG ((DEBUG_ERROR, "%a: Both RequestData and ResponseData are NULL\n= ", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Both RequestData and ResponseData are NULL\n= ", __func__)); return EFI_INVALID_PARAMETER; } =20 @@ -127,26 +127,26 @@ DxeMctpEntry ( &mTransportToken ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Failed to acquire transport interface for MC= TP protocol - %r\n", __FUNCTION__, Status)); + DEBUG ((DEBUG_ERROR, "%a: Failed to acquire transport interface for MC= TP protocol - %r\n", __func__, Status)); return Status; } =20 Status =3D GetTransportCapability (mTransportToken, &TransportCapability= ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Failed to GetTransportCapability().\n", __FU= NCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Failed to GetTransportCapability().\n", __fu= nc__)); return Status; } =20 mTransportMaximumPayload =3D MANAGEABILITY_TRANSPORT_PAYLOAD_SIZE_FROM_C= APABILITY (TransportCapability); if (mTransportMaximumPayload =3D=3D (1 << MANAGEABILITY_TRANSPORT_CAPABI= LITY_MAXIMUM_PAYLOAD_NOT_AVAILABLE)) { - DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface maximum pay= load is undefined.\n", __FUNCTION__)); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface maximum pay= load is undefined.\n", __func__)); } else { mTransportMaximumPayload -=3D 1; - DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface for MCTP pr= otocol has maximum payload 0x%x.\n", __FUNCTION__, mTransportMaximumPayload= )); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface for MCTP pr= otocol has maximum payload 0x%x.\n", __func__, mTransportMaximumPayload)); } =20 mTransportName =3D HelperManageabilitySpecName (mTransportToken->Transpo= rt->ManageabilityTransportSpecification); - DEBUG ((DEBUG_ERROR, "%a: MCTP protocol over %s.\n", __FUNCTION__, mTran= sportName)); + DEBUG ((DEBUG_ERROR, "%a: MCTP protocol over %s.\n", __func__, mTranspor= tName)); =20 // // Setup hardware information according to the transport interface. @@ -156,9 +156,9 @@ DxeMctpEntry ( ); if (EFI_ERROR (Status)) { if (Status =3D=3D EFI_UNSUPPORTED) { - DEBUG ((DEBUG_ERROR, "%a: No hardware information of %s transport in= terface.\n", __FUNCTION__, mTransportName)); + DEBUG ((DEBUG_ERROR, "%a: No hardware information of %s transport in= terface.\n", __func__, mTransportName)); } else { - DEBUG ((DEBUG_ERROR, "%a: Failed to setup hardware information of %s= transport interface.\n", __FUNCTION__, mTransportName)); + DEBUG ((DEBUG_ERROR, "%a: Failed to setup hardware information of %s= transport interface.\n", __func__, mTransportName)); } =20 return Status; @@ -184,7 +184,7 @@ DxeMctpEntry ( (VOID **)&mMctpProtocol ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Failed to install EDKII MCTP protocol - %r\n= ", __FUNCTION__, Status)); + DEBUG ((DEBUG_ERROR, "%a: Failed to install EDKII MCTP protocol - %r\n= ", __func__, Status)); } =20 return Status; diff --git a/Features/ManageabilityPkg/Universal/PldmProtocol/Common/PldmPr= otocolCommon.c b/Features/ManageabilityPkg/Universal/PldmProtocol/Common/Pl= dmProtocolCommon.c index bb4d3f61a0..ce1e2cba95 100644 --- a/Features/ManageabilityPkg/Universal/PldmProtocol/Common/PldmProtocolC= ommon.c +++ b/Features/ManageabilityPkg/Universal/PldmProtocol/Common/PldmProtocolC= ommon.c @@ -104,17 +104,17 @@ SetupPldmRequestTransportPacket ( (PacketTrailer =3D=3D NULL) || (PacketTrailerSize =3D=3D NULL) ) { - DEBUG ((DEBUG_ERROR, "%a: One or more than one of the required paramet= ers is NULL.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: One or more than one of the required paramet= ers is NULL.\n", __func__)); return EFI_INVALID_PARAMETER; } =20 if (CompareGuid (&gManageabilityTransportMctpGuid, TransportToken->Trans= port->ManageabilityTransportSpecification)) { - DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Setup transport header for PLDM= over MCTP.\n", __FUNCTION__)); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Setup transport header for PLDM= over MCTP.\n", __func__)); =20 // This is MCTP transport interface. MctpHeader =3D AllocateZeroPool (sizeof (MANAGEABILITY_MCTP_TRANSPORT_= HEADER)); if (MctpHeader =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: Not enough memory for MANAGEABILITY_MCTP_T= RANSPORT_HEADER.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Not enough memory for MANAGEABILITY_MCTP_T= RANSPORT_HEADER.\n", __func__)); return EFI_OUT_OF_RESOURCES; } =20 @@ -127,7 +127,7 @@ SetupPldmRequestTransportPacket ( *PacketTrailer =3D NULL; *PacketTrailerSize =3D 0; } else { - DEBUG ((DEBUG_ERROR, "%a: No implementation of building up packet.\n",= __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: No implementation of building up packet.\n",= __func__)); ASSERT (FALSE); } =20 @@ -136,7 +136,7 @@ SetupPldmRequestTransportPacket ( // PldmRequestHeader =3D (PLDM_REQUEST_HEADER *)AllocateZeroPool (sizeof (P= LDM_REQUEST_HEADER) + *PacketBodySize); if (PldmRequestHeader =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: Not enough memory for final PLDM request mes= sage.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Not enough memory for final PLDM request mes= sage.\n", __func__)); return EFI_OUT_OF_RESOURCES; } =20 @@ -202,7 +202,7 @@ CommonPldmSubmitCommand ( UINT16 TrailerSize; =20 if (TransportToken =3D=3D NULL) { - DEBUG ((DEBUG_ERROR, "%a: No transport token for PLDM\n", __FUNCTION__= )); + DEBUG ((DEBUG_ERROR, "%a: No transport token for PLDM\n", __func__)); return EFI_UNSUPPORTED; } =20 @@ -211,7 +211,7 @@ CommonPldmSubmitCommand ( &TransportAdd= itionalStatus ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Transport %s for PLDM has problem - (%r)\n",= __FUNCTION__, mTransportName, Status)); + DEBUG ((DEBUG_ERROR, "%a: Transport %s for PLDM has problem - (%r)\n",= __func__, mTransportName, Status)); return Status; } =20 @@ -233,7 +233,7 @@ CommonPldmSubmitCommand ( &TrailerSize ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Fail to build packets - (%r)\n", __FUNCTION_= _, Status)); + DEBUG ((DEBUG_ERROR, "%a: Fail to build packets - (%r)\n", __func__, S= tatus)); return Status; } =20 @@ -274,7 +274,7 @@ CommonPldmSubmitCommand ( DEBUG (( DEBUG_MANAGEABILITY_INFO, "%a: Send PLDM type: 0x%x, Command: 0x%x: Request size: 0x%x, Response= size: 0x%x\n", - __FUNCTION__, + __func__, PldmType, PldmCommand, TransferToken.TransmitPackage.TransmitSizeInByte, @@ -409,7 +409,7 @@ CommonPldmSubmitCommand ( ErrorExit: Status =3D TransferToken.TransferStatus; if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Failed to send PLDM command over %s\n", __FU= NCTION__, mTransportName)); + DEBUG ((DEBUG_ERROR, "%a: Failed to send PLDM command over %s\n", __fu= nc__, mTransportName)); } =20 ErrorExit2: diff --git a/Features/ManageabilityPkg/Universal/PldmProtocol/Dxe/PldmProto= col.c b/Features/ManageabilityPkg/Universal/PldmProtocol/Dxe/PldmProtocol.c index bb34fec16e..b2ca69b05f 100644 --- a/Features/ManageabilityPkg/Universal/PldmProtocol/Dxe/PldmProtocol.c +++ b/Features/ManageabilityPkg/Universal/PldmProtocol/Dxe/PldmProtocol.c @@ -57,7 +57,7 @@ PldmSubmitCommand ( EFI_STATUS Status; =20 if ((RequestData =3D=3D NULL) && (ResponseData =3D=3D NULL)) { - DEBUG ((DEBUG_ERROR, "%a: Both RequestData and ResponseData are NULL\n= ", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Both RequestData and ResponseData are NULL\n= ", __func__)); return EFI_INVALID_PARAMETER; } =20 @@ -106,26 +106,26 @@ DxePldmProtocolEntry ( &mTransportToken ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Failed to acquire transport interface for PL= DM protocol - %r\n", __FUNCTION__, Status)); + DEBUG ((DEBUG_ERROR, "%a: Failed to acquire transport interface for PL= DM protocol - %r\n", __func__, Status)); return Status; } =20 Status =3D GetTransportCapability (mTransportToken, &TransportCapability= ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Failed to GetTransportCapability().\n", __FU= NCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Failed to GetTransportCapability().\n", __fu= nc__)); return Status; } =20 TransportMaximumPayload =3D MANAGEABILITY_TRANSPORT_PAYLOAD_SIZE_FROM_CA= PABILITY (TransportCapability); if (TransportMaximumPayload =3D=3D (1 << MANAGEABILITY_TRANSPORT_CAPABIL= ITY_MAXIMUM_PAYLOAD_NOT_AVAILABLE)) { - DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface maximum pay= load is undefined.\n", __FUNCTION__)); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface maximum pay= load is undefined.\n", __func__)); } else { TransportMaximumPayload -=3D 1; - DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface for PLDM pr= otocol has maximum payload 0x%x.\n", __FUNCTION__, TransportMaximumPayload)= ); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: Transport interface for PLDM pr= otocol has maximum payload 0x%x.\n", __func__, TransportMaximumPayload)); } =20 mTransportName =3D HelperManageabilitySpecName (mTransportToken->Transpo= rt->ManageabilityTransportSpecification); - DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: PLDM protocol over %s.\n", __FUNC= TION__, mTransportName)); + DEBUG ((DEBUG_MANAGEABILITY_INFO, "%a: PLDM protocol over %s.\n", __func= __, mTransportName)); =20 // Initial transport interface with the hardware information assigned. HardwareInfo.Pointer =3D NULL; @@ -149,7 +149,7 @@ DxePldmProtocolEntry ( (VOID **)&mPldmProtocol ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Failed to install EDKII PLDM protocol - %r\n= ", __FUNCTION__, Status)); + DEBUG ((DEBUG_ERROR, "%a: Failed to install EDKII PLDM protocol - %r\n= ", __func__, Status)); } =20 return Status; diff --git a/Features/ManageabilityPkg/Universal/PldmSmbiosTransferDxe/Pldm= SmbiosTransferDxe.c b/Features/ManageabilityPkg/Universal/PldmSmbiosTransfe= rDxe/PldmSmbiosTransferDxe.c index 88b190757d..f897162bb5 100644 --- a/Features/ManageabilityPkg/Universal/PldmSmbiosTransferDxe/PldmSmbiosT= ransferDxe.c +++ b/Features/ManageabilityPkg/Universal/PldmSmbiosTransferDxe/PldmSmbiosT= ransferDxe.c @@ -147,7 +147,7 @@ GetSmbiosStructureTableMetaData ( EFI_STATUS Status; UINT32 ResponseSize; =20 - DEBUG ((DEBUG_INFO, "%a: Set SMBIOS structure table metafile.\n", __FUNC= TION__)); + DEBUG ((DEBUG_INFO, "%a: Set SMBIOS structure table metafile.\n", __func= __)); =20 ResponseSize =3D sizeof (PLDM_SMBIOS_STRUCTURE_TABLE_METADATA); Status =3D PldmSubmitCommand ( @@ -159,7 +159,7 @@ GetSmbiosStructureTableMetaData ( &ResponseSize ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Fails to get SMBIOS structure table metafile= .\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Fails to get SMBIOS structure table metafile= .\n", __func__)); } =20 if (ResponseSize !=3D 0) { @@ -194,7 +194,7 @@ SetSmbiosStructureTableMetaData ( UINT32 ResponseSize; UINT32 RequestSize; =20 - DEBUG ((DEBUG_INFO, "%a: Get SMBIOS structure table metafile.\n", __FUNC= TION__)); + DEBUG ((DEBUG_INFO, "%a: Get SMBIOS structure table metafile.\n", __func= __)); =20 RequestSize =3D sizeof (PLDM_SMBIOS_STRUCTURE_TABLE_METADATA); ResponseSize =3D 0; @@ -208,7 +208,7 @@ SetSmbiosStructureTableMetaData ( &ResponseSize ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Fails to set SMBIOS structure table metafile= .\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Fails to set SMBIOS structure table metafile= .\n", __func__)); } =20 return Status; @@ -236,7 +236,7 @@ GetSmbiosStructureTable ( OUT UINT32 *BufferSize ) { - DEBUG ((DEBUG_INFO, "%a: Unsupported.\n", __FUNCTION__)); + DEBUG ((DEBUG_INFO, "%a: Unsupported.\n", __func__)); // Only support PLDM SMBIOS Transfer push mode. return EFI_UNSUPPORTED; } @@ -272,7 +272,7 @@ SetSmbiosStructureTable ( EFI_SMBIOS_TABLE_HEADER *Record; PLDM_SET_SMBIOS_STRUCTURE_TABLE_REQUEST *PldmSetSmbiosStructureTable; =20 - DEBUG ((DEBUG_INFO, "%a: Set SMBIOS structure table.\n", __FUNCTION__)); + DEBUG ((DEBUG_INFO, "%a: Set SMBIOS structure table.\n", __func__)); =20 Status =3D gBS->LocateProtocol ( &gEfiSmbiosProtocolGuid, @@ -382,7 +382,7 @@ SetSmbiosStructureTable ( } =20 if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Set SMBIOS structure table.\n", __FUNCTION__= )); + DEBUG ((DEBUG_ERROR, "%a: Set SMBIOS structure table.\n", __func__)); } =20 if ((ResponseSize !=3D 0) && (ResponseSize <=3D sizeof (SetSmbiosStructu= reTableHandle))) { @@ -422,7 +422,7 @@ GetSmbiosStructureByType ( OUT UINT32 *BufferSize ) { - DEBUG ((DEBUG_INFO, "%a: Unsupported.\n", __FUNCTION__)); + DEBUG ((DEBUG_INFO, "%a: Unsupported.\n", __func__)); // Only support PLDM SMBIOS Transfer push mode. return EFI_UNSUPPORTED; } @@ -451,7 +451,7 @@ GetSmbiosStructureByHandle ( OUT UINT32 *BufferSize ) { - DEBUG ((DEBUG_INFO, "%a: Unsupported.\n", __FUNCTION__)); + DEBUG ((DEBUG_INFO, "%a: Unsupported.\n", __func__)); // Only support PLDM SMBIOS Transfer push mode. return EFI_UNSUPPORTED; } @@ -486,7 +486,7 @@ DxePldmSmbiosTransferEntry ( EFI_HANDLE Handle; EFI_STATUS Status; =20 - DEBUG ((DEBUG_INFO, "%a: Entry.\n", __FUNCTION__)); + DEBUG ((DEBUG_INFO, "%a: Entry.\n", __func__)); =20 SetSmbiosStructureTableHandle =3D 0; =20 @@ -500,7 +500,7 @@ DxePldmSmbiosTransferEntry ( (VOID **)&mPld= mSmbiosTransferProtocol ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_ERROR, "%a: Fail to install EDKII_PLDM_SMBIOS_TRANSFER_P= ROTOCOL.\n", __FUNCTION__)); + DEBUG ((DEBUG_ERROR, "%a: Fail to install EDKII_PLDM_SMBIOS_TRANSFER_P= ROTOCOL.\n", __func__)); } =20 return Status; --=20 2.37.1.windows.1