From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by spool.mail.gandi.net (Postfix) with ESMTPS id CE83E7803D0 for ; Wed, 1 Nov 2023 13:42:34 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=vV4wCAtHfkJgHwJeYiVYvv60k9h6R9sUeGPjdqvvtIQ=; c=relaxed/simple; d=groups.io; h=ARC-Seal:ARC-Message-Signature:ARC-Authentication-Results:From:To:CC:Subject:Thread-Topic:Thread-Index:Date:Message-ID:References:In-Reply-To:Accept-Language:msip_labels:MIME-Version:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Language:Content-Type:Content-Transfer-Encoding; s=20140610; t=1698846153; v=1; b=p14VR7pPKplxtyIWLGJZL3CmgSmk1fme4lTuNhaOtczZVSiUUrm19wk2utOK5HPnqg8eRNm+ 2cZr3yH+VQleOuUmHlanslEeCPnrKtgZJXUCTAdxUZewgq5drQT0qfx1mliuqjdD0ydJFPW4I/O IeqtxExIlHdfJv4qTF5qYOxo= X-Received: by 127.0.0.2 with SMTP id 9TCxYY7687511xJ1FrlBGEp1; Wed, 01 Nov 2023 06:42:33 -0700 X-Received: from NAM11-BN8-obe.outbound.protection.outlook.com (NAM11-BN8-obe.outbound.protection.outlook.com [40.107.236.41]) by mx.groups.io with SMTP id smtpd.web10.7597.1698846152552373600 for ; Wed, 01 Nov 2023 06:42:32 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=lR7w+Nu+gRZHme6LoxjCsYbz5v6DlT0P8K1ZcRhCgMQ2aLjd7TAUvLe7MLqeSKE8RvtQtt57Retna0DiN9zm0T2X+FXGMy0kP31ASncn1C03W+nWBBsfzwDn5xQz3H/Zc7MpiVQrxsJSbtaMV51T4AV/T7kpjp6Yovcc7D08DRC3WEyv3rRspSlWfp71TfHU+kEMkUxtuF8vj81kYYd/mNeoErD+moahoL4hkq+m7P6Dur44WsUEHROhtQk8ME3rbtknM0EkFxHJHfu6h4mq3OlXxfcGKQChhAda6bm5k2YiW8GXvEqZl8UGtp7qVRixNcdBG6Y65dm0hAgKg8wC3g== 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=tWnL0WLOgKprPilAsGB897f36bFOpu4fRRT2hlpyk+k=; b=F4JWB9bNjAccdEbBgWUyonV0vrxaPMUqwA6g2EeeDRJc4iET5O+FThv1n6UIEZM1yaFsWEm3Sk1CBkNdIwMEktfFaUceFNWOIQgwxWEtB1B1YTeboNz0pvYrA7yq2a7tWip0oUoQsbJ0m8YQos7/GmWdbh34mBixHmafWDGOx0eXOxGXe4NnJYQ5Fb7F4TbnWNQa79qeo9yPZS8THcQN+Ttn3dV8F+n34LBhpOW3SDDJjZ47v6uubjgXI2tbKGphhhYDpsyXnkNWlZFIYMo4XCPUUiaoAUe+cNaT5QDXQhODCEttYh4W3UYmxLpAJZDiUcQhSr3xBTDM5OsH0L88BA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=amd.com; dmarc=pass action=none header.from=amd.com; dkim=pass header.d=amd.com; arc=none X-Received: from MN2PR12MB3966.namprd12.prod.outlook.com (2603:10b6:208:165::18) by DM4PR12MB6613.namprd12.prod.outlook.com (2603:10b6:8:b8::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6954.19; Wed, 1 Nov 2023 13:42:24 +0000 X-Received: from MN2PR12MB3966.namprd12.prod.outlook.com ([fe80::e670:e45f:fa32:2420]) by MN2PR12MB3966.namprd12.prod.outlook.com ([fe80::e670:e45f:fa32:2420%7]) with mapi id 15.20.6954.019; Wed, 1 Nov 2023 13:42:24 +0000 From: "Chang, Abner via groups.io" To: Nickle Wang , "devel@edk2.groups.io" CC: Igor Kulchytskyy , Nick Ramirez Subject: Re: [edk2-devel] [edk2-redfish-client][PATCH 3/3] RedfishClientPkg/RedfishFeatureUtilityLib: fix issues and enhancement Thread-Topic: [edk2-redfish-client][PATCH 3/3] RedfishClientPkg/RedfishFeatureUtilityLib: fix issues and enhancement Thread-Index: AQHaC84/2qu094CoXUiGIj3zZAvCgLBletFQ Date: Wed, 1 Nov 2023 13:42:23 +0000 Message-ID: References: <20231031074508.23718-1-nicklew@nvidia.com> In-Reply-To: <20231031074508.23718-1-nicklew@nvidia.com> Accept-Language: en-US, zh-CN X-MS-Has-Attach: X-MS-TNEF-Correlator: msip_labels: MSIP_Label_4342314e-0df4-4b58-84bf-38bed6170a0f_ActionId=a7eb2928-62e2-45f5-9764-cace61c9d988;MSIP_Label_4342314e-0df4-4b58-84bf-38bed6170a0f_ContentBits=0;MSIP_Label_4342314e-0df4-4b58-84bf-38bed6170a0f_Enabled=true;MSIP_Label_4342314e-0df4-4b58-84bf-38bed6170a0f_Method=Standard;MSIP_Label_4342314e-0df4-4b58-84bf-38bed6170a0f_Name=General;MSIP_Label_4342314e-0df4-4b58-84bf-38bed6170a0f_SetDate=2023-11-01T13:42:11Z;MSIP_Label_4342314e-0df4-4b58-84bf-38bed6170a0f_SiteId=3dd8961f-e488-4e60-8e11-a82d994e183d; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: MN2PR12MB3966:EE_|DM4PR12MB6613:EE_ x-ms-office365-filtering-correlation-id: ef0e2a82-56fd-4b03-93d6-08dbdae0611b x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam-message-info: zZ0wEFeWbyKq2InmVuQij9xMz9vw/eod8lmz97NOx6UMPCgBo5yDYKDuCgPHIGUA4F5HUZuQaup1AN3SuRFtXCOedD/bJhx6AvzY/IeDLXTox6Nyzqw1ruK9YO0RE5ayHEWonoNSAZ+d1zywKDq6D+b5eIa6mlqtPc6/6OhCcqZR5DUnIcmTIaydxauIqxqiIAAQtyEfWEjk7vWi/pO+cVb5/tn/C+XPOep9w5IEOpVFn9pWgifvWIykU5gpSttF2tXSPBSqtnjuGpXU2jaAiyqHlb/5+IZbBvSiv4dANeEFqtRQsqzIXtoEnkPCVZVNXX54e0GH3q5zDDzdZosgjMKScu5iiuPl7f/HooixhwkoGA3/wi8/E3NURLmj3c9/kYhmCy8jwU2Zr5hqiuQ2YFbS6JFxta8GsKPtugLvim0qvAWItnfarCPF7royH7b0hgk1ahJYK8SDBzdEj0fBBGxfKczziZ04OQNerkOZIxsqg9l/DTOqchTIhtMzOi5YTqkOJLXZt12J92F6C407yIEmXri485RC8OeJzEF1xX0GmFM31aCtBMKHdHmbW1xDDtJbmlrL1kYd7T71+07xy2EWKXkCQIF7K5AVBNRlc3jSOCC1X0YZj1JqvLWD8Jeju1AMs4ocFnaBs5WV7FUiOt1JfMTPcbIg78T2cORFSfA= x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?9RFHRzODmv4xU8bW9i21xZjcKCHEVcRq13RxhOW9OXF/PzABawLIqAajSL2J?= =?us-ascii?Q?G8LGa+r2SKnPIhCJMZeDLyDSBh/qA1IJkSNSMRgnINVY02CyPzAG9YnfZjOW?= =?us-ascii?Q?WTS6XJDTWhh2ngC2cFpuv0w6+nH6xOx065XH7un4EFiPfWoiOV4y2x90a8iO?= =?us-ascii?Q?pNNT/YPlggMtrEiPnKRNGZS56AqjP+F8/5/MYTx2krGRvSyNjK1LxNeTDD/u?= =?us-ascii?Q?z3LsudfHuC9yky95cbyubv5b3muVBdIq9N2WcruUlNUu3xjMoquXPzqGz0EN?= =?us-ascii?Q?WwFaEl1DzsyoHjUqLup7AuI0eloRY54m0j3x0NmpwZXEEyY/gsMO1E10RudM?= =?us-ascii?Q?3t4GJS9lX5TNNkf8q2wGpeGNipHNe7IXXrb71JI0m/PZIFRBKWDUVp7wbZcE?= =?us-ascii?Q?bx7yg8nP3GG2t6p3aAdBltjc9lamR2FcIAHcQIHs6rOhl09TZ5gVfO/gufAk?= =?us-ascii?Q?g8cpR0iwFb5+WFfIjTVKmhgw/EWmzkp4gVTrnKrzBbha7sSN6SgnXIHMBOpP?= =?us-ascii?Q?4Z1nhEd838MwJbOQ8wk9PRiZEx7+DBkXdKIMGWs0w+Ft4ogmSbnHA/Ca6wgN?= =?us-ascii?Q?Zpdnx4IMHopnGnU0H3rZsfx2nMc5VbsvqtGYrCFow3X+nM+yaRGOltbzmw3h?= =?us-ascii?Q?JeUzACmWfXQJB/t+2Y7v735dQ8uUi6mml+fW42QB/mLB9382QdtDkZDvvuHh?= =?us-ascii?Q?Mz/2FHZl+IX3qYkhJcBg0XdiVJ5eT/Ae98k92FMRZ3orxPib574TFV/69R1P?= =?us-ascii?Q?q51eTsGq0ISHwdqBMhrDqBPwJjbKzllW7InCJr7wAbc0CHTMHhM1vxBrQry5?= =?us-ascii?Q?ICA8KD2Vl8B2Y6aCeom/vQNuEztVzo6Dqn+GfcG1ygqBiokHmdXNAgCVG7XB?= =?us-ascii?Q?OKBcowX+ExXJcnJwt2+kb+vudK1FkE5UZJ0rHnz9JfgUA1WxZ6QYZ07Vc1BJ?= =?us-ascii?Q?QyIO1b/ofN8bxADjhOnXGziZJO7Xh8rtizsG04APx4gi4cRqtjXNMVxYpuTe?= =?us-ascii?Q?cNZ1UZMJjB+PrBYWswCYgZoGOTsnQ4EJMxloUrsK+B9fn3wZDAB2/zZ4SfWZ?= =?us-ascii?Q?9o182/ZiqxG5sXWPequb3ZsPAVZtZKh0CZYA4yIrHU55Wj3RXmz7NJi2NEeX?= =?us-ascii?Q?8mDoau8YX/SB22mE21SzNP6jjsQtwwfEAhiNJXP1ZxaF4MdRdGt1XQafqlD8?= =?us-ascii?Q?hYLUpRg5DV7hbdIuzZDCyoWNsH/H54hYb3VjWtbcn9DRutZwqPoF/TF+yXED?= =?us-ascii?Q?od7YNKT/uc/idvBonxCnJcxoozXqEMx5kmuk59mEDpJ0Tnj4YSTYjOIJZUNQ?= =?us-ascii?Q?gWzycYaighS4f96ViVRG9vqxtVckVMbkYXY1Vv0NcNSEl8zt5+Bl2c13risZ?= =?us-ascii?Q?0Ebi5pyrMYEZ1N++mdRkiV665OkvvjzuG6bVydPtINtfwoRW1i3w4bbcUiFe?= =?us-ascii?Q?I7dHT+x9Su6YxdlWLNQx/g8Plg+MwBtaABsEXWXnkZhZMlnDNwQa0WqYsGod?= =?us-ascii?Q?PjapO5tNF1XUUJcGGw0Wy0icn3gzAvCaI9RsxeOBUrWW/+N1AKVA+MfABBif?= =?us-ascii?Q?DBs5BEv2pLn0mholM+0=3D?= MIME-Version: 1.0 X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: MN2PR12MB3966.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: ef0e2a82-56fd-4b03-93d6-08dbdae0611b X-MS-Exchange-CrossTenant-originalarrivaltime: 01 Nov 2023 13:42:23.9764 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: e9xiAZIYXMi3WihtfipmxY+uGPnBAWxQFDlpFSnUBlu/lAZqjZ8deV1eE14LzbYrq11/CxU/Dx1BSfm7C58Uyw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR12MB6613 Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,abner.chang@amd.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: ndrK2dFFsK06y0jKd7W9mdqEx7686176AA= Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20140610 header.b=p14VR7pP; arc=reject ("signature check failed: fail, {[1] = sig:microsoft.com:reject}"); dmarc=none; spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce@groups.io [AMD Official Use Only - General] Reviewed-by: Abner Chang > -----Original Message----- > From: Nickle Wang > Sent: Tuesday, October 31, 2023 3:45 PM > To: devel@edk2.groups.io > Cc: Chang, Abner ; Igor Kulchytskyy > ; Nick Ramirez > Subject: [edk2-redfish-client][PATCH 3/3] > RedfishClientPkg/RedfishFeatureUtilityLib: fix issues and enhancement > > Caution: This message originated from an External Source. Use proper caut= ion > when opening attachments, clicking links, or responding. > > > -Add RedfishDebugLib to print Redfish request and response > details when PATCH and POST request return error. > -Use "%a:" in debug print to align with the style in EDK2. > -Enhance GetConfigureLang() to handle pending resource URI. > Pending resource share ths same schema as its active resource. > So we can simply remove "/Settings" from URI and search again. > -Enhancement to GetPropertyVagueValue(). This function stops > to handle remaining property while error happens. We like to > process as much as properties as we can and skip the problematic > property. > -Enhancement to MatchPropertyWithJsonContext(). For string type > of BIOS attribute, it is possible that the attribute value is > empty by default. This does not means that the resource is not > provisioned yet. > -Fix typo. > > Signed-off-by: Nickle Wang > Cc: Abner Chang > Cc: Igor Kulchytskyy > Cc: Nick Ramirez > --- > RedfishClientPkg/RedfishClientLibs.dsc.inc | 1 + > .../RedfishFeatureUtilityLib.inf | 1 + > .../RedfishFeatureUtilityInternal.h | 2 + > .../RedfishFeatureUtilityLib.c | 272 ++++++++++-------- > 4 files changed, 154 insertions(+), 122 deletions(-) > > diff --git a/RedfishClientPkg/RedfishClientLibs.dsc.inc > b/RedfishClientPkg/RedfishClientLibs.dsc.inc > index 5ea38015..5eae6711 100644 > --- a/RedfishClientPkg/RedfishClientLibs.dsc.inc > +++ b/RedfishClientPkg/RedfishClientLibs.dsc.inc > @@ -39,3 +39,4 @@ > > RedfishEventLib|RedfishClientPkg/Library/RedfishEventLib/RedfishEventLib.= i > nf > > RedfishVersionLib|RedfishClientPkg/Library/RedfishVersionLib/RedfishVersi= o > nLib.inf > > RedfishAddendumLib|RedfishClientPkg/Library/RedfishAddendumLib/Redfis > hAddendumLib.inf > + RedfishDebugLib|RedfishPkg/Library/RedfishDebugLib/RedfishDebugLib.inf > diff --git > a/RedfishClientPkg/Library/RedfishFeatureUtilityLib/RedfishFeatureUtility= Lib.i > nf > b/RedfishClientPkg/Library/RedfishFeatureUtilityLib/RedfishFeatureUtility= Lib.i > nf > index 66d5dce6..fd66b8ac 100644 > --- > a/RedfishClientPkg/Library/RedfishFeatureUtilityLib/RedfishFeatureUtility= Lib.i > nf > +++ > b/RedfishClientPkg/Library/RedfishFeatureUtilityLib/RedfishFeatureUtility= Lib.i > nf > @@ -45,6 +45,7 @@ > UefiRuntimeServicesTableLib > PrintLib > HttpLib > + RedfishDebugLib > > [Protocols] > gEdkIIRedfishETagProtocolGuid ## CONSUMED ## > diff --git > a/RedfishClientPkg/Library/RedfishFeatureUtilityLib/RedfishFeatureUtility= Inte > rnal.h > b/RedfishClientPkg/Library/RedfishFeatureUtilityLib/RedfishFeatureUtility= Inte > rnal.h > index d2bd6507..5d39984c 100644 > --- > a/RedfishClientPkg/Library/RedfishFeatureUtilityLib/RedfishFeatureUtility= Inte > rnal.h > +++ > b/RedfishClientPkg/Library/RedfishFeatureUtilityLib/RedfishFeatureUtility= Inte > rnal.h > @@ -2,6 +2,7 @@ > Common header file for RedfishFeatureUtilityLib driver. > > (C) Copyright 2020-2022 Hewlett Packard Enterprise Development LP
> + Copyright (c) 2023, NVIDIA CORPORATION & AFFILIATES. All rights > reserved. > > SPDX-License-Identifier: BSD-2-Clause-Patent > > @@ -28,6 +29,7 @@ > #include > #include > #include > +#include > > #include > > diff --git > a/RedfishClientPkg/Library/RedfishFeatureUtilityLib/RedfishFeatureUtility= Lib. > c > b/RedfishClientPkg/Library/RedfishFeatureUtilityLib/RedfishFeatureUtility= Lib. > c > index e1899878..13e29902 100644 > --- > a/RedfishClientPkg/Library/RedfishFeatureUtilityLib/RedfishFeatureUtility= Lib. > c > +++ > b/RedfishClientPkg/Library/RedfishFeatureUtilityLib/RedfishFeatureUtility= Lib. > c > @@ -133,7 +133,7 @@ SetEtagWithUri ( > > Status =3D RedfishLocateProtocol ((VOID **)&mEtagProtocol, > &gEdkIIRedfishETagProtocolGuid); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, fail to locate gEdkIIRedfishETagProtocolGu= id: > %r\n", __func__, Status)); > + DEBUG ((DEBUG_ERROR, "%a: fail to locate > gEdkIIRedfishETagProtocolGuid: %r\n", __func__, Status)); > return Status; > } > > @@ -175,7 +175,7 @@ GetEtagWithUri ( > > Status =3D RedfishLocateProtocol ((VOID **)&mEtagProtocol, > &gEdkIIRedfishETagProtocolGuid); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, fail to locate gEdkIIRedfishETagProtocolGu= id: > %r\n", __func__, Status)); > + DEBUG ((DEBUG_ERROR, "%a: fail to locate > gEdkIIRedfishETagProtocolGuid: %r\n", __func__, Status)); > return NULL; > } > > @@ -306,10 +306,10 @@ ApplyFeatureSettingsStringType ( > // > Status =3D RedfishPlatformConfigGetValue (Schema, Version, ConfigureLa= ng, > &RedfishValue); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s failed: %r\n", __func__, Schema, > Version, ConfigureLang, Status)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s failed: %r\n", __func__, Schema, > Version, ConfigureLang, Status)); > } else { > if (RedfishValue.Type !=3D RedfishValueTypeString) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s value is not string type\n", > __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s value is not string type\n", > __func__, Schema, Version, ConfigureLang)); > return EFI_DEVICE_ERROR; > } > > @@ -317,7 +317,7 @@ ApplyFeatureSettingsStringType ( > // > // Apply settings from redfish > // > - DEBUG ((DEBUG_MANAGEABILITY, "%a, %a.%a apply %s from %a to > %a\n", __func__, Schema, Version, ConfigureLang, RedfishValue.Value.Buffe= r, > FeatureValue)); > + DEBUG ((DEBUG_MANAGEABILITY, "%a: %a.%a apply %s from %a to > %a\n", __func__, Schema, Version, ConfigureLang, RedfishValue.Value.Buffe= r, > FeatureValue)); > > FreePool (RedfishValue.Value.Buffer); > RedfishValue.Value.Buffer =3D FeatureValue; > @@ -329,10 +329,10 @@ ApplyFeatureSettingsStringType ( > // > REDFISH_ENABLE_SYSTEM_REBOOT (); > } else { > - DEBUG ((DEBUG_ERROR, "%a, apply %s to %s failed: %r\n", __func__= , > ConfigureLang, FeatureValue, Status)); > + DEBUG ((DEBUG_ERROR, "%a: apply %s to %s failed: %r\n", __func__= , > ConfigureLang, FeatureValue, Status)); > } > } else { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s value is: %a\n", __func__, > Schema, Version, ConfigureLang, RedfishValue.Value.Buffer)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s value is: %a\n", __func__, > Schema, Version, ConfigureLang, RedfishValue.Value.Buffer)); > } > } > > @@ -341,7 +341,7 @@ ApplyFeatureSettingsStringType ( > > /** > > - Apply property value to UEFI HII database in numric type. > + Apply property value to UEFI HII database in numeric type. > > @param[in] Schema Property schema. > @param[in] Version Property schema version. > @@ -372,10 +372,10 @@ ApplyFeatureSettingsNumericType ( > // > Status =3D RedfishPlatformConfigGetValue (Schema, Version, ConfigureLa= ng, > &RedfishValue); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s failed: %r\n", __func__, Schema, > Version, ConfigureLang, Status)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s failed: %r\n", __func__, Schema, > Version, ConfigureLang, Status)); > } else { > if (RedfishValue.Type !=3D RedfishValueTypeInteger) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s value is not numeric type\n", > __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s value is not numeric type\n", > __func__, Schema, Version, ConfigureLang)); > return EFI_DEVICE_ERROR; > } > > @@ -383,7 +383,7 @@ ApplyFeatureSettingsNumericType ( > // > // Apply settings from redfish > // > - DEBUG ((DEBUG_MANAGEABILITY, "%a, %a.%a apply %s from 0x%x to > 0x%x\n", __func__, Schema, Version, ConfigureLang, > RedfishValue.Value.Integer, FeatureValue)); > + DEBUG ((DEBUG_MANAGEABILITY, "%a: %a.%a apply %s from 0x%x to > 0x%x\n", __func__, Schema, Version, ConfigureLang, > RedfishValue.Value.Integer, FeatureValue)); > > RedfishValue.Value.Integer =3D (INT64)FeatureValue; > > @@ -394,10 +394,10 @@ ApplyFeatureSettingsNumericType ( > // > REDFISH_ENABLE_SYSTEM_REBOOT (); > } else { > - DEBUG ((DEBUG_ERROR, "%a, apply %s to 0x%x failed: %r\n", __func= __, > ConfigureLang, FeatureValue, Status)); > + DEBUG ((DEBUG_ERROR, "%a: apply %s to 0x%x failed: %r\n", __func= __, > ConfigureLang, FeatureValue, Status)); > } > } else { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s value is: 0x%x\n", __func__, > Schema, Version, ConfigureLang, RedfishValue.Value.Integer, Status)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s value is: 0x%x\n", __func__, > Schema, Version, ConfigureLang, RedfishValue.Value.Integer)); > } > } > > @@ -437,10 +437,10 @@ ApplyFeatureSettingsBooleanType ( > // > Status =3D RedfishPlatformConfigGetValue (Schema, Version, ConfigureLa= ng, > &RedfishValue); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s failed: %r\n", __func__, Schema, > Version, ConfigureLang, Status)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s failed: %r\n", __func__, Schema, > Version, ConfigureLang, Status)); > } else { > if (RedfishValue.Type !=3D RedfishValueTypeBoolean) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s value is not boolean type\n", > __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s value is not boolean type\n", > __func__, Schema, Version, ConfigureLang)); > return EFI_DEVICE_ERROR; > } > > @@ -448,7 +448,7 @@ ApplyFeatureSettingsBooleanType ( > // > // Apply settings from redfish > // > - DEBUG ((DEBUG_MANAGEABILITY, "%a, %a.%a apply %s from %a to > %a\n", __func__, Schema, Version, ConfigureLang, > (RedfishValue.Value.Boolean ? "True" : "False"), (FeatureValue ? "True" : > "False"))); > + DEBUG ((DEBUG_MANAGEABILITY, "%a: %a.%a apply %s from %a to > %a\n", __func__, Schema, Version, ConfigureLang, > (RedfishValue.Value.Boolean ? "True" : "False"), (FeatureValue ? "True" : > "False"))); > > RedfishValue.Value.Boolean =3D FeatureValue; > > @@ -459,10 +459,10 @@ ApplyFeatureSettingsBooleanType ( > // > REDFISH_ENABLE_SYSTEM_REBOOT (); > } else { > - DEBUG ((DEBUG_ERROR, "%a, apply %s to %a failed: %r\n", __func__= , > ConfigureLang, (FeatureValue ? "True" : "False"), Status)); > + DEBUG ((DEBUG_ERROR, "%a: apply %s to %a failed: %r\n", __func__= , > ConfigureLang, (FeatureValue ? "True" : "False"), Status)); > } > } else { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s value is: %a\n", __func__, > Schema, Version, ConfigureLang, (RedfishValue.Value.Boolean ? "True" : > "False"))); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s value is: %a\n", __func__, > Schema, Version, ConfigureLang, (RedfishValue.Value.Boolean ? "True" : > "False"))); > } > } > > @@ -508,13 +508,13 @@ ApplyFeatureSettingsVagueType ( > ConfigureLangAscii =3D AllocatePool (StrLen (ConfigureLang) + 1); > if (ConfigureLangAscii =3D=3D NULL) { > Status =3D EFI_OUT_OF_RESOURCES; > - DEBUG ((DEBUG_ERROR, "%a, Allocate memory for generate ConfigureLang > of vague key of %a.%a %s failed: %r\n", __func__, Schema, Version, > ConfigureLang, Status)); > + DEBUG ((DEBUG_ERROR, "%a: Allocate memory for generate > ConfigureLang of vague key of %a.%a %s failed: %r\n", __func__, Schema, > Version, ConfigureLang, Status)); > return Status; > } > > Status =3D UnicodeStrToAsciiStrS (ConfigureLang, ConfigureLangAscii, S= trLen > (ConfigureLang) + 1); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, Convert the configureLang of vague key of > %a.%a %s failed: %r\n", __func__, Schema, Version, ConfigureLang, Status)= ); > + DEBUG ((DEBUG_ERROR, "%a: Convert the configureLang of vague key of > %a.%a %s failed: %r\n", __func__, Schema, Version, ConfigureLang, Status)= ); > FreePool (ConfigureLangAscii); > return Status; > } > @@ -529,7 +529,7 @@ ApplyFeatureSettingsVagueType ( > ConfigureLangKeyAscii =3D AllocateZeroPool (StrSize); > ConfigureKeyLang =3D AllocateZeroPool (StrSize * sizeof (CHAR16= )); > if ((ConfigureLangKeyAscii =3D=3D NULL) || (ConfigureKeyLang =3D=3D = NULL)) { > - DEBUG ((DEBUG_ERROR, "%a, Generate ConfigureLang of vague key of > %a.%a %s %a failed!\n", __func__, Schema, Version, ConfigureLang, > CurrentVagueValuePtr->KeyNamePtr)); > + DEBUG ((DEBUG_ERROR, "%a: Generate ConfigureLang of vague key of > %a.%a %s %a failed!\n", __func__, Schema, Version, ConfigureLang, > CurrentVagueValuePtr->KeyNamePtr)); > goto ErrorContinue; > } > > @@ -549,7 +549,7 @@ ApplyFeatureSettingsVagueType ( > } else if (CurrentVagueValuePtr->Value->DataType =3D=3D > RedfishCS_Vague_DataType_Int64) { > PropertyDatatype =3D RedfishValueTypeInteger; > } else { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s Unsupported Redfish property > data type\n", __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s Unsupported Redfish property > data type\n", __func__, Schema, Version, ConfigureLang)); > goto ErrorContinue; > } > > @@ -558,10 +558,10 @@ ApplyFeatureSettingsVagueType ( > // > Status =3D RedfishPlatformConfigGetValue (Schema, Version, > ConfigureKeyLang, &RedfishValue); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s failed: %r\n", __func__, Schema= , > Version, ConfigureKeyLang, Status)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s failed: %r\n", __func__, Schema= , > Version, ConfigureKeyLang, Status)); > } else { > if (RedfishValue.Type !=3D PropertyDatatype) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s mismatched data type\n", > __func__, Schema, Version, ConfigureKeyLang)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s mismatched data type\n", > __func__, Schema, Version, ConfigureKeyLang)); > goto ErrorContinue; > } > > @@ -573,7 +573,7 @@ ApplyFeatureSettingsVagueType ( > // > // Apply settings from redfish > // > - DEBUG ((DEBUG_MANAGEABILITY, "%a, %a.%a apply %s from %a to > %a\n", __func__, Schema, Version, ConfigureKeyLang, > RedfishValue.Value.Buffer, CurrentVagueValuePtr->Value- > >DataValue.CharPtr)); > + DEBUG ((DEBUG_MANAGEABILITY, "%a: %a.%a apply %s from %a to > %a\n", __func__, Schema, Version, ConfigureKeyLang, > RedfishValue.Value.Buffer, CurrentVagueValuePtr->Value- > >DataValue.CharPtr)); > FreePool (RedfishValue.Value.Buffer); > RedfishValue.Value.Buffer =3D CurrentVagueValuePtr->Value- > >DataValue.CharPtr; > Status =3D RedfishPlatformConfigSetValue (S= chema, Version, > ConfigureKeyLang, RedfishValue); > @@ -583,10 +583,10 @@ ApplyFeatureSettingsVagueType ( > // > REDFISH_ENABLE_SYSTEM_REBOOT (); > } else { > - DEBUG ((DEBUG_ERROR, "%a, apply %a to %a failed: %r\n", __fu= nc__, > ConfigureKeyLang, CurrentVagueValuePtr->Value->DataValue.CharPtr, > Status)); > + DEBUG ((DEBUG_ERROR, "%a: apply %a to %a failed: %r\n", __fu= nc__, > ConfigureKeyLang, CurrentVagueValuePtr->Value->DataValue.CharPtr, > Status)); > } > } else { > - DEBUG ((DEBUG_MANAGEABILITY, "%a, %a.%a %s value is: %a\n", > __func__, Schema, Version, ConfigureKeyLang, RedfishValue.Value.Buffer)); > + DEBUG ((DEBUG_MANAGEABILITY, "%a: %a.%a %s value is: %a\n", > __func__, Schema, Version, ConfigureKeyLang, RedfishValue.Value.Buffer)); > } > } else if (PropertyDatatype =3D=3D RedfishValueTypeBoolean) { > // > @@ -598,7 +598,7 @@ ApplyFeatureSettingsVagueType ( > // > DEBUG (( > DEBUG_MANAGEABILITY, > - "%a, %a.%a apply %s from %a to %a\n", > + "%a: %a.%a apply %s from %a to %a\n", > __func__, > Schema, > Version, > @@ -615,10 +615,10 @@ ApplyFeatureSettingsVagueType ( > // > REDFISH_ENABLE_SYSTEM_REBOOT (); > } else { > - DEBUG ((DEBUG_ERROR, "%a, apply %s to %a failed: %r\n", __fu= nc__, > ConfigureKeyLang, (*CurrentVagueValuePtr->Value->DataValue.BoolPtr ? > "True" : "False"), Status)); > + DEBUG ((DEBUG_ERROR, "%a: apply %s to %a failed: %r\n", __fu= nc__, > ConfigureKeyLang, (*CurrentVagueValuePtr->Value->DataValue.BoolPtr ? > "True" : "False"), Status)); > } > } else { > - DEBUG ((DEBUG_MANAGEABILITY, "%a, %a.%a %s value is: %a\n", > __func__, Schema, Version, ConfigureKeyLang, (RedfishValue.Value.Boolean = ? > "True" : "False"))); > + DEBUG ((DEBUG_MANAGEABILITY, "%a: %a.%a %s value is: %a\n", > __func__, Schema, Version, ConfigureKeyLang, (RedfishValue.Value.Boolean = ? > "True" : "False"))); > } > } else if (PropertyDatatype =3D=3D RedfishValueTypeInteger) { > // > @@ -628,7 +628,7 @@ ApplyFeatureSettingsVagueType ( > // > // Apply settings from redfish > // > - DEBUG ((DEBUG_MANAGEABILITY, "%a, %a.%a apply %s from 0x%x to > 0x%x\n", __func__, Schema, Version, ConfigureKeyLang, > RedfishValue.Value.Integer, *CurrentVagueValuePtr->Value- > >DataValue.Int64Ptr)); > + DEBUG ((DEBUG_MANAGEABILITY, "%a: %a.%a apply %s from 0x%x to > 0x%x\n", __func__, Schema, Version, ConfigureKeyLang, > RedfishValue.Value.Integer, *CurrentVagueValuePtr->Value- > >DataValue.Int64Ptr)); > > RedfishValue.Value.Integer =3D (INT64)*CurrentVagueValuePtr->V= alue- > >DataValue.Int64Ptr; > Status =3D RedfishPlatformConfigSetValue (= Schema, Version, > ConfigureKeyLang, RedfishValue); > @@ -638,13 +638,13 @@ ApplyFeatureSettingsVagueType ( > // > REDFISH_ENABLE_SYSTEM_REBOOT (); > } else { > - DEBUG ((DEBUG_ERROR, "%a, apply %s to 0x%x failed: %r\n", > __func__, ConfigureKeyLang, *CurrentVagueValuePtr->Value- > >DataValue.Int64Ptr, Status)); > + DEBUG ((DEBUG_ERROR, "%a: apply %s to 0x%x failed: %r\n", > __func__, ConfigureKeyLang, *CurrentVagueValuePtr->Value- > >DataValue.Int64Ptr, Status)); > } > } else { > - DEBUG ((DEBUG_MANAGEABILITY, "%a, %a.%a %s value is: 0x%x\n", > __func__, Schema, Version, ConfigureKeyLang, RedfishValue.Value.Integer))= ; > + DEBUG ((DEBUG_MANAGEABILITY, "%a: %a.%a %s value is: 0x%x\n", > __func__, Schema, Version, ConfigureKeyLang, RedfishValue.Value.Integer))= ; > } > } else { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s Unsupported Redfish property > data type\n", __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s Unsupported Redfish property > data type\n", __func__, Schema, Version, ConfigureLang)); > goto ErrorContinue; > } > } > @@ -762,12 +762,12 @@ ApplyFeatureSettingsStringArrayType ( > // > Status =3D RedfishPlatformConfigGetValue (Schema, Version, ConfigureLa= ng, > &RedfishValue); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s failed: %r\n", __func__, Schema, > Version, ConfigureLang, Status)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s failed: %r\n", __func__, Schema, > Version, ConfigureLang, Status)); > return Status; > } > > if (RedfishValue.Type !=3D RedfishValueTypeStringArray) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s value is not string array type\n"= , > __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s value is not string array type\n"= , > __func__, Schema, Version, ConfigureLang)); > return EFI_DEVICE_ERROR; > } > > @@ -778,7 +778,7 @@ ApplyFeatureSettingsStringArrayType ( > // > // Apply settings from redfish > // > - DEBUG ((DEBUG_MANAGEABILITY, "%a, %a.%a apply %s for array\n", > __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_MANAGEABILITY, "%a: %a.%a apply %s for array\n", > __func__, Schema, Version, ConfigureLang)); > FreeArrayTypeRedfishValue (&RedfishValue); > > // > @@ -823,10 +823,10 @@ ApplyFeatureSettingsStringArrayType ( > // > REDFISH_ENABLE_SYSTEM_REBOOT (); > } else { > - DEBUG ((DEBUG_ERROR, "%a, apply %s array failed: %r\n", __func__, > ConfigureLang, Status)); > + DEBUG ((DEBUG_ERROR, "%a: apply %s array failed: %r\n", __func__, > ConfigureLang, Status)); > } > } else { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s array value has no change\n", > __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s array value has no change\n", > __func__, Schema, Version, ConfigureLang)); > } > > for (Index =3D 0; Index < RedfishValue.ArrayCount; Index++) { > @@ -873,12 +873,12 @@ ApplyFeatureSettingsNumericArrayType ( > // > Status =3D RedfishPlatformConfigGetValue (Schema, Version, ConfigureLa= ng, > &RedfishValue); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s failed: %r\n", __func__, Schema, > Version, ConfigureLang, Status)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s failed: %r\n", __func__, Schema, > Version, ConfigureLang, Status)); > return Status; > } > > if (RedfishValue.Type !=3D RedfishValueTypeIntegerArray) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s value is not string array type\n"= , > __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s value is not string array type\n"= , > __func__, Schema, Version, ConfigureLang)); > return EFI_DEVICE_ERROR; > } > > @@ -889,7 +889,7 @@ ApplyFeatureSettingsNumericArrayType ( > // > // Apply settings from redfish > // > - DEBUG ((DEBUG_MANAGEABILITY, "%a, %a.%a apply %s for array\n", > __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_MANAGEABILITY, "%a: %a.%a apply %s for array\n", > __func__, Schema, Version, ConfigureLang)); > FreeArrayTypeRedfishValue (&RedfishValue); > > // > @@ -928,10 +928,10 @@ ApplyFeatureSettingsNumericArrayType ( > // > REDFISH_ENABLE_SYSTEM_REBOOT (); > } else { > - DEBUG ((DEBUG_ERROR, "%a, apply %s array failed: %r\n", __func__, > ConfigureLang, Status)); > + DEBUG ((DEBUG_ERROR, "%a: apply %s array failed: %r\n", __func__, > ConfigureLang, Status)); > } > } else { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s array value has no change\n", > __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s array value has no change\n", > __func__, Schema, Version, ConfigureLang)); > } > > FreePool (RedfishValue.Value.IntegerArray); > @@ -946,7 +946,7 @@ ApplyFeatureSettingsNumericArrayType ( > @param[in] Schema Property schema. > @param[in] Version Property schema version. > @param[in] ConfigureLang Configure language refers to this property. > - @param[in] ArrayHead Head of Redfich CS boolean array value. > + @param[in] ArrayHead Head of Redfish CS boolean array value. > > @retval EFI_SUCCESS New value is applied successfully. > @retval Others Errors occur. > @@ -974,12 +974,12 @@ ApplyFeatureSettingsBooleanArrayType ( > // > Status =3D RedfishPlatformConfigGetValue (Schema, Version, ConfigureLa= ng, > &RedfishValue); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s failed: %r\n", __func__, Schema, > Version, ConfigureLang, Status)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s failed: %r\n", __func__, Schema, > Version, ConfigureLang, Status)); > return Status; > } > > if (RedfishValue.Type !=3D RedfishValueTypeBooleanArray) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s value is not string array type\n"= , > __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s value is not string array type\n"= , > __func__, Schema, Version, ConfigureLang)); > return EFI_DEVICE_ERROR; > } > > @@ -990,7 +990,7 @@ ApplyFeatureSettingsBooleanArrayType ( > // > // Apply settings from redfish > // > - DEBUG ((DEBUG_MANAGEABILITY, "%a, %a.%a apply %s for array\n", > __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_MANAGEABILITY, "%a: %a.%a apply %s for array\n", > __func__, Schema, Version, ConfigureLang)); > FreeArrayTypeRedfishValue (&RedfishValue); > > // > @@ -1029,10 +1029,10 @@ ApplyFeatureSettingsBooleanArrayType ( > // > REDFISH_ENABLE_SYSTEM_REBOOT (); > } else { > - DEBUG ((DEBUG_ERROR, "%a, apply %s array failed: %r\n", __func__, > ConfigureLang, Status)); > + DEBUG ((DEBUG_ERROR, "%a: apply %s array failed: %r\n", __func__, > ConfigureLang, Status)); > } > } else { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s array value has no change\n", > __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s array value has no change\n", > __func__, Schema, Version, ConfigureLang)); > } > > FreePool (RedfishValue.Value.BooleanArray); > @@ -1044,7 +1044,7 @@ ApplyFeatureSettingsBooleanArrayType ( > > Read redfish resource by given resource URI. > > - @param[in] Service Redfish srvice instacne to make query. > + @param[in] Service Redfish service instance to make query. > @param[in] ResourceUri Target resource URI. > @param[out] Response HTTP response from redfish service. > > @@ -1080,7 +1080,7 @@ GetResourceByUri ( > Response > ); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, RedfishGetByUri to %a failed: %r\n", > __func__, AsciiResourceUri, Status)); > + DEBUG ((DEBUG_ERROR, "%a: RedfishGetByUri to %a failed: %r\n", > __func__, AsciiResourceUri, Status)); > if (Response->Payload !=3D NULL) { > RedfishDumpPayload (Response->Payload); > RedfishFreeResponse ( > @@ -1110,7 +1110,7 @@ GetResourceByUri ( > @param[out] ArraySignatureClose String to the close of array sig= nature. > > @retval EFI_SUCCESS Index is found. > - @retval EFI_NOT_FOUND The non-array configure language st= ring is > retured. > + @retval EFI_NOT_FOUND The non-array configure language st= ring is > returned. > @retval EFI_INVALID_PARAMETER The format of input ConfigureLang i= s > wrong. > @retval Others Errors occur. > > @@ -1169,7 +1169,7 @@ IsRedpathArray ( > > /** > > - Get number of node from the string. Node is seperated by '/'. > + Get number of node from the string. Node is separated by '/'. > > @param[in] NodeString The node string to parse. > > @@ -1207,10 +1207,10 @@ GetNumberOfRedpathNodes ( > > @param[in] NodeString The node string to parse. > @param[in] Index Zero-based index of the node. > - @param[out] EndOfNodePtr Pointer to receive the poitner to > + @param[out] EndOfNodePtr Pointer to receive the pointer to > the last character of node string. > > - @retval EFI_STRING the begining of the node string. > + @retval EFI_STRING the beginning of the node string. > > **/ > EFI_STRING > @@ -1263,7 +1263,7 @@ GetRedpathNodeByIndex ( > @param[out] Index The array index number. > > @retval EFI_SUCCESS Index is found. > - @retval EFI_NOT_FOUND The non-array configure language st= ring is > retured. > + @retval EFI_NOT_FOUND The non-array configure language st= ring is > returned. > @retval EFI_INVALID_PARAMETER The format of input ConfigureLang i= s > wrong. > @retval Others Errors occur. > > @@ -1303,7 +1303,7 @@ GetArrayIndexFromArrayTypeConfigureLang ( > *Index =3D StrDecimalToUintn (ArrayOpenStr + StrLen > (BIOS_CONFIG_TO_REDFISH_REDPATH_ARRAY_START_SIGNATURE)); > > // > - // Resotre the '}' character and remove rest of string. > + // Restore the '}' character and remove rest of string. > // > ArrayCloseStr[0] =3D L'}'; > ArrayCloseStr[1] =3D '\0'; > @@ -1359,7 +1359,7 @@ CopyConfiglanguageList ( > DestConfigureLangList->List =3D > (REDFISH_FEATURE_ARRAY_TYPE_CONFIG_LANG *)AllocateZeroPool (sizeof > (REDFISH_FEATURE_ARRAY_TYPE_CONFIG_LANG) * DestConfigureLangList- > >Count); > if (DestConfigureLangList->List =3D=3D NULL) { > - DEBUG ((DEBUG_ERROR, "%a, Fail to allocate memory for > REDFISH_FEATURE_ARRAY_TYPE_CONFIG_LANG.\n", __func__)); > + DEBUG ((DEBUG_ERROR, "%a: Fail to allocate memory for > REDFISH_FEATURE_ARRAY_TYPE_CONFIG_LANG.\n", __func__)); > return EFI_OUT_OF_RESOURCES; > } > > @@ -1471,7 +1471,7 @@ DestroyConfiglanguageList ( > > @param[in] DestConfigLang Pointer to the node's configure lang= uage > string. > The memory pointed by ConfigLang mus= t be allocated > - through memory allocation interface.= Becasue we will > replace > + through memory allocation interface.= Because we will > replace > the pointer in this function. > @param[in] MaxtLengthConfigLang The maximum length of ConfigLang. > @param[in] ConfigLangInstance Pointer to Collection member instanc= e. > @@ -1518,7 +1518,7 @@ SetResourceConfigLangMemberInstance ( > if (MaxtLengthConfigLang < MaxStrLength) { > NewConfigLang =3D (EFI_STRING)AllocateZeroPool (MaxStrLength); > if (NewConfigLang =3D=3D NULL) { > - DEBUG ((DEBUG_ERROR, "%a, Fail to allocate memory for > NewConfigLang.\n", __func__)); > + DEBUG ((DEBUG_ERROR, "%a: Fail to allocate memory for > NewConfigLang.\n", __func__)); > return EFI_OUT_OF_RESOURCES; > } > } > @@ -1601,7 +1601,7 @@ RedfishFeatureGetUnifiedArrayTypeConfigureLang > ( > > Status =3D RedfishPlatformConfigGetConfigureLang (Schema, Version, Pat= tern, > &ConfigureLangList, &Count); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, RedfishFeatureGetConfigureLangRegex > failed: %r\n", __func__, Status)); > + DEBUG ((DEBUG_ERROR, "%a: RedfishFeatureGetConfigureLangRegex > failed: %r\n", __func__, Status)); > return Status; > } > > @@ -1815,6 +1815,13 @@ CreatePayloadToPatchResource ( > Status =3D RedfishPatchToPayload (TargetPayload, Payload, &PostRespons= e); > if (EFI_ERROR (Status)) { > DEBUG ((DEBUG_ERROR, "%a:%d Failed to PATCH payload to Redfish > service.\n", __func__, __LINE__)); > + > + DEBUG_CODE_BEGIN (); > + DEBUG ((DEBUG_ERROR, "%a: Request:\n", __func__)); > + DumpRedfishPayload (DEBUG_ERROR, Payload); > + DEBUG ((DEBUG_ERROR, "%a: Response:\n", __func__)); > + DumpRedfishResponse (__func__, DEBUG_ERROR, &PostResponse); > + DEBUG_CODE_END (); > goto EXIT_FREE_JSON_VALUE; > } > > @@ -1886,12 +1893,20 @@ CreatePayloadToPostResource ( > ZeroMem (&PostResponse, sizeof (REDFISH_RESPONSE)); > Status =3D RedfishPostToPayload (TargetPayload, Payload, &PostResponse= ); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a:%d Failed to POST Attribute Registry to > Redfish service.\n", __func__, __LINE__)); > + DEBUG ((DEBUG_ERROR, "%a:%d Failed to POST payload to Redfish > service.\n", __func__, __LINE__)); > + > + DEBUG_CODE_BEGIN (); > + DEBUG ((DEBUG_ERROR, "%a: Request:\n", __func__)); > + DumpRedfishPayload (DEBUG_ERROR, Payload); > + DEBUG ((DEBUG_ERROR, "%a: Response:\n", __func__)); > + DumpRedfishResponse (__func__, DEBUG_ERROR, &PostResponse); > + DEBUG_CODE_END (); > + > goto EXIT_FREE_JSON_VALUE; > } > > // > - // per Redfish spec. the URL of new eresource will be returned in "Loc= ation" > header. > + // per Redfish spec. the URL of new resource will be returned in "Loca= tion" > header. > // > Status =3D GetEtagAndLocation (&PostResponse, Etag, Location); > if (EFI_ERROR (Status)) { > @@ -1941,11 +1956,11 @@ RedfishGetUri ( > > Status =3D RedfishLocateProtocol ((VOID **)&mConfigLangMapProtocol, > &gEdkIIRedfishConfigLangMapProtocolGuid); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, fail to locate > gEdkIIRedfishConfigLangMapProtocolGuid: %r\n", __func__, Status)); > + DEBUG ((DEBUG_ERROR, "%a: fail to locate > gEdkIIRedfishConfigLangMapProtocolGuid: %r\n", __func__, Status)); > return NULL; > } > > - DEBUG ((REDFISH_DEBUG_TRACE, "%a, Get: %s\n", __func__, ConfigLang)); > + DEBUG ((REDFISH_DEBUG_TRACE, "%a: Get: %s\n", __func__, ConfigLang)); > > CloseBracket =3D StrStr (ConfigLang, L"{"); > if (CloseBracket =3D=3D NULL) { > @@ -1986,7 +2001,7 @@ RedfishGetUri ( > // Invalid format. No '}' found > // > if (*Target =3D=3D '\0') { > - DEBUG ((DEBUG_ERROR, "%a, invalid format: %s\n", __func__, > ConfigLang)); > + DEBUG ((DEBUG_ERROR, "%a: invalid format: %s\n", __func__, > ConfigLang)); > FreePool (ResultStr); > return NULL; > } > @@ -2012,13 +2027,13 @@ RedfishGetUri ( > &Found > ); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, Can not find: %s\n", __func__, TempStr))= ; > + DEBUG ((DEBUG_ERROR, "%a: Can not find: %s\n", __func__, TempStr))= ; > FreePool (ResultStr); > FreePool (TempStr); > return NULL; > } > > - DEBUG ((REDFISH_DEBUG_TRACE, "%a, Found: %s\n", __func__, Found)); > + DEBUG ((REDFISH_DEBUG_TRACE, "%a: Found: %s\n", __func__, Found)); > > // > // Keep result in final string pool > @@ -2040,7 +2055,7 @@ RedfishGetUri ( > StrCatS (ResultStr, MAX_REDFISH_URL_LEN, Head); > } > > - DEBUG ((REDFISH_DEBUG_TRACE, "%a, return: %s\n", __func__, > ResultStr)); > + DEBUG ((REDFISH_DEBUG_TRACE, "%a: return: %s\n", __func__, > ResultStr)); > > return ResultStr; > } > @@ -2067,11 +2082,11 @@ RedfishGetConfigLanguage ( > return NULL; > } > > - DEBUG ((REDFISH_DEBUG_TRACE, "%a, search config lang for URI: %s\n", > __func__, Uri)); > + DEBUG ((REDFISH_DEBUG_TRACE, "%a: search config lang for URI: %s\n", > __func__, Uri)); > > Status =3D RedfishLocateProtocol ((VOID **)&mConfigLangMapProtocol, > &gEdkIIRedfishConfigLangMapProtocolGuid); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, fail to locate > gEdkIIRedfishConfigLangMapProtocolGuid: %r\n", __func__, Status)); > + DEBUG ((DEBUG_ERROR, "%a: fail to locate > gEdkIIRedfishConfigLangMapProtocolGuid: %r\n", __func__, Status)); > return NULL; > } > > @@ -2088,7 +2103,7 @@ RedfishGetConfigLanguage ( > > /** > > - Return config language from given URI and prperty name. It's call > responsibility to release returned buffer. > + Return config language from given URI and property name. It's call > responsibility to release returned buffer. > > @param[in] Uri The URI to match > @param[in] PropertyName The property name of resource. This is optio= nal. > @@ -2108,6 +2123,7 @@ GetConfigureLang ( > EFI_STRING ResultStr; > EFI_STRING UnicodeUri; > EFI_STATUS Status; > + EFI_STRING StrFound; > > if (IS_EMPTY_STRING (Uri)) { > return NULL; > @@ -2126,6 +2142,18 @@ GetConfigureLang ( > } > > ConfigLang =3D RedfishGetConfigLanguage (UnicodeUri); > + if (ConfigLang =3D=3D NULL) { > + // > + // @Redfish.Settings share the same schema as its parent. > + // Remove "Settings" and try again. > + // > + StrFound =3D StrStr (UnicodeUri, L"/Settings"); > + if (StrFound !=3D NULL) { > + StrFound[0] =3D L'\0'; > + DEBUG ((REDFISH_DEBUG_TRACE, "%a: \"Settings\" found in URI, try: > %s\n", __func__, UnicodeUri)); > + ConfigLang =3D RedfishGetConfigLanguage (UnicodeUri); > + } > + } > > FreePool (UnicodeUri); > > @@ -2158,7 +2186,7 @@ GetConfigureLang ( > @param[in] ConfigLang ConfigLang to save > @param[in] Uri Redfish Uri to save > > - @retval EFI_INVALID_PARAMETR SystemId is NULL or EMPTY > + @retval EFI_INVALID_PARAMETER SystemId is NULL or EMPTY > @retval EFI_SUCCESS Redfish uri is saved > > **/ > @@ -2176,11 +2204,11 @@ RedfisSetRedfishUri ( > > Status =3D RedfishLocateProtocol ((VOID **)&mConfigLangMapProtocol, > &gEdkIIRedfishConfigLangMapProtocolGuid); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, fail to locate > gEdkIIRedfishConfigLangMapProtocolGuid: %r\n", __func__, Status)); > + DEBUG ((DEBUG_ERROR, "%a: fail to locate > gEdkIIRedfishConfigLangMapProtocolGuid: %r\n", __func__, Status)); > return Status; > } > > - DEBUG ((REDFISH_DEBUG_TRACE, "%a, Saved: %s -> %s\n", __func__, > ConfigLang, Uri)); > + DEBUG ((REDFISH_DEBUG_TRACE, "%a: Saved: %s -> %s\n", __func__, > ConfigLang, Uri)); > > return mConfigLangMapProtocol->Set (mConfigLangMapProtocol, > ConfigLang, Uri); > } > @@ -2228,7 +2256,7 @@ GetOdataId ( > > /** > > - Get the property name by given Configure Langauge. > + Get the property name by given Configure Language. > > @param[in] ResourceUri URI of root of resource. > @param[in] ConfigureLang Configure Language string. > @@ -2261,7 +2289,7 @@ GetPropertyFromConfigureLang ( > } > > // > - // The ConigLang has no '{}' > + // The ConfigLang has no '{}' > // > if (GetNumberOfRedpathNodes (ConfigureLang) =3D=3D 1) { > return NULL; > @@ -2321,7 +2349,7 @@ GetPropertyStringValue ( > UnicodeSPrint (ConfigureLangBuffer, BufferSize, L"%s/%s", ConfigureLan= g, > PropertyName); > Status =3D RedfishPlatformConfigGetValue (Schema, Version, > ConfigureLangBuffer, &RedfishValue); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a query current setting for %s failed: > %r\n", __func__, Schema, Version, ConfigureLangBuffer, Status)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a query current setting for %s failed: > %r\n", __func__, Schema, Version, ConfigureLangBuffer, Status)); > FreePool (ConfigureLangBuffer); > return NULL; > } > @@ -2329,7 +2357,7 @@ GetPropertyStringValue ( > FreePool (ConfigureLangBuffer); > > if (RedfishValue.Type !=3D RedfishValueTypeString) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s value is not string type\n", > __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s value is not string type\n", > __func__, Schema, Version, ConfigureLang)); > return NULL; > } > > @@ -2382,7 +2410,7 @@ GetPropertyNumericValue ( > UnicodeSPrint (ConfigureLangBuffer, BufferSize, L"%s/%s", ConfigureLan= g, > PropertyName); > Status =3D RedfishPlatformConfigGetValue (Schema, Version, > ConfigureLangBuffer, &RedfishValue); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a query current setting for %s failed: > %r\n", __func__, Schema, Version, ConfigureLangBuffer, Status)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a query current setting for %s failed: > %r\n", __func__, Schema, Version, ConfigureLangBuffer, Status)); > FreePool (ConfigureLangBuffer); > return NULL; > } > @@ -2390,7 +2418,7 @@ GetPropertyNumericValue ( > FreePool (ConfigureLangBuffer); > > if (RedfishValue.Type !=3D RedfishValueTypeInteger) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s value is not numeric type\n", > __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s value is not numeric type\n", > __func__, Schema, Version, ConfigureLang)); > return NULL; > } > > @@ -2447,7 +2475,7 @@ GetPropertyBooleanValue ( > UnicodeSPrint (ConfigureLangBuffer, BufferSize, L"%s/%s", ConfigureLan= g, > PropertyName); > Status =3D RedfishPlatformConfigGetValue (Schema, Version, > ConfigureLangBuffer, &RedfishValue); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a query current setting for %s failed: > %r\n", __func__, Schema, Version, ConfigureLangBuffer, Status)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a query current setting for %s failed: > %r\n", __func__, Schema, Version, ConfigureLangBuffer, Status)); > FreePool (ConfigureLangBuffer); > return NULL; > } > @@ -2455,7 +2483,7 @@ GetPropertyBooleanValue ( > FreePool (ConfigureLangBuffer); > > if (RedfishValue.Type !=3D RedfishValueTypeBoolean) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s value is not boolean type\n", > __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s value is not boolean type\n", > __func__, Schema, Version, ConfigureLang)); > return NULL; > } > > @@ -2544,14 +2572,14 @@ GetPropertyStringArrayValue ( > BufferSize =3D sizeof (CHAR16) * MAX_CONF_LANG_LEN; > ConfigureLangBuffer =3D AllocatePool (BufferSize); > if (ConfigureLangBuffer =3D=3D NULL) { > - DEBUG ((DEBUG_ERROR, "%a, out of resource\n", __func__)); > + DEBUG ((DEBUG_ERROR, "%a: out of resource\n", __func__)); > return NULL; > } > > UnicodeSPrint (ConfigureLangBuffer, BufferSize, L"%s/%s", ConfigureLan= g, > PropertyName); > Status =3D RedfishPlatformConfigGetValue (Schema, Version, > ConfigureLangBuffer, &RedfishValue); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a query current setting for %s failed: > %r\n", __func__, Schema, Version, ConfigureLangBuffer, Status)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a query current setting for %s failed: > %r\n", __func__, Schema, Version, ConfigureLangBuffer, Status)); > FreePool (ConfigureLangBuffer); > return NULL; > } > @@ -2559,13 +2587,13 @@ GetPropertyStringArrayValue ( > FreePool (ConfigureLangBuffer); > > if (RedfishValue.Type !=3D RedfishValueTypeStringArray) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s value is not string array type\n"= , > __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s value is not string array type\n"= , > __func__, Schema, Version, ConfigureLang)); > return NULL; > } > > StringArray =3D AllocatePool (sizeof (CHAR8 *) * RedfishValue.ArrayCou= nt); > if (StringArray =3D=3D NULL) { > - DEBUG ((DEBUG_ERROR, "%a, out of resource\n", __func__)); > + DEBUG ((DEBUG_ERROR, "%a: out of resource\n", __func__)); > return NULL; > } > > @@ -2618,14 +2646,14 @@ GetPropertyNumericArrayValue ( > BufferSize =3D sizeof (CHAR16) * MAX_CONF_LANG_LEN; > ConfigureLangBuffer =3D AllocatePool (BufferSize); > if (ConfigureLangBuffer =3D=3D NULL) { > - DEBUG ((DEBUG_ERROR, "%a, out of resource\n", __func__)); > + DEBUG ((DEBUG_ERROR, "%a: out of resource\n", __func__)); > return NULL; > } > > UnicodeSPrint (ConfigureLangBuffer, BufferSize, L"%s/%s", ConfigureLan= g, > PropertyName); > Status =3D RedfishPlatformConfigGetValue (Schema, Version, > ConfigureLangBuffer, &RedfishValue); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a query current setting for %s failed: > %r\n", __func__, Schema, Version, ConfigureLangBuffer, Status)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a query current setting for %s failed: > %r\n", __func__, Schema, Version, ConfigureLangBuffer, Status)); > FreePool (ConfigureLangBuffer); > return NULL; > } > @@ -2633,13 +2661,13 @@ GetPropertyNumericArrayValue ( > FreePool (ConfigureLangBuffer); > > if (RedfishValue.Type !=3D RedfishValueTypeIntegerArray) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s value is not string array type\n"= , > __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s value is not string array type\n"= , > __func__, Schema, Version, ConfigureLang)); > return NULL; > } > > IntegerArray =3D AllocatePool (sizeof (INT64) * RedfishValue.ArrayCoun= t); > if (IntegerArray =3D=3D NULL) { > - DEBUG ((DEBUG_ERROR, "%a, out of resource\n", __func__)); > + DEBUG ((DEBUG_ERROR, "%a: out of resource\n", __func__)); > return NULL; > } > > @@ -2692,14 +2720,14 @@ GetPropertyBooleanArrayValue ( > BufferSize =3D sizeof (CHAR16) * MAX_CONF_LANG_LEN; > ConfigureLangBuffer =3D AllocatePool (BufferSize); > if (ConfigureLangBuffer =3D=3D NULL) { > - DEBUG ((DEBUG_ERROR, "%a, out of resource\n", __func__)); > + DEBUG ((DEBUG_ERROR, "%a: out of resource\n", __func__)); > return NULL; > } > > UnicodeSPrint (ConfigureLangBuffer, BufferSize, L"%s/%s", ConfigureLan= g, > PropertyName); > Status =3D RedfishPlatformConfigGetValue (Schema, Version, > ConfigureLangBuffer, &RedfishValue); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a query current setting for %s failed: > %r\n", __func__, Schema, Version, ConfigureLangBuffer, Status)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a query current setting for %s failed: > %r\n", __func__, Schema, Version, ConfigureLangBuffer, Status)); > FreePool (ConfigureLangBuffer); > return NULL; > } > @@ -2707,13 +2735,13 @@ GetPropertyBooleanArrayValue ( > FreePool (ConfigureLangBuffer); > > if (RedfishValue.Type !=3D RedfishValueTypeBooleanArray) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a %s value is not string array type\n"= , > __func__, Schema, Version, ConfigureLang)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a %s value is not string array type\n"= , > __func__, Schema, Version, ConfigureLang)); > return NULL; > } > > BooleanArray =3D AllocatePool (sizeof (INT64) * RedfishValue.ArrayCoun= t); > if (BooleanArray =3D=3D NULL) { > - DEBUG ((DEBUG_ERROR, "%a, out of resource\n", __func__)); > + DEBUG ((DEBUG_ERROR, "%a: out of resource\n", __func__)); > return NULL; > } > > @@ -2784,19 +2812,19 @@ NewEmptyPropKeyValueFromRedfishValue ( > > KeyNameChar =3D StrUnicodeToAscii (KeyName); > if (KeyNameChar =3D=3D NULL) { > - DEBUG ((DEBUG_ERROR, "%a, Failed to convert unicode to ASCII.\n", > __func__)); > + DEBUG ((DEBUG_ERROR, "%a: Failed to convert unicode to ASCII.\n", > __func__)); > return NULL; > } > > EmptyPropKeyValue =3D (RedfishCS_EmptyProp_KeyValue *)AllocateZeroPool > (sizeof (RedfishCS_EmptyProp_KeyValue)); > if (EmptyPropKeyValue =3D=3D NULL) { > - DEBUG ((DEBUG_ERROR, "%a, Failed to allocate memory for > EmptyPropKeyValue\n", __func__)); > + DEBUG ((DEBUG_ERROR, "%a: Failed to allocate memory for > EmptyPropKeyValue\n", __func__)); > return NULL; > } > > VagueValue =3D (RedfishCS_Vague *)AllocateZeroPool (sizeof > (RedfishCS_Vague)); > if (VagueValue =3D=3D NULL) { > - DEBUG ((DEBUG_ERROR, "%a, Failed to allocate memory for > VagueValue\n", __func__)); > + DEBUG ((DEBUG_ERROR, "%a: Failed to allocate memory for > VagueValue\n", __func__)); > FreePool (EmptyPropKeyValue); > return NULL; > } > @@ -2805,7 +2833,7 @@ NewEmptyPropKeyValueFromRedfishValue ( > VagueValue->DataType =3D RedfishCS_Vague_DataType_Bool; > DataSize =3D sizeof (BOOLEAN); > // > - // Redfish JSON to C strcuture converter uses > + // Redfish JSON to C structure converter uses > // "int" for the BOOLEAN. > // > Bool32 =3D (INT32)RedfishValue->Value.Boolean; > @@ -2819,7 +2847,7 @@ NewEmptyPropKeyValueFromRedfishValue ( > DataSize =3D AsciiStrSize (RedfishValue->Value.Buffer); > Data =3D (VOID *)RedfishValue->Value.Buffer; > } else { > - DEBUG ((DEBUG_ERROR, "%a, wrong type of RedfishValue: %x\n", > __func__, RedfishValue->Type)); > + DEBUG ((DEBUG_ERROR, "%a: wrong type of RedfishValue: %x\n", > __func__, RedfishValue->Type)); > FreePool (VagueValue); > FreePool (EmptyPropKeyValue); > return NULL; > @@ -2866,18 +2894,21 @@ GetPropertyVagueValue ( > UINTN BufferSize; > UINTN ConfigListCount; > UINTN ConfigListCountIndex; > + UINTN ListCount; > > if (IS_EMPTY_STRING (Schema) || IS_EMPTY_STRING (Version) || > IS_EMPTY_STRING (ConfigureLang) || IS_EMPTY_STRING (PropertyName)) { > return NULL; > } > > + ListCount =3D 0; > + FirstEmptyPropKeyValueList =3D 0; > // > // Configure Language buffer. > // > BufferSize =3D sizeof (CHAR16) * MAX_CONF_LANG_LEN; > ConfigureLangBuffer =3D AllocatePool (BufferSize); > if (ConfigureLangBuffer =3D=3D NULL) { > - DEBUG ((DEBUG_ERROR, "%a, Failed to allocate memory for > ConfigureLangBuffer\n", __func__)); > + DEBUG ((DEBUG_ERROR, "%a: Failed to allocate memory for > ConfigureLangBuffer\n", __func__)); > return NULL; > } > > @@ -2889,7 +2920,7 @@ GetPropertyVagueValue ( > BufferSize =3D (StrLen (ConfigureLangBuffer) + StrLen (L"/.*") + 1)= * sizeof > (CHAR16); // Increase one for the NULL terminator. > SearchPattern =3D AllocatePool (BufferSize); > if (SearchPattern =3D=3D NULL) { > - DEBUG ((DEBUG_ERROR, "%a, Failed to allocate memory for > SearchPattern\n", __func__)); > + DEBUG ((DEBUG_ERROR, "%a: Failed to allocate memory for > SearchPattern\n", __func__)); > FreePool (ConfigureLangBuffer); > return NULL; > } > @@ -2899,21 +2930,23 @@ GetPropertyVagueValue ( > StrCatS (SearchPattern, BufferSize, L"/.*"); > Status =3D RedfishPlatformConfigGetConfigureLang (Schema, Version, > SearchPattern, &ConfigureLangList, &ConfigListCount); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a Get configure language of vague type > values of %s failed: %r\n", __func__, Schema, Version, ConfigureLangBuffe= r, > Status)); > + DEBUG ((DEBUG_ERROR, "%a: %a.%a Get configure language of vague type > values of %s failed: %r\n", __func__, Schema, Version, ConfigureLangBuffe= r, > Status)); > goto ErrorLeave; > } > > // > // Build up the list of RedfishCS_EmptyProp_KeyValue. > // > + ListCount =3D 0; > ConfigListCountIndex =3D 0; > PreEmptyPropKeyValueList =3D NULL; > FirstEmptyPropKeyValueList =3D NULL; > while (ConfigListCountIndex < ConfigListCount) { > Status =3D RedfishPlatformConfigGetValue (Schema, Version, > ConfigureLangList[ConfigListCountIndex], &RedfishValue); > if (EFI_ERROR (Status)) { > - DEBUG ((DEBUG_ERROR, "%a, %a.%a query current setting for %s faile= d: > %r\n", __func__, Schema, Version, ConfigureLangList[ConfigListCountIndex]= , > Status)); > - goto ErrorLeave; > + DEBUG ((DEBUG_ERROR, "%a: %a.%a query current setting for %s faile= d: > %r\n", __func__, Schema, Version, ConfigureLangList[ConfigListCountIndex]= , > Status)); > + ConfigListCountIndex++; > + continue; > } > > // > @@ -2925,7 +2958,7 @@ GetPropertyVagueValue ( > // > EmptyPropKeyValueList =3D NewEmptyPropKeyValueFromRedfishValue > (KeyName, &RedfishValue); > if (EmptyPropKeyValueList =3D=3D NULL) { > - DEBUG ((DEBUG_ERROR, "%a, Failed to create an entry of > EmptyPropKeyValueList\n", __func__)); > + DEBUG ((DEBUG_ERROR, "%a: Failed to create an entry of > EmptyPropKeyValueList\n", __func__)); > ConfigListCountIndex++; > continue; > } > @@ -2941,6 +2974,7 @@ GetPropertyVagueValue ( > > PreEmptyPropKeyValueList =3D EmptyPropKeyValueList; > ConfigListCountIndex++; > + ListCount++; > } > > goto LeaveFunction; > @@ -2963,7 +2997,7 @@ LeaveFunction: > > FreePool (ConfigureLangList); > > - *NumberOfValues =3D (UINT32)ConfigListCount; > + *NumberOfValues =3D (UINT32)ListCount; > return FirstEmptyPropKeyValueList; > } > > @@ -3030,7 +3064,7 @@ CheckEtag ( > EtagInDb =3D NULL; > EtagInDb =3D GetEtagWithUri (Uri); > if (EtagInDb =3D=3D NULL) { > - DEBUG ((REDFISH_DEBUG_TRACE, "%a, no ETAG record cound be found > for: %s\n", __func__, Uri)); > + DEBUG ((REDFISH_DEBUG_TRACE, "%a: no ETAG record could be found for: > %s\n", __func__, Uri)); > return FALSE; > } > > @@ -3173,12 +3207,6 @@ MatchPropertyWithJsonContext ( > MatchObj =3D NULL; > } > > - break; > - case EdkiiJsonTypeString: > - if (IS_EMPTY_STRING (JsonValueGetString (MatchObj))) { > - MatchObj =3D NULL; > - } > - > break; > case EdkiiJsonTypeNull: > MatchObj =3D NULL; > @@ -3195,7 +3223,7 @@ MatchPropertyWithJsonContext ( > > /** > > - Create string array and append to arry node in Redfish JSON convert fo= rmat. > + Create string array and append to array node in Redfish JSON convert > format. > > @param[in,out] Head The head of string array. > @param[in] StringArray Input string array. > @@ -3246,7 +3274,7 @@ AddRedfishCharArray ( > > /** > > - Create numeric array and append to arry node in Redfish JSON convert > format. > + Create numeric array and append to array node in Redfish JSON convert > format. > > @param[in,out] Head The head of string array. > @param[in] NumericArray Input numeric array. > @@ -3303,7 +3331,7 @@ AddRedfishNumericArray ( > > /** > > - Create boolean array and append to arry node in Redfish JSON convert > format. > + Create boolean array and append to array node in Redfish JSON convert > format. > > @param[in,out] Head The head of string array. > @param[in] BooleanArray Input boolean array. > @@ -3409,7 +3437,7 @@ CompareRedfishStringArrayValues ( > Check and see if value in Redfish numeric array are all the same as th= e one > from HII configuration. > > - @param[in] Head The head of Redfish CS numeraic array. > + @param[in] Head The head of Redfish CS numeric array. > @param[in] NumericArray Input numeric array. > @param[in] ArraySize The size of NumericArray. > > @@ -3502,9 +3530,9 @@ CompareRedfishBooleanArrayValues ( > This is just a simple check. > > @param[in] RedfishVagueKeyValuePtr The vague key value sets on > Redfish service. > - @param[in] RedfishVagueKeyValueNumber The numebr of vague key value > sets > + @param[in] RedfishVagueKeyValueNumber The number of vague key > value sets > @param[in] ConfigVagueKeyValuePtr The vague configuration on > platform. > - @param[in] ConfigVagueKeyValueNumber The numebr of vague key value > sets > + @param[in] ConfigVagueKeyValueNumber The number of vague key value > sets > > @retval TRUE All values are the same. > FALSE There is some difference. > @@ -3599,7 +3627,7 @@ CompareRedfishPropertyVagueValues ( > @param[in] ImageHandle The image handle. > @param[in] SystemTable The system table. > > - @retval EFI_SUCEESS Install Boot manager menu success. > + @retval EFI_SUCCESS Install Boot manager menu success. > @retval Other Return error status. > > **/ > -- > 2.17.1 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#110483): https://edk2.groups.io/g/devel/message/110483 Mute This Topic: https://groups.io/mt/102293054/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-