public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [edk2-redfish-client][PATCH 2/5] RedfishClientPkg: Rename Memory feature driver
@ 2023-05-10 13:04 Nickle Wang
  2023-05-10 14:06 ` Chang, Abner
  0 siblings, 1 reply; 2+ messages in thread
From: Nickle Wang @ 2023-05-10 13:04 UTC (permalink / raw)
  To: devel; +Cc: Abner Chang, Igor Kulchytskyy

Rename Memory driver by removing Redfish prefix. Update memory feature
driver and support all properties under Memory schema. Also support
"Identify" action in this driver. Corresponding changes are made to
honor newly introduced library and protocol.

Signed-off-by: Nickle Wang <nicklew@nvidia.com>
Cc: Abner Chang <abner.chang@amd.com>
Cc: Igor Kulchytskyy <igork@ami.com>
---
 .../RedfishClientComponents.dsc.inc           |    2 +-
 .../{RedfishMemoryDxe.inf => MemoryDxe.inf}   |   21 +-
 .../{RedfishMemoryCommon.h => MemoryCommon.h} |    2 +-
 .../Include/RedfishResourceCommon.h           |   38 +-
 .../{RedfishMemoryCommon.c => MemoryCommon.c} | 1138 ++++++++++-------
 .../Dxe/{RedfishMemoryDxe.c => MemoryDxe.c}   |  136 +-
 RedfishClientPkg/RedfishClient.fdf.inc        |    2 +-
 7 files changed, 838 insertions(+), 501 deletions(-)
 rename RedfishClientPkg/Features/Memory/V1_7_1/Dxe/{RedfishMemoryDxe.inf => MemoryDxe.inf} (58%)
 rename RedfishClientPkg/Features/Memory/V1_7_1/Common/{RedfishMemoryCommon.h => MemoryCommon.h} (89%)
 rename RedfishClientPkg/Features/Memory/V1_7_1/Common/{RedfishMemoryCommon.c => MemoryCommon.c} (64%)
 rename RedfishClientPkg/Features/Memory/V1_7_1/Dxe/{RedfishMemoryDxe.c => MemoryDxe.c} (75%)

diff --git a/RedfishClientPkg/RedfishClientComponents.dsc.inc b/RedfishClientPkg/RedfishClientComponents.dsc.inc
index 5f1aff3c..031d8755 100644
--- a/RedfishClientPkg/RedfishClientComponents.dsc.inc
+++ b/RedfishClientPkg/RedfishClientComponents.dsc.inc
@@ -19,7 +19,7 @@
   #
   # Below two modules should be pulled in by build tool.
   #
-  RedfishClientPkg/Features/Memory/V1_7_1/Dxe/RedfishMemoryDxe.inf
+  RedfishClientPkg/Features/Memory/V1_7_1/Dxe/MemoryDxe.inf
   RedfishClientPkg/Features/MemoryCollectionDxe/MemoryCollectionDxe.inf
 
   !include RedfishClientPkg/RedfishJsonStructureDxe.dsc.inc
diff --git a/RedfishClientPkg/Features/Memory/V1_7_1/Dxe/RedfishMemoryDxe.inf b/RedfishClientPkg/Features/Memory/V1_7_1/Dxe/MemoryDxe.inf
similarity index 58%
rename from RedfishClientPkg/Features/Memory/V1_7_1/Dxe/RedfishMemoryDxe.inf
rename to RedfishClientPkg/Features/Memory/V1_7_1/Dxe/MemoryDxe.inf
index 3e2ad884..594a8749 100644
--- a/RedfishClientPkg/Features/Memory/V1_7_1/Dxe/RedfishMemoryDxe.inf
+++ b/RedfishClientPkg/Features/Memory/V1_7_1/Dxe/MemoryDxe.inf
@@ -1,6 +1,6 @@
 ## @file
 #
-#  (C) Copyright 2020-2021 Hewlett Packard Enterprise Development LP<BR>
+#  (C) Copyright 2020-2022 Hewlett Packard Enterprise Development LP<BR>
 #
 #  SPDX-License-Identifier: BSD-2-Clause-Patent
 #
@@ -9,8 +9,8 @@
 
 [Defines]
   INF_VERSION               = 0x00010005
-  BASE_NAME                 = RedfishMemoryDxe
-  FILE_GUID                 = f5594d13-ca13-485f-ac3a-80c5074dd8c0
+  BASE_NAME                 = MemoryDxe
+  FILE_GUID                 = 69a1a994-ec14-4dd0-ba0d-d1b51b7bab51
   MODULE_TYPE               = DXE_DRIVER
   VERSION_STRING            = 1.0
   ENTRY_POINT               = RedfishResourceEntryPoint
@@ -23,22 +23,25 @@
   RedfishClientPkg/RedfishClientPkg.dec
 
 [Sources]
-  ../Common/RedfishMemoryCommon.h
-  ../Common/RedfishMemoryCommon.c
-  RedfishMemoryDxe.c
+  ../Common/MemoryCommon.h
+  ../Common/MemoryCommon.c
+  MemoryDxe.c
 
 [LibraryClasses]
   BaseMemoryLib
   DebugLib
+  EdkIIRedfishResourceConfigLib
   RedfishLib
   RedfishFeatureUtilityLib
+  RedfishResourceIdentifyLib
   UefiLib
   UefiDriverEntryPoint
 
+
 [Protocols]
-  gEdkIIRedfishConfigHandlerProtocolGuid    ## PRODUCED
-  gEfiRestJsonStructureProtocolGuid         ## CONSUMED
-  gEdkIIRedfishResourceConfigProtocolGuid   ## PRODUCED
+  gEdkIIRedfishConfigHandlerProtocolGuid          ## PRODUCED
+  gEfiRestJsonStructureProtocolGuid               ## CONSUMED
+  gEdkIIRedfishResourceConfigProtocolGuid         ## PRODUCED
 
 [Pcd]
   gEfiRedfishClientPkgTokenSpaceGuid.PcdMaxRedfishSchemaStringSize
diff --git a/RedfishClientPkg/Features/Memory/V1_7_1/Common/RedfishMemoryCommon.h b/RedfishClientPkg/Features/Memory/V1_7_1/Common/MemoryCommon.h
similarity index 89%
rename from RedfishClientPkg/Features/Memory/V1_7_1/Common/RedfishMemoryCommon.h
rename to RedfishClientPkg/Features/Memory/V1_7_1/Common/MemoryCommon.h
index f6fe5111..c857868e 100644
--- a/RedfishClientPkg/Features/Memory/V1_7_1/Common/RedfishMemoryCommon.h
+++ b/RedfishClientPkg/Features/Memory/V1_7_1/Common/MemoryCommon.h
@@ -20,7 +20,7 @@
 #define RESOURCE_SCHEMA_MINOR    "7"
 #define RESOURCE_SCHEMA_ERRATA   "1"
 #define RESOURCE_SCHEMA_VERSION  "v1_7_1"
-#define REDPATH_ARRAY_PATTERN    L"/Memory/{.*}/"
+#define REDPATH_ARRAY_PATTERN    L"/Memory/\\{.*\\}/"
 #define REDPATH_ARRAY_PREFIX     L"/Memory/"
 #define RESOURCE_SCHEMA_FULL     "x-uefi-redfish-Memory.v1_7_1"
 
diff --git a/RedfishClientPkg/Include/RedfishResourceCommon.h b/RedfishClientPkg/Include/RedfishResourceCommon.h
index 95e8004d..c270f92b 100644
--- a/RedfishClientPkg/Include/RedfishResourceCommon.h
+++ b/RedfishClientPkg/Include/RedfishResourceCommon.h
@@ -1,7 +1,7 @@
 /** @file
   Redfish feature driver common header file.
 
-  (C) Copyright 2020-2021 Hewlett Packard Enterprise Development LP<BR>
+  (C) Copyright 2020-2022 Hewlett Packard Enterprise Development LP<BR>
 
   SPDX-License-Identifier: BSD-2-Clause-Patent
 
@@ -11,8 +11,8 @@
 #define EFI_REDFISH_RESOURCE_COMMON_H_
 
 #define MAX_RED_PATH_LEN  128
-#define IS_EMPTY_STRING(a)  ((a) == NULL || (a)[0] == '\0')
-#define REDFISH_DEBUG_TRACE  DEBUG_INFO
+
+#include <RedfishBase.h>
 
 //
 // Libraries
@@ -23,14 +23,18 @@
 #include <Library/PrintLib.h>
 #include <Library/PcdLib.h>
 #include <Library/RedfishLib.h>
+#include <Library/RedfishVersionLib.h>
 #include <Library/RedfishFeatureUtilityLib.h>
 #include <Library/RedfishPlatformConfigLib.h>
 #include <Library/UefiLib.h>
 #include <Library/UefiBootServicesTableLib.h>
+#include <Library/RedfishResourceIdentifyLib.h>
+#include <Library/EdkIIRedfishResourceConfigLib.h>
 
 //
 // Protocols
 //
+#include <Protocol/EdkIIRedfishFeature.h>
 #include <Protocol/EdkIIRedfishConfigHandler.h>
 #include <Protocol/EdkIIRedfishResourceConfigProtocol.h>
 #include <Protocol/RestJsonStructure.h>
@@ -42,9 +46,15 @@ typedef struct _REDFISH_RESOURCE_COMMON_PRIVATE {
   EDKII_REDFISH_RESOURCE_CONFIG_PROTOCOL    RedfishResourceConfig;
   EDKII_REDFISH_CONFIG_HANDLER_PROTOCOL     ConfigHandler;
   EFI_EVENT                                 Event;
-  CHAR8                                     *Uri;
+  EFI_STRING                                Uri;
   CHAR8                                     *Json;
   REDFISH_PAYLOAD                           Payload;
+  //
+  //  Below are used for the external resource.
+  //
+  EDKII_REDFISH_FEATURE_PROTOCOL            *FeatureProtocol;
+  RESOURCE_INFORMATION_EXCHANGE             *InformationExchange;
+  EFI_STRING                                RedfishVersion;
 } REDFISH_RESOURCE_COMMON_PRIVATE;
 
 #define REDFISH_RESOURCE_COMMON_PRIVATE_DATA_FROM_CONFIG_PROTOCOL(This) \
@@ -58,6 +68,7 @@ typedef struct _REDFISH_RESOURCE_COMMON_PRIVATE {
 
   @param[in]   This                Pointer to REDFISH_RESOURCE_COMMON_PRIVATE instance.
   @param[in]   Json                The JSON to consume.
+  @param[in]   HeaderEtag          The Etag string returned in HTTP header.
 
   @retval EFI_SUCCESS              Value is returned successfully.
   @retval Others                   Some error happened.
@@ -66,7 +77,8 @@ typedef struct _REDFISH_RESOURCE_COMMON_PRIVATE {
 EFI_STATUS
 RedfishConsumeResourceCommon (
   IN     REDFISH_RESOURCE_COMMON_PRIVATE  *Private,
-  IN     CHAR8                            *Json
+  IN     CHAR8                            *Json,
+  IN     CHAR8                            *HeaderEtag OPTIONAL
   );
 
 /**
@@ -118,4 +130,20 @@ RedfishUpdateResourceCommon (
   IN     CHAR8                            *Json
   );
 
+/**
+  Identify resource from given URI.
+
+  @param[in]   This                Pointer to REDFISH_RESOURCE_COMMON_PRIVATE instance.
+  @param[in]   Json                The JSON to consume.
+
+  @retval EFI_SUCCESS              Value is returned successfully.
+  @retval Others                   Some error happened.
+
+**/
+EFI_STATUS
+RedfishIdentifyResourceCommon (
+  IN     REDFISH_RESOURCE_COMMON_PRIVATE  *Private,
+  IN     CHAR8                            *Json
+  );
+
 #endif
diff --git a/RedfishClientPkg/Features/Memory/V1_7_1/Common/RedfishMemoryCommon.c b/RedfishClientPkg/Features/Memory/V1_7_1/Common/MemoryCommon.c
similarity index 64%
rename from RedfishClientPkg/Features/Memory/V1_7_1/Common/RedfishMemoryCommon.c
rename to RedfishClientPkg/Features/Memory/V1_7_1/Common/MemoryCommon.c
index 756c42ba..bf87f185 100644
--- a/RedfishClientPkg/Features/Memory/V1_7_1/Common/RedfishMemoryCommon.c
+++ b/RedfishClientPkg/Features/Memory/V1_7_1/Common/MemoryCommon.c
@@ -1,45 +1,53 @@
 /** @file
   Redfish feature driver implementation - common functions
 
-  (C) Copyright 2020-2021 Hewlett Packard Enterprise Development LP<BR>
+  (C) Copyright 2020-2022 Hewlett Packard Enterprise Development LP<BR>
 
   SPDX-License-Identifier: BSD-2-Clause-Patent
 
 **/
 
-#include "RedfishMemoryCommon.h"
+#include "MemoryCommon.h"
 
 CHAR8  MemoryEmptyJson[] = "{\"@odata.id\": \"\", \"@odata.type\": \"#Memory.v1_7_1.Memory\", \"Id\": \"\", \"Name\": \"\"}";
 
 REDFISH_RESOURCE_COMMON_PRIVATE  *mRedfishResourcePrivate = NULL;
 
+/**
+  Consume resource from given URI.
+
+  @param[in]   This                Pointer to REDFISH_RESOURCE_COMMON_PRIVATE instance.
+  @param[in]   Json                The JSON to consume.
+  @param[in]   HeaderEtag          The Etag string returned in HTTP header.
+
+  @retval EFI_SUCCESS              Value is returned successfully.
+  @retval Others                   Some error happened.
+
+**/
 EFI_STATUS
 RedfishConsumeResourceCommon (
   IN  REDFISH_RESOURCE_COMMON_PRIVATE  *Private,
-  IN  CHAR8                            *MemoryJson
+  IN  CHAR8                            *Json,
+  IN  CHAR8                            *HeaderEtag OPTIONAL
   )
 {
   EFI_STATUS                    Status;
   EFI_REDFISH_MEMORY_V1_7_1     *Memory;
   EFI_REDFISH_MEMORY_V1_7_1_CS  *MemoryCs;
   EFI_STRING                    ConfigureLang;
-  CHAR8                         *Arraykey;
-  CHAR8                         *EtagInDb;
 
-  if ((Private == NULL) || IS_EMPTY_STRING (MemoryJson)) {
+  if ((Private == NULL) || IS_EMPTY_STRING (Json)) {
     return EFI_INVALID_PARAMETER;
   }
 
   Memory        = NULL;
   MemoryCs      = NULL;
   ConfigureLang = NULL;
-  Arraykey      = NULL;
-  EtagInDb      = NULL;
 
   Status = Private->JsonStructProtocol->ToStructure (
                                           Private->JsonStructProtocol,
                                           NULL,
-                                          MemoryJson,
+                                          Json,
                                           (EFI_REST_JSON_STRUCTURE_HEADER **)&Memory
                                           );
   if (EFI_ERROR (Status)) {
@@ -52,24 +60,13 @@ RedfishConsumeResourceCommon (
   //
   // Check ETAG to see if we need to consume it
   //
-  EtagInDb = GetEtagWithUri (Private->Uri);
-  if ((EtagInDb != NULL) && (MemoryCs->odata_etag != NULL)) {
-    if (AsciiStrCmp (EtagInDb, MemoryCs->odata_etag) == 0) {
-      //
-      // No change
-      //
-      DEBUG ((DEBUG_INFO, "%a, ETAG: [%a] no change, ignore consume action\n", __FUNCTION__, EtagInDb));
-      goto ON_RELEASE;
-    }
-  }
-
-  //
-  // Find array key from URI
-  //
-  Status = GetArraykeyFromUri (Private->Uri, &Arraykey);
-  if (EFI_ERROR (Status)) {
-    ASSERT (FALSE);
-    return Status;
+  if (CheckEtag (Private->Uri, HeaderEtag, MemoryCs->odata_etag)) {
+    //
+    // No change
+    //
+    DEBUG ((DEBUG_INFO, "%a, ETAG: %s has no change, ignore consume action\n", __FUNCTION__, Private->Uri));
+    Status = EFI_ALREADY_STARTED;
+    goto ON_RELEASE;
   }
 
   //
@@ -77,9 +74,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->AllocationAlignmentMiB != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "AllocationAlignmentMiB", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "AllocationAlignmentMiB");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->AllocationAlignmentMiB);
       if (EFI_ERROR (Status)) {
@@ -88,7 +85,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -97,9 +94,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->AllocationIncrementMiB != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "AllocationIncrementMiB", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "AllocationIncrementMiB");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->AllocationIncrementMiB);
       if (EFI_ERROR (Status)) {
@@ -108,18 +105,42 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
+  //
+  // Handle ALLOWEDSPEEDSMHZ
+  //
+  if (MemoryCs->AllowedSpeedsMHz != NULL) {
+    //
+    // Find corresponding configure language for collection resource.
+    //
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "AllowedSpeedsMHz");
+    if (ConfigureLang != NULL) {
+      Status = ApplyFeatureSettingsNumericArrayType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->AllowedSpeedsMHz);
+      if (EFI_ERROR (Status)) {
+        DEBUG ((DEBUG_ERROR, "%a, apply setting for %s failed: %r\n", __FUNCTION__, ConfigureLang, Status));
+      }
+
+      FreePool (ConfigureLang);
+    } else {
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
+    }
+  }
+
+  //
+  // ASSEMBLY will be handled by feature driver.
+  //
+
   //
   // Handle BASEMODULETYPE
   //
   if (MemoryCs->BaseModuleType != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "BaseModuleType", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "BaseModuleType");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->BaseModuleType);
       if (EFI_ERROR (Status)) {
@@ -128,7 +149,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -137,9 +158,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->BusWidthBits != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "BusWidthBits", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "BusWidthBits");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->BusWidthBits);
       if (EFI_ERROR (Status)) {
@@ -148,7 +169,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -157,9 +178,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->CacheSizeMiB != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "CacheSizeMiB", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "CacheSizeMiB");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->CacheSizeMiB);
       if (EFI_ERROR (Status)) {
@@ -168,7 +189,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -177,9 +198,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->CapacityMiB != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "CapacityMiB", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "CapacityMiB");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->CapacityMiB);
       if (EFI_ERROR (Status)) {
@@ -188,7 +209,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -197,9 +218,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->ConfigurationLocked != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "ConfigurationLocked", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "ConfigurationLocked");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsBooleanType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (BOOLEAN)*MemoryCs->ConfigurationLocked);
       if (EFI_ERROR (Status)) {
@@ -208,7 +229,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -217,9 +238,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->DataWidthBits != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "DataWidthBits", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "DataWidthBits");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->DataWidthBits);
       if (EFI_ERROR (Status)) {
@@ -228,7 +249,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -237,9 +258,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->DeviceID != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "DeviceID", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "DeviceID");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->DeviceID);
       if (EFI_ERROR (Status)) {
@@ -248,7 +269,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -257,9 +278,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->DeviceLocator != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "DeviceLocator", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "DeviceLocator");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->DeviceLocator);
       if (EFI_ERROR (Status)) {
@@ -268,7 +289,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -277,9 +298,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->ErrorCorrection != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "ErrorCorrection", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "ErrorCorrection");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->ErrorCorrection);
       if (EFI_ERROR (Status)) {
@@ -288,7 +309,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -297,9 +318,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->FirmwareApiVersion != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "FirmwareApiVersion", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "FirmwareApiVersion");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->FirmwareApiVersion);
       if (EFI_ERROR (Status)) {
@@ -308,7 +329,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -317,9 +338,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->FirmwareRevision != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "FirmwareRevision", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "FirmwareRevision");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->FirmwareRevision);
       if (EFI_ERROR (Status)) {
@@ -328,7 +349,27 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
+    }
+  }
+
+  //
+  // Handle FUNCTIONCLASSES
+  //
+  if (MemoryCs->FunctionClasses != NULL) {
+    //
+    // Find corresponding configure language for collection resource.
+    //
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "FunctionClasses");
+    if (ConfigureLang != NULL) {
+      Status = ApplyFeatureSettingsStringArrayType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->FunctionClasses);
+      if (EFI_ERROR (Status)) {
+        DEBUG ((DEBUG_ERROR, "%a, apply setting for %s failed: %r\n", __FUNCTION__, ConfigureLang, Status));
+      }
+
+      FreePool (ConfigureLang);
+    } else {
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -337,9 +378,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->IsRankSpareEnabled != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "IsRankSpareEnabled", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "IsRankSpareEnabled");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsBooleanType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (BOOLEAN)*MemoryCs->IsRankSpareEnabled);
       if (EFI_ERROR (Status)) {
@@ -348,7 +389,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -357,9 +398,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->IsSpareDeviceEnabled != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "IsSpareDeviceEnabled", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "IsSpareDeviceEnabled");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsBooleanType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (BOOLEAN)*MemoryCs->IsSpareDeviceEnabled);
       if (EFI_ERROR (Status)) {
@@ -368,18 +409,22 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
+  //
+  // LOCATION is not handled. Defined in http://redfish.dmtf.org/schemas/v1/Resource.json
+  //
+
   //
   // Handle LOGICALSIZEMIB
   //
   if (MemoryCs->LogicalSizeMiB != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "LogicalSizeMiB", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "LogicalSizeMiB");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->LogicalSizeMiB);
       if (EFI_ERROR (Status)) {
@@ -388,7 +433,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -397,9 +442,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->Manufacturer != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "Manufacturer", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "Manufacturer");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->Manufacturer);
       if (EFI_ERROR (Status)) {
@@ -408,7 +453,27 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
+    }
+  }
+
+  //
+  // Handle MAXTDPMILLIWATTS
+  //
+  if (MemoryCs->MaxTDPMilliWatts != NULL) {
+    //
+    // Find corresponding configure language for collection resource.
+    //
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "MaxTDPMilliWatts");
+    if (ConfigureLang != NULL) {
+      Status = ApplyFeatureSettingsNumericArrayType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->MaxTDPMilliWatts);
+      if (EFI_ERROR (Status)) {
+        DEBUG ((DEBUG_ERROR, "%a, apply setting for %s failed: %r\n", __FUNCTION__, ConfigureLang, Status));
+      }
+
+      FreePool (ConfigureLang);
+    } else {
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -417,9 +482,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->MemoryDeviceType != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "MemoryDeviceType", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "MemoryDeviceType");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->MemoryDeviceType);
       if (EFI_ERROR (Status)) {
@@ -428,7 +493,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -445,9 +510,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->MemoryLocation->Channel != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "MemoryLocation/Channel", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "MemoryLocation/Channel");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->MemoryLocation->Channel);
       if (EFI_ERROR (Status)) {
@@ -456,7 +521,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -465,9 +530,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->MemoryLocation->MemoryController != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "MemoryLocation/MemoryController", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "MemoryLocation/MemoryController");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->MemoryLocation->MemoryController);
       if (EFI_ERROR (Status)) {
@@ -476,7 +541,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -485,9 +550,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->MemoryLocation->Slot != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "MemoryLocation/Slot", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "MemoryLocation/Slot");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->MemoryLocation->Slot);
       if (EFI_ERROR (Status)) {
@@ -496,7 +561,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -505,9 +570,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->MemoryLocation->Socket != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "MemoryLocation/Socket", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "MemoryLocation/Socket");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->MemoryLocation->Socket);
       if (EFI_ERROR (Status)) {
@@ -516,18 +581,26 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
+  //
+  // Handle MEMORYMEDIA
+  //
+  //
+  // ****** Warning ******
+  // Unsupported array type:
+  //
+
   //
   // Handle MEMORYSUBSYSTEMCONTROLLERMANUFACTURERID
   //
   if (MemoryCs->MemorySubsystemControllerManufacturerID != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "MemorySubsystemControllerManufacturerID", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "MemorySubsystemControllerManufacturerID");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->MemorySubsystemControllerManufacturerID);
       if (EFI_ERROR (Status)) {
@@ -536,7 +609,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -545,9 +618,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->MemorySubsystemControllerProductID != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "MemorySubsystemControllerProductID", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "MemorySubsystemControllerProductID");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->MemorySubsystemControllerProductID);
       if (EFI_ERROR (Status)) {
@@ -556,7 +629,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -565,9 +638,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->MemoryType != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "MemoryType", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "MemoryType");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->MemoryType);
       if (EFI_ERROR (Status)) {
@@ -576,18 +649,22 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
+  //
+  // METRICS will be handled by feature driver.
+  //
+
   //
   // Handle MODULEMANUFACTURERID
   //
   if (MemoryCs->ModuleManufacturerID != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "ModuleManufacturerID", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "ModuleManufacturerID");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->ModuleManufacturerID);
       if (EFI_ERROR (Status)) {
@@ -596,7 +673,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -605,9 +682,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->ModuleProductID != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "ModuleProductID", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "ModuleProductID");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->ModuleProductID);
       if (EFI_ERROR (Status)) {
@@ -616,7 +693,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -625,9 +702,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->NonVolatileSizeMiB != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "NonVolatileSizeMiB", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "NonVolatileSizeMiB");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->NonVolatileSizeMiB);
       if (EFI_ERROR (Status)) {
@@ -636,18 +713,26 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
+  //
+  // Handle OPERATINGMEMORYMODES
+  //
+  //
+  // ****** Warning ******
+  // Unsupported array type:
+  //
+
   //
   // Handle OPERATINGSPEEDMHZ
   //
   if (MemoryCs->OperatingSpeedMhz != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "OperatingSpeedMhz", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "OperatingSpeedMhz");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->OperatingSpeedMhz);
       if (EFI_ERROR (Status)) {
@@ -656,7 +741,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -665,9 +750,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->PartNumber != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "PartNumber", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "PartNumber");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->PartNumber);
       if (EFI_ERROR (Status)) {
@@ -676,7 +761,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -685,9 +770,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->PersistentRegionNumberLimit != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "PersistentRegionNumberLimit", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "PersistentRegionNumberLimit");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->PersistentRegionNumberLimit);
       if (EFI_ERROR (Status)) {
@@ -696,7 +781,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -705,9 +790,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->PersistentRegionSizeLimitMiB != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "PersistentRegionSizeLimitMiB", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "PersistentRegionSizeLimitMiB");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->PersistentRegionSizeLimitMiB);
       if (EFI_ERROR (Status)) {
@@ -716,7 +801,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -725,9 +810,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->PersistentRegionSizeMaxMiB != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "PersistentRegionSizeMaxMiB", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "PersistentRegionSizeMaxMiB");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->PersistentRegionSizeMaxMiB);
       if (EFI_ERROR (Status)) {
@@ -736,7 +821,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -753,9 +838,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->PowerManagementPolicy->AveragePowerBudgetMilliWatts != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "PowerManagementPolicy/AveragePowerBudgetMilliWatts", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "PowerManagementPolicy/AveragePowerBudgetMilliWatts");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->PowerManagementPolicy->AveragePowerBudgetMilliWatts);
       if (EFI_ERROR (Status)) {
@@ -764,7 +849,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -773,9 +858,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->PowerManagementPolicy->MaxTDPMilliWatts != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "PowerManagementPolicy/MaxTDPMilliWatts", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "PowerManagementPolicy/MaxTDPMilliWatts");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->PowerManagementPolicy->MaxTDPMilliWatts);
       if (EFI_ERROR (Status)) {
@@ -784,7 +869,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -793,9 +878,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->PowerManagementPolicy->PeakPowerBudgetMilliWatts != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "PowerManagementPolicy/PeakPowerBudgetMilliWatts", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "PowerManagementPolicy/PeakPowerBudgetMilliWatts");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->PowerManagementPolicy->PeakPowerBudgetMilliWatts);
       if (EFI_ERROR (Status)) {
@@ -804,7 +889,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -813,9 +898,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->PowerManagementPolicy->PolicyEnabled != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "PowerManagementPolicy/PolicyEnabled", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "PowerManagementPolicy/PolicyEnabled");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsBooleanType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (BOOLEAN)*MemoryCs->PowerManagementPolicy->PolicyEnabled);
       if (EFI_ERROR (Status)) {
@@ -824,7 +909,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -833,9 +918,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->RankCount != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "RankCount", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "RankCount");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->RankCount);
       if (EFI_ERROR (Status)) {
@@ -844,10 +929,18 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
+  //
+  // Handle REGIONS
+  //
+  //
+  // ****** Warning ******
+  // Unsupported array type:
+  //
+
   //
   // Handle SECURITYCAPABILITIES
   //
@@ -861,9 +954,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->SecurityCapabilities->ConfigurationLockCapable != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "SecurityCapabilities/ConfigurationLockCapable", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "SecurityCapabilities/ConfigurationLockCapable");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsBooleanType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (BOOLEAN)*MemoryCs->SecurityCapabilities->ConfigurationLockCapable);
       if (EFI_ERROR (Status)) {
@@ -872,7 +965,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -881,9 +974,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->SecurityCapabilities->DataLockCapable != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "SecurityCapabilities/DataLockCapable", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "SecurityCapabilities/DataLockCapable");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsBooleanType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (BOOLEAN)*MemoryCs->SecurityCapabilities->DataLockCapable);
       if (EFI_ERROR (Status)) {
@@ -892,7 +985,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -901,9 +994,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->SecurityCapabilities->MaxPassphraseCount != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "SecurityCapabilities/MaxPassphraseCount", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "SecurityCapabilities/MaxPassphraseCount");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->SecurityCapabilities->MaxPassphraseCount);
       if (EFI_ERROR (Status)) {
@@ -912,7 +1005,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -921,9 +1014,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->SecurityCapabilities->PassphraseCapable != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "SecurityCapabilities/PassphraseCapable", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "SecurityCapabilities/PassphraseCapable");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsBooleanType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (BOOLEAN)*MemoryCs->SecurityCapabilities->PassphraseCapable);
       if (EFI_ERROR (Status)) {
@@ -932,7 +1025,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -941,9 +1034,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->SecurityCapabilities->PassphraseLockLimit != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "SecurityCapabilities/PassphraseLockLimit", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "SecurityCapabilities/PassphraseLockLimit");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->SecurityCapabilities->PassphraseLockLimit);
       if (EFI_ERROR (Status)) {
@@ -952,18 +1045,26 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
+  //
+  // Handle SECURITYCAPABILITIES->SECURITYSTATES
+  //
+  //
+  // ****** Warning ******
+  // Unsupported array type:
+  //
+
   //
   // Handle SECURITYSTATE
   //
   if (MemoryCs->SecurityState != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "SecurityState", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "SecurityState");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->SecurityState);
       if (EFI_ERROR (Status)) {
@@ -972,7 +1073,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -981,9 +1082,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->SerialNumber != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "SerialNumber", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "SerialNumber");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->SerialNumber);
       if (EFI_ERROR (Status)) {
@@ -992,7 +1093,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -1001,9 +1102,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->SpareDeviceCount != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "SpareDeviceCount", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "SpareDeviceCount");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->SpareDeviceCount);
       if (EFI_ERROR (Status)) {
@@ -1012,7 +1113,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -1021,9 +1122,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->SubsystemDeviceID != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "SubsystemDeviceID", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "SubsystemDeviceID");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->SubsystemDeviceID);
       if (EFI_ERROR (Status)) {
@@ -1032,7 +1133,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -1041,9 +1142,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->SubsystemVendorID != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "SubsystemVendorID", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "SubsystemVendorID");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->SubsystemVendorID);
       if (EFI_ERROR (Status)) {
@@ -1052,7 +1153,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -1061,9 +1162,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->VendorID != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "VendorID", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "VendorID");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->VendorID);
       if (EFI_ERROR (Status)) {
@@ -1072,7 +1173,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -1081,9 +1182,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->VolatileRegionNumberLimit != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "VolatileRegionNumberLimit", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "VolatileRegionNumberLimit");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->VolatileRegionNumberLimit);
       if (EFI_ERROR (Status)) {
@@ -1092,7 +1193,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -1101,9 +1202,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->VolatileRegionSizeLimitMiB != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "VolatileRegionSizeLimitMiB", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "VolatileRegionSizeLimitMiB");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->VolatileRegionSizeLimitMiB);
       if (EFI_ERROR (Status)) {
@@ -1112,7 +1213,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -1121,9 +1222,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->VolatileRegionSizeMaxMiB != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "VolatileRegionSizeMaxMiB", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "VolatileRegionSizeMaxMiB");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->VolatileRegionSizeMaxMiB);
       if (EFI_ERROR (Status)) {
@@ -1132,7 +1233,7 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
@@ -1141,9 +1242,9 @@ RedfishConsumeResourceCommon (
   //
   if (MemoryCs->VolatileSizeMiB != NULL) {
     //
-    // Find corresponding redpath for collection resource.
+    // Find corresponding configure language for collection resource.
     //
-    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, "VolatileSizeMiB", Arraykey);
+    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "VolatileSizeMiB");
     if (ConfigureLang != NULL) {
       Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs->VolatileSizeMiB);
       if (EFI_ERROR (Status)) {
@@ -1152,24 +1253,15 @@ RedfishConsumeResourceCommon (
 
       FreePool (ConfigureLang);
     } else {
-      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key: %a\n", __FUNCTION__, Arraykey));
+      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI: %s\n", __FUNCTION__, Private->Uri));
     }
   }
 
-  //
 ON_RELEASE:
 
   //
   // Release resource.
   //
-  if (EtagInDb != NULL) {
-    FreePool (EtagInDb);
-  }
-
-  if (Arraykey != NULL) {
-    FreePool (Arraykey);
-  }
-
   Private->JsonStructProtocol->DestoryStructure (
                                  Private->JsonStructProtocol,
                                  (EFI_REST_JSON_STRUCTURE_HEADER *)Memory
@@ -1181,7 +1273,7 @@ ON_RELEASE:
 EFI_STATUS
 ProvisioningMemoryProperties (
   IN  EFI_REST_JSON_STRUCTURE_PROTOCOL *JsonStructProtocol,
-  IN  CHAR8 *IputJson,
+  IN  CHAR8 *InputJson,
   IN  CHAR8 *ResourceId, OPTIONAL
   IN  EFI_STRING                        ConfigureLang,
   IN  BOOLEAN                           ProvisionMode,
@@ -1191,28 +1283,29 @@ ProvisioningMemoryProperties (
   EFI_REDFISH_MEMORY_V1_7_1     *Memory;
   EFI_REDFISH_MEMORY_V1_7_1_CS  *MemoryCs;
   EFI_STATUS                    Status;
+  BOOLEAN                       PropertyChanged;
   INT64                         *NumericValue;
-  INT32                         *IntegerValue;
-  BOOLEAN                       *BooleanValue;
+  INT64                         *NumericArrayValue;
+  UINTN                         ArraySize;
   CHAR8                         *AsciiStringValue;
-  BOOLEAN                       PropertyChanged;
-  BOOLEAN                       UnusedProperty;
+  BOOLEAN                       *BooleanValue;
+  INT32                         *IntegerValue;
+  CHAR8                         **AsciiStringArrayValue;
 
-  if ((JsonStructProtocol == NULL) || (ResultJson == NULL) || IS_EMPTY_STRING (IputJson) || IS_EMPTY_STRING (ConfigureLang)) {
+  if ((JsonStructProtocol == NULL) || (ResultJson == NULL) || IS_EMPTY_STRING (InputJson) || IS_EMPTY_STRING (ConfigureLang)) {
     return EFI_INVALID_PARAMETER;
   }
 
-  DEBUG ((REDFISH_DEBUG_TRACE, "%a provision for %s with: %s\n", __FUNCTION__, ConfigureLang, (ProvisionMode ? L"Provision all resource" : L"Provision existing resource")));
+  DEBUG ((REDFISH_DEBUG_TRACE, "%a provision for %s with: %s\n", __FUNCTION__, ConfigureLang, (ProvisionMode ? L"Provision resource" : L"Update resource")));
 
   *ResultJson     = NULL;
   PropertyChanged = FALSE;
-  UnusedProperty  = TRUE;
 
   Memory = NULL;
   Status = JsonStructProtocol->ToStructure (
                                  JsonStructProtocol,
                                  NULL,
-                                 IputJson,
+                                 InputJson,
                                  (EFI_REST_JSON_STRUCTURE_HEADER **)&Memory
                                  );
   if (EFI_ERROR (Status)) {
@@ -1255,6 +1348,19 @@ ProvisioningMemoryProperties (
     }
   }
 
+  //
+  // Handle ALLOWEDSPEEDSMHZ
+  //
+  if (PropertyChecker (MemoryCs->AllowedSpeedsMHz, ProvisionMode)) {
+    NumericArrayValue = GetPropertyNumericArrayValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"AllowedSpeedsMHz", ConfigureLang, &ArraySize);
+    if (NumericArrayValue != NULL) {
+      if (ProvisionMode || !CompareRedfishNumericArrayValues (MemoryCs->AllowedSpeedsMHz, NumericArrayValue, ArraySize)) {
+        AddRedfishNumericArray (&MemoryCs->AllowedSpeedsMHz, NumericArrayValue, ArraySize);
+        PropertyChanged = TRUE;
+      }
+    }
+  }
+
   //
   // Handle BASEMODULETYPE
   //
@@ -1318,6 +1424,7 @@ ProvisioningMemoryProperties (
         if (IntegerValue != NULL) {
           *IntegerValue                 = (BooleanValue ? 0x01 : 0x00);
           MemoryCs->ConfigurationLocked = IntegerValue;
+          PropertyChanged               = TRUE;
         }
       }
     }
@@ -1401,6 +1508,19 @@ ProvisioningMemoryProperties (
     }
   }
 
+  //
+  // Handle FUNCTIONCLASSES
+  //
+  if (PropertyChecker (MemoryCs->FunctionClasses, ProvisionMode)) {
+    AsciiStringArrayValue = GetPropertyStringArrayValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"FunctionClasses", ConfigureLang, &ArraySize);
+    if (AsciiStringArrayValue != NULL) {
+      if (ProvisionMode || !CompareRedfishStringArrayValues (MemoryCs->FunctionClasses, AsciiStringArrayValue, ArraySize)) {
+        AddRedfishCharArray (&MemoryCs->FunctionClasses, AsciiStringArrayValue, ArraySize);
+        PropertyChanged = TRUE;
+      }
+    }
+  }
+
   //
   // Handle ISRANKSPAREENABLED
   //
@@ -1412,6 +1532,7 @@ ProvisioningMemoryProperties (
         if (IntegerValue != NULL) {
           *IntegerValue                = (BooleanValue ? 0x01 : 0x00);
           MemoryCs->IsRankSpareEnabled = IntegerValue;
+          PropertyChanged              = TRUE;
         }
       }
     }
@@ -1428,6 +1549,7 @@ ProvisioningMemoryProperties (
         if (IntegerValue != NULL) {
           *IntegerValue                  = (BooleanValue ? 0x01 : 0x00);
           MemoryCs->IsSpareDeviceEnabled = IntegerValue;
+          PropertyChanged                = TRUE;
         }
       }
     }
@@ -1459,6 +1581,19 @@ ProvisioningMemoryProperties (
     }
   }
 
+  //
+  // Handle MAXTDPMILLIWATTS
+  //
+  if (PropertyChecker (MemoryCs->MaxTDPMilliWatts, ProvisionMode)) {
+    NumericArrayValue = GetPropertyNumericArrayValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"MaxTDPMilliWatts", ConfigureLang, &ArraySize);
+    if (NumericArrayValue != NULL) {
+      if (ProvisionMode || !CompareRedfishNumericArrayValues (MemoryCs->MaxTDPMilliWatts, NumericArrayValue, ArraySize)) {
+        AddRedfishNumericArray (&MemoryCs->MaxTDPMilliWatts, NumericArrayValue, ArraySize);
+        PropertyChanged = TRUE;
+      }
+    }
+  }
+
   //
   // Handle MEMORYDEVICETYPE
   //
@@ -1475,74 +1610,66 @@ ProvisioningMemoryProperties (
   //
   // Handle MEMORYLOCATION
   //
-  if (MemoryCs->MemoryLocation == NULL) {
-    MemoryCs->MemoryLocation = AllocateZeroPool (sizeof (RedfishMemory_V1_7_1_MemoryLocation_CS));
-    ASSERT (MemoryCs->MemoryLocation != NULL);
-    UnusedProperty = TRUE;
-  } else {
-    UnusedProperty = FALSE;
-  }
-
-  //
-  // Handle MEMORYLOCATION->CHANNEL
-  //
-  if (PropertyChecker (MemoryCs->MemoryLocation->Channel, ProvisionMode)) {
-    NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"MemoryLocation/Channel", ConfigureLang);
-    if (NumericValue != NULL) {
-      if (ProvisionMode || (*MemoryCs->MemoryLocation->Channel != *NumericValue)) {
-        MemoryCs->MemoryLocation->Channel = NumericValue;
-        PropertyChanged                   = TRUE;
-        UnusedProperty                    = FALSE;
+  if (MemoryCs->MemoryLocation != NULL) {
+    //
+    // Handle MEMORYLOCATION->CHANNEL
+    //
+    if (PropertyChecker (MemoryCs->MemoryLocation->Channel, ProvisionMode)) {
+      NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"MemoryLocation/Channel", ConfigureLang);
+      if (NumericValue != NULL) {
+        if (ProvisionMode || (*MemoryCs->MemoryLocation->Channel != *NumericValue)) {
+          MemoryCs->MemoryLocation->Channel = NumericValue;
+          PropertyChanged                   = TRUE;
+        }
       }
     }
-  }
 
-  //
-  // Handle MEMORYLOCATION->MEMORYCONTROLLER
-  //
-  if (PropertyChecker (MemoryCs->MemoryLocation->MemoryController, ProvisionMode)) {
-    NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"MemoryLocation/MemoryController", ConfigureLang);
-    if (NumericValue != NULL) {
-      if (ProvisionMode || (*MemoryCs->MemoryLocation->MemoryController != *NumericValue)) {
-        MemoryCs->MemoryLocation->MemoryController = NumericValue;
-        PropertyChanged                            = TRUE;
-        UnusedProperty                             = FALSE;
+    //
+    // Handle MEMORYLOCATION->MEMORYCONTROLLER
+    //
+    if (PropertyChecker (MemoryCs->MemoryLocation->MemoryController, ProvisionMode)) {
+      NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"MemoryLocation/MemoryController", ConfigureLang);
+      if (NumericValue != NULL) {
+        if (ProvisionMode || (*MemoryCs->MemoryLocation->MemoryController != *NumericValue)) {
+          MemoryCs->MemoryLocation->MemoryController = NumericValue;
+          PropertyChanged                            = TRUE;
+        }
       }
     }
-  }
 
-  //
-  // Handle MEMORYLOCATION->SLOT
-  //
-  if (PropertyChecker (MemoryCs->MemoryLocation->Slot, ProvisionMode)) {
-    NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"MemoryLocation/Slot", ConfigureLang);
-    if (NumericValue != NULL) {
-      if (ProvisionMode || (*MemoryCs->MemoryLocation->Slot != *NumericValue)) {
-        MemoryCs->MemoryLocation->Slot = NumericValue;
-        PropertyChanged                = TRUE;
-        UnusedProperty                 = FALSE;
+    //
+    // Handle MEMORYLOCATION->SLOT
+    //
+    if (PropertyChecker (MemoryCs->MemoryLocation->Slot, ProvisionMode)) {
+      NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"MemoryLocation/Slot", ConfigureLang);
+      if (NumericValue != NULL) {
+        if (ProvisionMode || (*MemoryCs->MemoryLocation->Slot != *NumericValue)) {
+          MemoryCs->MemoryLocation->Slot = NumericValue;
+          PropertyChanged                = TRUE;
+        }
       }
     }
-  }
 
-  //
-  // Handle MEMORYLOCATION->SOCKET
-  //
-  if (PropertyChecker (MemoryCs->MemoryLocation->Socket, ProvisionMode)) {
-    NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"MemoryLocation/Socket", ConfigureLang);
-    if (NumericValue != NULL) {
-      if (ProvisionMode || (*MemoryCs->MemoryLocation->Socket != *NumericValue)) {
-        MemoryCs->MemoryLocation->Socket = NumericValue;
-        PropertyChanged                  = TRUE;
-        UnusedProperty                   = FALSE;
+    //
+    // Handle MEMORYLOCATION->SOCKET
+    //
+    if (PropertyChecker (MemoryCs->MemoryLocation->Socket, ProvisionMode)) {
+      NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"MemoryLocation/Socket", ConfigureLang);
+      if (NumericValue != NULL) {
+        if (ProvisionMode || (*MemoryCs->MemoryLocation->Socket != *NumericValue)) {
+          MemoryCs->MemoryLocation->Socket = NumericValue;
+          PropertyChanged                  = TRUE;
+        }
       }
     }
   }
 
-  if (UnusedProperty) {
-    FreePool (MemoryCs->MemoryLocation);
-    MemoryCs->MemoryLocation = NULL;
-  }
+  //
+  // Handle MEMORYMEDIA
+  //
+  //// ****** Warning ******
+  // Unsupported array type:
+  //
 
   //
   // Handle MEMORYSUBSYSTEMCONTROLLERMANUFACTURERID
@@ -1622,6 +1749,13 @@ ProvisioningMemoryProperties (
     }
   }
 
+  //
+  // Handle OPERATINGMEMORYMODES
+  //
+  //// ****** Warning ******
+  // Unsupported array type:
+  //
+
   //
   // Handle OPERATINGSPEEDMHZ
   //
@@ -1690,79 +1824,64 @@ ProvisioningMemoryProperties (
   //
   // Handle POWERMANAGEMENTPOLICY
   //
-  if (MemoryCs->PowerManagementPolicy == NULL) {
-    MemoryCs->PowerManagementPolicy = AllocateZeroPool (sizeof (RedfishMemory_V1_7_1_PowerManagementPolicy_CS));
-    ASSERT (MemoryCs->PowerManagementPolicy != NULL);
-    UnusedProperty = TRUE;
-  } else {
-    UnusedProperty = FALSE;
-  }
-
-  //
-  // Handle POWERMANAGEMENTPOLICY->AVERAGEPOWERBUDGETMILLIWATTS
-  //
-  if (PropertyChecker (MemoryCs->PowerManagementPolicy->AveragePowerBudgetMilliWatts, ProvisionMode)) {
-    NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"PowerManagementPolicy/AveragePowerBudgetMilliWatts", ConfigureLang);
-    if (NumericValue != NULL) {
-      if (ProvisionMode || (*MemoryCs->PowerManagementPolicy->AveragePowerBudgetMilliWatts != *NumericValue)) {
-        MemoryCs->PowerManagementPolicy->AveragePowerBudgetMilliWatts = NumericValue;
-        PropertyChanged                                               = TRUE;
-        UnusedProperty                                                = FALSE;
+  if (MemoryCs->PowerManagementPolicy != NULL) {
+    //
+    // Handle POWERMANAGEMENTPOLICY->AVERAGEPOWERBUDGETMILLIWATTS
+    //
+    if (PropertyChecker (MemoryCs->PowerManagementPolicy->AveragePowerBudgetMilliWatts, ProvisionMode)) {
+      NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"PowerManagementPolicy/AveragePowerBudgetMilliWatts", ConfigureLang);
+      if (NumericValue != NULL) {
+        if (ProvisionMode || (*MemoryCs->PowerManagementPolicy->AveragePowerBudgetMilliWatts != *NumericValue)) {
+          MemoryCs->PowerManagementPolicy->AveragePowerBudgetMilliWatts = NumericValue;
+          PropertyChanged                                               = TRUE;
+        }
       }
     }
-  }
 
-  //
-  // Handle POWERMANAGEMENTPOLICY->MAXTDPMILLIWATTS
-  //
-  if (PropertyChecker (MemoryCs->PowerManagementPolicy->MaxTDPMilliWatts, ProvisionMode)) {
-    NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"PowerManagementPolicy/MaxTDPMilliWatts", ConfigureLang);
-    if (NumericValue != NULL) {
-      if (ProvisionMode || (*MemoryCs->PowerManagementPolicy->MaxTDPMilliWatts != *NumericValue)) {
-        MemoryCs->PowerManagementPolicy->MaxTDPMilliWatts = NumericValue;
-        PropertyChanged                                   = TRUE;
-        UnusedProperty                                    = FALSE;
+    //
+    // Handle POWERMANAGEMENTPOLICY->MAXTDPMILLIWATTS
+    //
+    if (PropertyChecker (MemoryCs->PowerManagementPolicy->MaxTDPMilliWatts, ProvisionMode)) {
+      NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"PowerManagementPolicy/MaxTDPMilliWatts", ConfigureLang);
+      if (NumericValue != NULL) {
+        if (ProvisionMode || (*MemoryCs->PowerManagementPolicy->MaxTDPMilliWatts != *NumericValue)) {
+          MemoryCs->PowerManagementPolicy->MaxTDPMilliWatts = NumericValue;
+          PropertyChanged                                   = TRUE;
+        }
       }
     }
-  }
 
-  //
-  // Handle POWERMANAGEMENTPOLICY->PEAKPOWERBUDGETMILLIWATTS
-  //
-  if (PropertyChecker (MemoryCs->PowerManagementPolicy->PeakPowerBudgetMilliWatts, ProvisionMode)) {
-    NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"PowerManagementPolicy/PeakPowerBudgetMilliWatts", ConfigureLang);
-    if (NumericValue != NULL) {
-      if (ProvisionMode || (*MemoryCs->PowerManagementPolicy->PeakPowerBudgetMilliWatts != *NumericValue)) {
-        MemoryCs->PowerManagementPolicy->PeakPowerBudgetMilliWatts = NumericValue;
-        PropertyChanged                                            = TRUE;
-        UnusedProperty                                             = FALSE;
+    //
+    // Handle POWERMANAGEMENTPOLICY->PEAKPOWERBUDGETMILLIWATTS
+    //
+    if (PropertyChecker (MemoryCs->PowerManagementPolicy->PeakPowerBudgetMilliWatts, ProvisionMode)) {
+      NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"PowerManagementPolicy/PeakPowerBudgetMilliWatts", ConfigureLang);
+      if (NumericValue != NULL) {
+        if (ProvisionMode || (*MemoryCs->PowerManagementPolicy->PeakPowerBudgetMilliWatts != *NumericValue)) {
+          MemoryCs->PowerManagementPolicy->PeakPowerBudgetMilliWatts = NumericValue;
+          PropertyChanged                                            = TRUE;
+        }
       }
     }
-  }
 
-  //
-  // Handle POWERMANAGEMENTPOLICY->POLICYENABLED
-  //
-  if (PropertyChecker (MemoryCs->PowerManagementPolicy->PolicyEnabled, ProvisionMode)) {
-    BooleanValue = GetPropertyBooleanValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"PowerManagementPolicy/PolicyEnabled", ConfigureLang);
-    if (BooleanValue != NULL) {
-      if (ProvisionMode || (*MemoryCs->PowerManagementPolicy->PolicyEnabled != *BooleanValue)) {
-        IntegerValue = AllocatePool (sizeof (*IntegerValue));
-        if (IntegerValue != NULL) {
-          *IntegerValue                                  = (BooleanValue ? 0x01 : 0x00);
-          MemoryCs->PowerManagementPolicy->PolicyEnabled = IntegerValue;
-          PropertyChanged                                = TRUE;
-          UnusedProperty                                 = FALSE;
+    //
+    // Handle POWERMANAGEMENTPOLICY->POLICYENABLED
+    //
+    if (PropertyChecker (MemoryCs->PowerManagementPolicy->PolicyEnabled, ProvisionMode)) {
+      BooleanValue = GetPropertyBooleanValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"PowerManagementPolicy/PolicyEnabled", ConfigureLang);
+      if (BooleanValue != NULL) {
+        if (ProvisionMode || (*MemoryCs->PowerManagementPolicy->PolicyEnabled != *BooleanValue)) {
+          IntegerValue = AllocatePool (sizeof (*IntegerValue));
+          if (IntegerValue != NULL) {
+            *IntegerValue                                  = (BooleanValue ? 0x01 : 0x00);
+            MemoryCs->PowerManagementPolicy->PolicyEnabled = IntegerValue;
+            PropertyChanged                                = TRUE;
+          }
         }
       }
     }
   }
 
-  if (UnusedProperty) {
-    FreePool (MemoryCs->PowerManagementPolicy);
-    MemoryCs->PowerManagementPolicy = NULL;
-  }
-
   //
   // Handle RANKCOUNT
   //
@@ -1777,101 +1896,99 @@ ProvisioningMemoryProperties (
   }
 
   //
-  // Handle SECURITYCAPABILITIES
+  // Handle REGIONS
+  //
+  //// ****** Warning ******
+  // Unsupported array type:
   //
-  if (MemoryCs->SecurityCapabilities == NULL) {
-    MemoryCs->SecurityCapabilities = AllocateZeroPool (sizeof (RedfishMemory_V1_7_1_SecurityCapabilities_CS));
-    ASSERT (MemoryCs->SecurityCapabilities != NULL);
-    UnusedProperty = TRUE;
-  } else {
-    UnusedProperty = FALSE;
-  }
 
   //
-  // Handle SECURITYCAPABILITIES->CONFIGURATIONLOCKCAPABLE
+  // Handle SECURITYCAPABILITIES
   //
-  if (PropertyChecker (MemoryCs->SecurityCapabilities->ConfigurationLockCapable, ProvisionMode)) {
-    BooleanValue = GetPropertyBooleanValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"SecurityCapabilities/ConfigurationLockCapable", ConfigureLang);
-    if (BooleanValue != NULL) {
-      if (ProvisionMode || (*MemoryCs->SecurityCapabilities->ConfigurationLockCapable != *BooleanValue)) {
-        IntegerValue = AllocatePool (sizeof (*IntegerValue));
-        if (IntegerValue != NULL) {
-          *IntegerValue                                            = (BooleanValue ? 0x01 : 0x00);
-          MemoryCs->SecurityCapabilities->ConfigurationLockCapable = IntegerValue;
-          PropertyChanged                                          = TRUE;
-          UnusedProperty                                           = FALSE;
+  if (MemoryCs->SecurityCapabilities != NULL) {
+    //
+    // Handle SECURITYCAPABILITIES->CONFIGURATIONLOCKCAPABLE
+    //
+    if (PropertyChecker (MemoryCs->SecurityCapabilities->ConfigurationLockCapable, ProvisionMode)) {
+      BooleanValue = GetPropertyBooleanValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"SecurityCapabilities/ConfigurationLockCapable", ConfigureLang);
+      if (BooleanValue != NULL) {
+        if (ProvisionMode || (*MemoryCs->SecurityCapabilities->ConfigurationLockCapable != *BooleanValue)) {
+          IntegerValue = AllocatePool (sizeof (*IntegerValue));
+          if (IntegerValue != NULL) {
+            *IntegerValue                                            = (BooleanValue ? 0x01 : 0x00);
+            MemoryCs->SecurityCapabilities->ConfigurationLockCapable = IntegerValue;
+            PropertyChanged                                          = TRUE;
+          }
         }
       }
     }
-  }
 
-  //
-  // Handle SECURITYCAPABILITIES->DATALOCKCAPABLE
-  //
-  if (PropertyChecker (MemoryCs->SecurityCapabilities->DataLockCapable, ProvisionMode)) {
-    BooleanValue = GetPropertyBooleanValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"SecurityCapabilities/DataLockCapable", ConfigureLang);
-    if (BooleanValue != NULL) {
-      if (ProvisionMode || (*MemoryCs->SecurityCapabilities->DataLockCapable != *BooleanValue)) {
-        IntegerValue = AllocatePool (sizeof (*IntegerValue));
-        if (IntegerValue != NULL) {
-          *IntegerValue                                   = (BooleanValue ? 0x01 : 0x00);
-          MemoryCs->SecurityCapabilities->DataLockCapable = IntegerValue;
-          PropertyChanged                                 = TRUE;
-          UnusedProperty                                  = FALSE;
+    //
+    // Handle SECURITYCAPABILITIES->DATALOCKCAPABLE
+    //
+    if (PropertyChecker (MemoryCs->SecurityCapabilities->DataLockCapable, ProvisionMode)) {
+      BooleanValue = GetPropertyBooleanValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"SecurityCapabilities/DataLockCapable", ConfigureLang);
+      if (BooleanValue != NULL) {
+        if (ProvisionMode || (*MemoryCs->SecurityCapabilities->DataLockCapable != *BooleanValue)) {
+          IntegerValue = AllocatePool (sizeof (*IntegerValue));
+          if (IntegerValue != NULL) {
+            *IntegerValue                                   = (BooleanValue ? 0x01 : 0x00);
+            MemoryCs->SecurityCapabilities->DataLockCapable = IntegerValue;
+            PropertyChanged                                 = TRUE;
+          }
         }
       }
     }
-  }
 
-  //
-  // Handle SECURITYCAPABILITIES->MAXPASSPHRASECOUNT
-  //
-  if (PropertyChecker (MemoryCs->SecurityCapabilities->MaxPassphraseCount, ProvisionMode)) {
-    NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"SecurityCapabilities/MaxPassphraseCount", ConfigureLang);
-    if (NumericValue != NULL) {
-      if (ProvisionMode || (*MemoryCs->SecurityCapabilities->MaxPassphraseCount != *NumericValue)) {
-        MemoryCs->SecurityCapabilities->MaxPassphraseCount = NumericValue;
-        PropertyChanged                                    = TRUE;
-        UnusedProperty                                     = FALSE;
+    //
+    // Handle SECURITYCAPABILITIES->MAXPASSPHRASECOUNT
+    //
+    if (PropertyChecker (MemoryCs->SecurityCapabilities->MaxPassphraseCount, ProvisionMode)) {
+      NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"SecurityCapabilities/MaxPassphraseCount", ConfigureLang);
+      if (NumericValue != NULL) {
+        if (ProvisionMode || (*MemoryCs->SecurityCapabilities->MaxPassphraseCount != *NumericValue)) {
+          MemoryCs->SecurityCapabilities->MaxPassphraseCount = NumericValue;
+          PropertyChanged                                    = TRUE;
+        }
       }
     }
-  }
 
-  //
-  // Handle SECURITYCAPABILITIES->PASSPHRASECAPABLE
-  //
-  if (PropertyChecker (MemoryCs->SecurityCapabilities->PassphraseCapable, ProvisionMode)) {
-    BooleanValue = GetPropertyBooleanValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"SecurityCapabilities/PassphraseCapable", ConfigureLang);
-    if (BooleanValue != NULL) {
-      if (ProvisionMode || (*MemoryCs->SecurityCapabilities->PassphraseCapable != *BooleanValue)) {
-        IntegerValue = AllocatePool (sizeof (*IntegerValue));
-        if (IntegerValue != NULL) {
-          *IntegerValue                                     = (BooleanValue ? 0x01 : 0x00);
-          MemoryCs->SecurityCapabilities->PassphraseCapable = IntegerValue;
-          PropertyChanged                                   = TRUE;
-          UnusedProperty                                    = FALSE;
+    //
+    // Handle SECURITYCAPABILITIES->PASSPHRASECAPABLE
+    //
+    if (PropertyChecker (MemoryCs->SecurityCapabilities->PassphraseCapable, ProvisionMode)) {
+      BooleanValue = GetPropertyBooleanValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"SecurityCapabilities/PassphraseCapable", ConfigureLang);
+      if (BooleanValue != NULL) {
+        if (ProvisionMode || (*MemoryCs->SecurityCapabilities->PassphraseCapable != *BooleanValue)) {
+          IntegerValue = AllocatePool (sizeof (*IntegerValue));
+          if (IntegerValue != NULL) {
+            *IntegerValue                                     = (BooleanValue ? 0x01 : 0x00);
+            MemoryCs->SecurityCapabilities->PassphraseCapable = IntegerValue;
+            PropertyChanged                                   = TRUE;
+          }
         }
       }
     }
-  }
 
-  //
-  // Handle SECURITYCAPABILITIES->PASSPHRASELOCKLIMIT
-  //
-  if (PropertyChecker (MemoryCs->SecurityCapabilities->PassphraseLockLimit, ProvisionMode)) {
-    NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"SecurityCapabilities/PassphraseLockLimit", ConfigureLang);
-    if (NumericValue != NULL) {
-      if (ProvisionMode || (*MemoryCs->SecurityCapabilities->PassphraseLockLimit != *NumericValue)) {
-        MemoryCs->SecurityCapabilities->PassphraseLockLimit = NumericValue;
-        PropertyChanged                                     = TRUE;
-        UnusedProperty                                      = FALSE;
+    //
+    // Handle SECURITYCAPABILITIES->PASSPHRASELOCKLIMIT
+    //
+    if (PropertyChecker (MemoryCs->SecurityCapabilities->PassphraseLockLimit, ProvisionMode)) {
+      NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"SecurityCapabilities/PassphraseLockLimit", ConfigureLang);
+      if (NumericValue != NULL) {
+        if (ProvisionMode || (*MemoryCs->SecurityCapabilities->PassphraseLockLimit != *NumericValue)) {
+          MemoryCs->SecurityCapabilities->PassphraseLockLimit = NumericValue;
+          PropertyChanged                                     = TRUE;
+        }
       }
     }
-  }
 
-  if (UnusedProperty) {
-    FreePool (MemoryCs->SecurityCapabilities);
-    MemoryCs->SecurityCapabilities = NULL;
+    //
+    // Handle SECURITYCAPABILITIES->SECURITYSTATES
+    //
+    //// ****** Warning ******
+    // Unsupported array type:
+    //
   }
 
   //
@@ -2035,18 +2152,17 @@ ProvisioningMemoryResource (
   IN  EFI_STRING                       ConfigureLang
   )
 {
-  CHAR8       *MemoryJson;
+  CHAR8       *Json;
   EFI_STATUS  Status;
-  CHAR8       *NewResourceLocation;
-  CHAR8       *NewKey;
+  EFI_STRING  NewResourceLocation;
   CHAR8       *EtagStr;
   CHAR8       ResourceId[16];
-  CHAR8       NewUri[255];
 
   if (IS_EMPTY_STRING (ConfigureLang) || (Private == NULL)) {
     return EFI_INVALID_PARAMETER;
   }
 
+  EtagStr = NULL;
   AsciiSPrint (ResourceId, sizeof (ResourceId), "%d", Index);
 
   Status = ProvisioningMemoryProperties (
@@ -2055,55 +2171,45 @@ ProvisioningMemoryResource (
              ResourceId,
              ConfigureLang,
              TRUE,
-             &MemoryJson
+             &Json
              );
   if (EFI_ERROR (Status)) {
     DEBUG ((DEBUG_ERROR, "%a, provisioning resource for %s failed: %r\n", __FUNCTION__, ConfigureLang, Status));
     return Status;
   }
 
-  Status = CreatePayloadToPostResource (Private->RedfishService, Private->Payload, MemoryJson, &NewResourceLocation, &EtagStr);
+  Status = CreatePayloadToPostResource (Private->RedfishService, Private->Payload, Json, &NewResourceLocation, &EtagStr);
   if (EFI_ERROR (Status)) {
-    DEBUG ((DEBUG_ERROR, "%a, post memory resource for %s failed: %r\n", __FUNCTION__, ConfigureLang, Status));
+    DEBUG ((DEBUG_ERROR, "%a, post Memory resource for %s failed: %r\n", __FUNCTION__, ConfigureLang, Status));
     goto RELEASE_RESOURCE;
   }
 
-  NewUri[0] = '\0';
+  ASSERT (NewResourceLocation != NULL);
 
   //
   // Keep location of new resource.
   //
   if (NewResourceLocation != NULL) {
-    //
-    // Find key
-    //
-    NewKey = AsciiStrStr (NewResourceLocation, RESOURCE_SCHEMA);
-    if (NewKey != NULL) {
-      NewKey += 6;
-      //
-      // skip '/'
-      //
-      if (NewKey[0] == '/') {
-        NewKey++;
-      }
-
-      SetConfigureLangWithkey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, NewKey, Index);
-      AsciiSPrint (NewUri, sizeof (NewUri), "%a[%a]", Private->Uri, NewKey);
-      FreePool (NewResourceLocation);
-    }
+    RedfisSetRedfishUri (ConfigureLang, NewResourceLocation);
   }
 
   //
   // Handle Etag
   //
   if (EtagStr != NULL) {
-    SetEtagWithUri (EtagStr, NewUri);
+    SetEtagWithUri (EtagStr, NewResourceLocation);
     FreePool (EtagStr);
   }
 
 RELEASE_RESOURCE:
 
-  FreePool (MemoryJson);
+  if (NewResourceLocation != NULL) {
+    FreePool (NewResourceLocation);
+  }
+
+  if (Json != NULL) {
+    FreePool (Json);
+  }
 
   return Status;
 }
@@ -2123,12 +2229,18 @@ ProvisioningMemoryResources (
 
   Status = RedfishFeatureGetUnifiedArrayTypeConfigureLang (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, REDPATH_ARRAY_PATTERN, &UnifiedConfigureLangList);
   if (EFI_ERROR (Status) || (UnifiedConfigureLangList.Count == 0)) {
-    DEBUG ((DEBUG_ERROR, "%a, No HII question found with redpath: %s: %r\n", __FUNCTION__, REDPATH_ARRAY_PATTERN, Status));
+    DEBUG ((DEBUG_ERROR, "%a, No HII question found with configure language: %s: %r\n", __FUNCTION__, REDPATH_ARRAY_PATTERN, Status));
     return EFI_NOT_FOUND;
   }
 
+  //
+  // Set the configuration language in the RESOURCE_INFORMATION_EXCHANGE.
+  // This information is sent back to the parent resource (e.g. the collection driver).
+  //
+  EdkIIRedfishResourceSetConfigureLang (&UnifiedConfigureLangList);
+
   for (Index = 0; Index < UnifiedConfigureLangList.Count; Index++) {
-    DEBUG ((DEBUG_INFO, "[%d] create memory resource from: %s\n", UnifiedConfigureLangList.List[Index].Index, UnifiedConfigureLangList.List[Index].ConfigureLang));
+    DEBUG ((DEBUG_INFO, "[%d] create Memory resource from: %s\n", UnifiedConfigureLangList.List[Index].Index, UnifiedConfigureLangList.List[Index].ConfigureLang));
     ProvisioningMemoryResource (Private, UnifiedConfigureLangList.List[Index].Index, UnifiedConfigureLangList.List[Index].ConfigureLang);
     FreePool (UnifiedConfigureLangList.List[Index].ConfigureLang);
   }
@@ -2141,14 +2253,70 @@ ProvisioningMemoryExistResource (
   IN  REDFISH_RESOURCE_COMMON_PRIVATE  *Private
   )
 {
+  EFI_STATUS  Status;
+  EFI_STRING  ConfigureLang;
+  CHAR8       *EtagStr;
+  CHAR8       *Json;
+
   if (Private == NULL) {
     return EFI_INVALID_PARAMETER;
   }
 
-  Private->Json = JsonDumpString (RedfishJsonInPayload (Private->Payload), EDKII_JSON_COMPACT);
-  ASSERT (Private->Json != NULL);
+  EtagStr       = NULL;
+  Json          = NULL;
+  ConfigureLang = NULL;
+
+  ConfigureLang = RedfishGetConfigLanguage (Private->Uri);
+  if (ConfigureLang == NULL) {
+    return EFI_NOT_FOUND;
+  }
+
+  Status = ProvisioningMemoryProperties (
+             Private->JsonStructProtocol,
+             MemoryEmptyJson,
+             NULL,
+             ConfigureLang,
+             TRUE,
+             &Json
+             );
+  if (EFI_ERROR (Status)) {
+    if (Status == EFI_NOT_FOUND) {
+      DEBUG ((REDFISH_DEBUG_TRACE, "%a, provisioning existing resource for %s ignored. Nothing changed\n", __FUNCTION__, ConfigureLang));
+    } else {
+      DEBUG ((DEBUG_ERROR, "%a, provisioning existing resource for %s failed: %r\n", __FUNCTION__, ConfigureLang, Status));
+    }
+
+    goto ON_RELEASE;
+  }
+
+  DEBUG ((REDFISH_DEBUG_TRACE, "%a, provisioning existing resource for %s\n", __FUNCTION__, ConfigureLang));
+  //
+  // PUT back to instance
+  //
+  Status = CreatePayloadToPatchResource (Private->RedfishService, Private->Payload, Json, &EtagStr);
+  if (EFI_ERROR (Status)) {
+    DEBUG ((DEBUG_ERROR, "%a, patch resource for %s failed: %r\n", __FUNCTION__, ConfigureLang, Status));
+  }
 
-  return RedfishUpdateResourceCommon (Private, Private->Json);
+  //
+  // Handle Etag
+  //
+  if (EtagStr != NULL) {
+    SetEtagWithUri (EtagStr, Private->Uri);
+    FreePool (EtagStr);
+  }
+
+ON_RELEASE:
+
+  if (Json != NULL) {
+    FreePool (Json);
+  }
+
+  if (ConfigureLang != NULL) {
+    FreePool (ConfigureLang);
+  }
+
+  return Status;
 }
 
 /**
@@ -2196,9 +2364,6 @@ RedfishCheckResourceCommon (
   EFI_STRING  *ConfigureLangList;
   UINTN       Count;
   EFI_STRING  Property;
-  CHAR8       *PropertyAscii;
-  UINTN       BuffSize;
-  CHAR8       *Match;
 
   if ((Private == NULL) || IS_EMPTY_STRING (Json)) {
     return EFI_INVALID_PARAMETER;
@@ -2216,27 +2381,16 @@ RedfishCheckResourceCommon (
 
   Status = EFI_SUCCESS;
   for (Index = 0; Index < Count; Index++) {
-    Property = GetPropertyFromConfigureLang (ConfigureLangList[Index]);
+    Property = GetPropertyFromConfigureLang (Private->Uri, ConfigureLangList[Index]);
     if (Property == NULL) {
       continue;
     }
 
-    DEBUG ((DEBUG_INFO, "[%d] check resource from: %s\n", Index, Property));
-
-    BuffSize      = StrLen (Property) + 1;
-    PropertyAscii = AllocatePool (BuffSize);
-    UnicodeStrToAsciiStrS (Property, PropertyAscii, BuffSize);
-
-    //
-    // check to see if it is partial match.
-    //
-    Match = AsciiStrStr (Json, PropertyAscii);
-    if ((Match == NULL) || (AsciiStrnCmp (Match, PropertyAscii, AsciiStrLen (PropertyAscii)) != 0)) {
+    DEBUG ((DEBUG_INFO, "%a, [%d] check attribute for: %s\n", __FUNCTION__, Index, Property));
+    if (!MatchPropertyWithJsonContext (Property, Json)) {
+      DEBUG ((DEBUG_INFO, "%a, property is missing: %s\n", __FUNCTION__, Property));
       Status = EFI_NOT_FOUND;
-      DEBUG ((DEBUG_ERROR, "%a, property %a is missing\n", __FUNCTION__, PropertyAscii));
     }
-
-    FreePool (PropertyAscii);
   }
 
   FreePool (ConfigureLangList);
@@ -2257,40 +2411,34 @@ RedfishCheckResourceCommon (
 EFI_STATUS
 RedfishUpdateResourceCommon (
   IN     REDFISH_RESOURCE_COMMON_PRIVATE  *Private,
-  IN     CHAR8                            *Json
+  IN     CHAR8                            *InputJson
   )
 {
   EFI_STATUS  Status;
-  CHAR8       *MemoryJson;
-  CHAR8       *ArrayKey;
+  CHAR8       *Json;
   EFI_STRING  ConfigureLang;
   CHAR8       *EtagStr;
 
-  if ((Private == NULL) || IS_EMPTY_STRING (Json)) {
+  if ((Private == NULL) || IS_EMPTY_STRING (InputJson)) {
     return EFI_INVALID_PARAMETER;
   }
 
-  MemoryJson    = NULL;
+  EtagStr       = NULL;
+  Json          = NULL;
   ConfigureLang = NULL;
-  ArrayKey      = NULL;
 
-  Status = GetArraykeyFromUri (Private->Uri, &ArrayKey);
-  if (EFI_ERROR (Status)) {
-    return Status;
-  }
-
-  ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, NULL, ArrayKey);
+  ConfigureLang = RedfishGetConfigLanguage (Private->Uri);
   if (ConfigureLang == NULL) {
     return EFI_NOT_FOUND;
   }
 
   Status = ProvisioningMemoryProperties (
              Private->JsonStructProtocol,
-             Json,
+             InputJson,
              NULL,
              ConfigureLang,
              FALSE,
-             &MemoryJson
+             &Json
              );
   if (EFI_ERROR (Status)) {
     if (Status == EFI_NOT_FOUND) {
@@ -2306,9 +2454,9 @@ RedfishUpdateResourceCommon (
   //
   // PUT back to instance
   //
-  Status = CreatePayloadToPatchResource (Private->RedfishService, Private->Payload, MemoryJson, &EtagStr);
+  Status = CreatePayloadToPatchResource (Private->RedfishService, Private->Payload, Json, &EtagStr);
   if (EFI_ERROR (Status)) {
-    DEBUG ((DEBUG_ERROR, "%a, post memory resource for %s failed: %r\n", __FUNCTION__, ConfigureLang, Status));
+    DEBUG ((DEBUG_ERROR, "%a, patch resource for %s failed: %r\n", __FUNCTION__, ConfigureLang, Status));
   }
 
   //
@@ -2321,17 +2469,83 @@ RedfishUpdateResourceCommon (
 
 ON_RELEASE:
 
-  if (MemoryJson != NULL) {
-    FreePool (MemoryJson);
+  if (Json != NULL) {
+    FreePool (Json);
   }
 
   if (ConfigureLang != NULL) {
     FreePool (ConfigureLang);
   }
 
-  if (ArrayKey != NULL) {
-    FreePool (ArrayKey);
+  return Status;
+}
+
+/**
+  Identify resource from given URI.
+
+  @param[in]   This                Pointer to REDFISH_RESOURCE_COMMON_PRIVATE instance.
+  @param[in]   Json                The JSON to consume.
+
+  @retval EFI_SUCCESS              Value is returned successfully.
+  @retval Others                   Some error happened.
+
+**/
+EFI_STATUS
+RedfishIdentifyResourceCommon (
+  IN     REDFISH_RESOURCE_COMMON_PRIVATE  *Private,
+  IN     CHAR8                            *Json
+  )
+{
+  BOOLEAN                                      Supported;
+  EFI_STATUS                                   Status;
+  EFI_STRING                                   EndOfChar;
+  REDFISH_FEATURE_ARRAY_TYPE_CONFIG_LANG_LIST  ConfigLangList;
+
+  Supported = RedfishIdentifyResource (Private->Uri, Private->Json);
+  if (Supported) {
+    Status = RedfishFeatureGetUnifiedArrayTypeConfigureLang (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, REDPATH_ARRAY_PATTERN, &ConfigLangList);
+    if (EFI_ERROR (Status)) {
+      DEBUG ((DEBUG_ERROR, "%a, BiosConfigToRedfishGetConfigureLangRegex failed: %r\n", __FUNCTION__, Status));
+      return Status;
+    }
+
+    if (ConfigLangList.Count == 0) {
+      return EFI_SUCCESS;
+    }
+
+    EndOfChar = StrStr (ConfigLangList.List[0].ConfigureLang, L"}");
+    if (EndOfChar == NULL) {
+      ASSERT (FALSE);
+      return EFI_DEVICE_ERROR;
+    }
+
+    // EndOfChar = StrStr (ConfigLangList.List[0].ConfigureLang, L"}");
+    Status = IsRedpathArray (ConfigLangList.List[0].ConfigureLang, NULL, &EndOfChar);
+    if (EFI_ERROR (Status) && (Status != EFI_NOT_FOUND)) {
+      ASSERT (FALSE);
+      return EFI_DEVICE_ERROR;
+    }
+
+    if (Status != EFI_SUCCESS) {
+      //
+      // This is not the collection redpath.
+      //
+      GetRedpathNodeByIndex (ConfigLangList.List[0].ConfigureLang, 0, &EndOfChar);
+    }
+
+    *(++EndOfChar) = '\0';
+    //
+    // Keep URI and ConfigLang mapping
+    //
+    RedfisSetRedfishUri (ConfigLangList.List[0].ConfigureLang, Private->Uri);
+    //
+    // Set the configuration language in the RESOURCE_INFORMATION_EXCHANGE.
+    // This information is sent back to the parent resource (e.g. the collection driver).
+    //
+    EdkIIRedfishResourceSetConfigureLang (&ConfigLangList);
+    DestroyConfiglanguageList (&ConfigLangList);
+    return EFI_SUCCESS;
   }
 
-  return Status;
+  return EFI_UNSUPPORTED;
 }
diff --git a/RedfishClientPkg/Features/Memory/V1_7_1/Dxe/RedfishMemoryDxe.c b/RedfishClientPkg/Features/Memory/V1_7_1/Dxe/MemoryDxe.c
similarity index 75%
rename from RedfishClientPkg/Features/Memory/V1_7_1/Dxe/RedfishMemoryDxe.c
rename to RedfishClientPkg/Features/Memory/V1_7_1/Dxe/MemoryDxe.c
index 90e3a20f..07058888 100644
--- a/RedfishClientPkg/Features/Memory/V1_7_1/Dxe/RedfishMemoryDxe.c
+++ b/RedfishClientPkg/Features/Memory/V1_7_1/Dxe/MemoryDxe.c
@@ -1,16 +1,18 @@
 /** @file
   Redfish feature driver implementation - Memory
 
-  (C) Copyright 2020-2021 Hewlett Packard Enterprise Development LP<BR>
+  (C) Copyright 2020-2022 Hewlett Packard Enterprise Development LP<BR>
 
   SPDX-License-Identifier: BSD-2-Clause-Patent
 
 **/
 
-#include "../Common/RedfishMemoryCommon.h"
+#include "../Common/MemoryCommon.h"
 
 extern REDFISH_RESOURCE_COMMON_PRIVATE  *mRedfishResourcePrivate;
 
+EFI_HANDLE  medfishResourceConfigProtocolHandle;
+
 /**
   Provising redfish resource by given URI.
 
@@ -26,7 +28,7 @@ extern REDFISH_RESOURCE_COMMON_PRIVATE  *mRedfishResourcePrivate;
 EFI_STATUS
 RedfishResourceProvisioningResource (
   IN     EDKII_REDFISH_RESOURCE_CONFIG_PROTOCOL  *This,
-  IN     CHAR8                                   *Uri,
+  IN     EFI_STRING                              Uri,
   IN     BOOLEAN                                 PostMode
   )
 {
@@ -38,9 +40,7 @@ RedfishResourceProvisioningResource (
     return EFI_INVALID_PARAMETER;
   }
 
-  if (!PostMode) {
-    return EFI_UNSUPPORTED;
-  }
+  DEBUG ((DEBUG_INFO, "%a, provisioning in %s mode\n", __FUNCTION__, (PostMode ? L"POST" : L"PATCH")));
 
   Private = REDFISH_RESOURCE_COMMON_PRIVATE_DATA_FROM_RESOURCE_PROTOCOL (This);
 
@@ -48,9 +48,9 @@ RedfishResourceProvisioningResource (
     return EFI_NOT_READY;
   }
 
-  Status = GetResourceByPath (Private->RedfishService, Uri, &Response);
+  Status = GetResourceByUri (Private->RedfishService, Uri, &Response);
   if (EFI_ERROR (Status)) {
-    DEBUG ((DEBUG_ERROR, "%a, get resource from: %a failed\n", __FUNCTION__, Uri));
+    DEBUG ((DEBUG_ERROR, "%a, get resource from: %s failed\n", __FUNCTION__, Uri));
     return Status;
   }
 
@@ -58,7 +58,7 @@ RedfishResourceProvisioningResource (
   Private->Payload = Response.Payload;
   ASSERT (Private->Payload != NULL);
 
-  Status = RedfishProvisioningResourceCommon (Private, FALSE);
+  Status = RedfishProvisioningResourceCommon (Private, !PostMode);
 
   //
   // Release resource
@@ -89,12 +89,13 @@ RedfishResourceProvisioningResource (
 EFI_STATUS
 RedfishResourceConsumeResource (
   IN     EDKII_REDFISH_RESOURCE_CONFIG_PROTOCOL  *This,
-  IN     CHAR8                                   *Uri
+  IN     EFI_STRING                              Uri
   )
 {
   REDFISH_RESOURCE_COMMON_PRIVATE  *Private;
   EFI_STATUS                       Status;
   REDFISH_RESPONSE                 Response;
+  CHAR8                            *Etag;
 
   if ((This == NULL) || IS_EMPTY_STRING (Uri)) {
     return EFI_INVALID_PARAMETER;
@@ -106,9 +107,9 @@ RedfishResourceConsumeResource (
     return EFI_NOT_READY;
   }
 
-  Status = GetResourceByPath (Private->RedfishService, Uri, &Response);
+  Status = GetResourceByUri (Private->RedfishService, Uri, &Response);
   if (EFI_ERROR (Status)) {
-    DEBUG ((DEBUG_ERROR, "%a, get resource from: %a failed\n", __FUNCTION__, Uri));
+    DEBUG ((DEBUG_ERROR, "%a, get resource from: %s failed\n", __FUNCTION__, Uri));
     return Status;
   }
 
@@ -119,9 +120,27 @@ RedfishResourceConsumeResource (
   Private->Json = JsonDumpString (RedfishJsonInPayload (Private->Payload), EDKII_JSON_COMPACT);
   ASSERT (Private->Json != NULL);
 
-  Status = RedfishConsumeResourceCommon (Private, Private->Json);
+  //
+  // Find etag in HTTP response header
+  //
+  Etag   = NULL;
+  Status = GetEtagAndLocation (&Response, &Etag, NULL);
   if (EFI_ERROR (Status)) {
-    DEBUG ((DEBUG_ERROR, "%a, failed to consume resource from: %a %r\n", __FUNCTION__, Uri, Status));
+    DEBUG ((DEBUG_ERROR, "%a, failed to get ETag from HTTP header\n", __FUNCTION__));
+  }
+
+  Status = RedfishConsumeResourceCommon (Private, Private->Json, Etag);
+  if (EFI_ERROR (Status)) {
+    if (Status != EFI_ALREADY_STARTED) {
+      DEBUG ((DEBUG_ERROR, "%a, failed to consume resource from: %s: %r\n", __FUNCTION__, Uri, Status));
+    }
+  } else {
+    //
+    // Keep etag after consuming pending settings.
+    //
+    if (Etag != NULL) {
+      SetEtagWithUri (Etag, Private->Uri);
+    }
   }
 
   //
@@ -193,7 +212,7 @@ RedfishResourceGetInfo (
 EFI_STATUS
 RedfishResourceUpdate (
   IN     EDKII_REDFISH_RESOURCE_CONFIG_PROTOCOL  *This,
-  IN     CHAR8                                   *Uri
+  IN     EFI_STRING                              Uri
   )
 {
   REDFISH_RESOURCE_COMMON_PRIVATE  *Private;
@@ -210,9 +229,9 @@ RedfishResourceUpdate (
     return EFI_NOT_READY;
   }
 
-  Status = GetResourceByPath (Private->RedfishService, Uri, &Response);
+  Status = GetResourceByUri (Private->RedfishService, Uri, &Response);
   if (EFI_ERROR (Status)) {
-    DEBUG ((DEBUG_ERROR, "%a, get resource from: %a failed\n", __FUNCTION__, Uri));
+    DEBUG ((DEBUG_ERROR, "%a, get resource from: %s failed\n", __FUNCTION__, Uri));
     return Status;
   }
 
@@ -225,7 +244,7 @@ RedfishResourceUpdate (
 
   Status = RedfishUpdateResourceCommon (Private, Private->Json);
   if (EFI_ERROR (Status)) {
-    DEBUG ((DEBUG_ERROR, "%a, failed to update resource from: %a %r\n", __FUNCTION__, Uri, Status));
+    DEBUG ((DEBUG_ERROR, "%a, failed to update resource from: %s: %r\n", __FUNCTION__, Uri, Status));
   }
 
   //
@@ -262,7 +281,7 @@ RedfishResourceUpdate (
 EFI_STATUS
 RedfishResourceCheck (
   IN     EDKII_REDFISH_RESOURCE_CONFIG_PROTOCOL  *This,
-  IN     CHAR8                                   *Uri
+  IN     EFI_STRING                              Uri
   )
 {
   REDFISH_RESOURCE_COMMON_PRIVATE  *Private;
@@ -279,9 +298,9 @@ RedfishResourceCheck (
     return EFI_NOT_READY;
   }
 
-  Status = GetResourceByPath (Private->RedfishService, Uri, &Response);
+  Status = GetResourceByUri (Private->RedfishService, Uri, &Response);
   if (EFI_ERROR (Status)) {
-    DEBUG ((DEBUG_ERROR, "%a, get resource from: %a failed\n", __FUNCTION__, Uri));
+    DEBUG ((DEBUG_ERROR, "%a, get resource from: %s failed\n", __FUNCTION__, Uri));
     return Status;
   }
 
@@ -294,7 +313,77 @@ RedfishResourceCheck (
 
   Status = RedfishCheckResourceCommon (Private, Private->Json);
   if (EFI_ERROR (Status)) {
-    DEBUG ((DEBUG_ERROR, "%a, failed to check resource from: %a %r\n", __FUNCTION__, Uri, Status));
+    DEBUG ((DEBUG_ERROR, "%a, failed to check resource from: %s: %r\n", __FUNCTION__, Uri, Status));
+  }
+
+  //
+  // Release resource
+  //
+  if (Private->Payload != NULL) {
+    RedfishFreeResponse (
+      Response.StatusCode,
+      Response.HeaderCount,
+      Response.Headers,
+      Response.Payload
+      );
+    Private->Payload = NULL;
+  }
+
+  if (Private->Json != NULL) {
+    FreePool (Private->Json);
+    Private->Json = NULL;
+  }
+
+  return Status;
+}
+
+/**
+  Identify resource on given URI.
+
+  @param[in]   This                Pointer to EDKII_REDFISH_RESOURCE_CONFIG_PROTOCOL instance.
+  @param[in]   Uri                 The target URI to consume.
+
+  @retval EFI_SUCCESS              This is target resource which we want to handle.
+  @retval EFI_UNSUPPORTED          This is not the target resource.
+  @retval Others                   Some error happened.
+
+**/
+EFI_STATUS
+RedfishResourceIdentify (
+  IN     EDKII_REDFISH_RESOURCE_CONFIG_PROTOCOL  *This,
+  IN     EFI_STRING                              Uri
+  )
+{
+  REDFISH_RESOURCE_COMMON_PRIVATE  *Private;
+  EFI_STATUS                       Status;
+  REDFISH_RESPONSE                 Response;
+
+  if ((This == NULL) || IS_EMPTY_STRING (Uri)) {
+    return EFI_INVALID_PARAMETER;
+  }
+
+  Private = REDFISH_RESOURCE_COMMON_PRIVATE_DATA_FROM_RESOURCE_PROTOCOL (This);
+
+  if (Private->RedfishService == NULL) {
+    return EFI_NOT_READY;
+  }
+
+  Status = GetResourceByUri (Private->RedfishService, Uri, &Response);
+  if (EFI_ERROR (Status)) {
+    DEBUG ((DEBUG_ERROR, "%a, get resource from: %s failed\n", __FUNCTION__, Uri));
+    return Status;
+  }
+
+  Private->Uri     = Uri;
+  Private->Payload = Response.Payload;
+  ASSERT (Private->Payload != NULL);
+
+  Private->Json = JsonDumpString (RedfishJsonInPayload (Private->Payload), EDKII_JSON_COMPACT);
+  ASSERT (Private->Json != NULL);
+
+  Status = RedfishIdentifyResourceCommon (Private, Private->Json);
+  if (EFI_ERROR (Status)) {
+    DEBUG ((DEBUG_ERROR, "%a, identify %s failed: %r\n", __FUNCTION__, Uri, Status));
   }
 
   //
@@ -323,6 +412,7 @@ EDKII_REDFISH_RESOURCE_CONFIG_PROTOCOL  mRedfishResourceConfig = {
   RedfishResourceConsumeResource,
   RedfishResourceUpdate,
   RedfishResourceCheck,
+  RedfishResourceIdentify,
   RedfishResourceGetInfo
 };
 
@@ -523,6 +613,8 @@ RedfishResourceEntryPoint (
     return EFI_ALREADY_STARTED;
   }
 
+  medfishResourceConfigProtocolHandle = ImageHandle;
+
   mRedfishResourcePrivate = AllocateZeroPool (sizeof (REDFISH_RESOURCE_COMMON_PRIVATE));
   CopyMem (&mRedfishResourcePrivate->ConfigHandler, &mRedfishConfigHandler, sizeof (EDKII_REDFISH_CONFIG_HANDLER_PROTOCOL));
   CopyMem (&mRedfishResourcePrivate->RedfishResourceConfig, &mRedfishResourceConfig, sizeof (EDKII_REDFISH_RESOURCE_CONFIG_PROTOCOL));
diff --git a/RedfishClientPkg/RedfishClient.fdf.inc b/RedfishClientPkg/RedfishClient.fdf.inc
index 3619d2a2..5c4b9670 100644
--- a/RedfishClientPkg/RedfishClient.fdf.inc
+++ b/RedfishClientPkg/RedfishClient.fdf.inc
@@ -14,7 +14,7 @@
   INF RedfishClientPkg/RedfishFeatureCoreDxe/RedfishFeatureCoreDxe.inf
   INF RedfishClientPkg/RedfishETagDxe/RedfishETagDxe.inf
   INF RedfishClientPkg/RedfishConfigLangMapDxe/RedfishConfigLangMapDxe.inf
-  INF RedfishClientPkg/Features/Memory/V1_7_1/Dxe/RedfishMemoryDxe.inf
+  INF RedfishClientPkg/Features/Memory/V1_7_1/Dxe/MemoryDxe.inf
   INF RedfishClientPkg/Features/MemoryCollectionDxe/MemoryCollectionDxe.inf
 
   !include RedfishClientPkg/RedfishJsonStructureDxe.fdf.inc
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [edk2-redfish-client][PATCH 2/5] RedfishClientPkg: Rename Memory feature driver
  2023-05-10 13:04 [edk2-redfish-client][PATCH 2/5] RedfishClientPkg: Rename Memory feature driver Nickle Wang
@ 2023-05-10 14:06 ` Chang, Abner
  0 siblings, 0 replies; 2+ messages in thread
From: Chang, Abner @ 2023-05-10 14:06 UTC (permalink / raw)
  To: Nickle Wang, devel@edk2.groups.io; +Cc: Igor Kulchytskyy

[AMD Official Use Only - General]

Reviewed-by: <abner.chang@amd.com>

> -----Original Message-----
> From: Nickle Wang <nicklew@nvidia.com>
> Sent: Wednesday, May 10, 2023 9:04 PM
> To: devel@edk2.groups.io
> Cc: Chang, Abner <Abner.Chang@amd.com>; Igor Kulchytskyy
> <igork@ami.com>
> Subject: [edk2-redfish-client][PATCH 2/5] RedfishClientPkg: Rename
> Memory feature driver
> 
> Caution: This message originated from an External Source. Use proper
> caution when opening attachments, clicking links, or responding.
> 
> 
> Rename Memory driver by removing Redfish prefix. Update memory feature
> driver and support all properties under Memory schema. Also support
> "Identify" action in this driver. Corresponding changes are made to
> honor newly introduced library and protocol.
> 
> Signed-off-by: Nickle Wang <nicklew@nvidia.com>
> Cc: Abner Chang <abner.chang@amd.com>
> Cc: Igor Kulchytskyy <igork@ami.com>
> ---
>  .../RedfishClientComponents.dsc.inc           |    2 +-
>  .../{RedfishMemoryDxe.inf => MemoryDxe.inf}   |   21 +-
>  .../{RedfishMemoryCommon.h => MemoryCommon.h} |    2 +-
>  .../Include/RedfishResourceCommon.h           |   38 +-
>  .../{RedfishMemoryCommon.c => MemoryCommon.c} | 1138 ++++++++++-
> ------
>  .../Dxe/{RedfishMemoryDxe.c => MemoryDxe.c}   |  136 +-
>  RedfishClientPkg/RedfishClient.fdf.inc        |    2 +-
>  7 files changed, 838 insertions(+), 501 deletions(-)
>  rename
> RedfishClientPkg/Features/Memory/V1_7_1/Dxe/{RedfishMemoryDxe.inf
> => MemoryDxe.inf} (58%)
>  rename
> RedfishClientPkg/Features/Memory/V1_7_1/Common/{RedfishMemoryCo
> mmon.h => MemoryCommon.h} (89%)
>  rename
> RedfishClientPkg/Features/Memory/V1_7_1/Common/{RedfishMemoryCo
> mmon.c => MemoryCommon.c} (64%)
>  rename
> RedfishClientPkg/Features/Memory/V1_7_1/Dxe/{RedfishMemoryDxe.c =>
> MemoryDxe.c} (75%)
> 
> diff --git a/RedfishClientPkg/RedfishClientComponents.dsc.inc
> b/RedfishClientPkg/RedfishClientComponents.dsc.inc
> index 5f1aff3c..031d8755 100644
> --- a/RedfishClientPkg/RedfishClientComponents.dsc.inc
> +++ b/RedfishClientPkg/RedfishClientComponents.dsc.inc
> @@ -19,7 +19,7 @@
>    #
>    # Below two modules should be pulled in by build tool.
>    #
> -  RedfishClientPkg/Features/Memory/V1_7_1/Dxe/RedfishMemoryDxe.inf
> +  RedfishClientPkg/Features/Memory/V1_7_1/Dxe/MemoryDxe.inf
> 
> RedfishClientPkg/Features/MemoryCollectionDxe/MemoryCollectionDxe.inf
> 
>    !include RedfishClientPkg/RedfishJsonStructureDxe.dsc.inc
> diff --git
> a/RedfishClientPkg/Features/Memory/V1_7_1/Dxe/RedfishMemoryDxe.inf
> b/RedfishClientPkg/Features/Memory/V1_7_1/Dxe/MemoryDxe.inf
> similarity index 58%
> rename from
> RedfishClientPkg/Features/Memory/V1_7_1/Dxe/RedfishMemoryDxe.inf
> rename to
> RedfishClientPkg/Features/Memory/V1_7_1/Dxe/MemoryDxe.inf
> index 3e2ad884..594a8749 100644
> ---
> a/RedfishClientPkg/Features/Memory/V1_7_1/Dxe/RedfishMemoryDxe.inf
> +++ b/RedfishClientPkg/Features/Memory/V1_7_1/Dxe/MemoryDxe.inf
> @@ -1,6 +1,6 @@
>  ## @file
>  #
> -#  (C) Copyright 2020-2021 Hewlett Packard Enterprise Development LP<BR>
> +#  (C) Copyright 2020-2022 Hewlett Packard Enterprise Development
> LP<BR>
>  #
>  #  SPDX-License-Identifier: BSD-2-Clause-Patent
>  #
> @@ -9,8 +9,8 @@
> 
>  [Defines]
>    INF_VERSION               = 0x00010005
> -  BASE_NAME                 = RedfishMemoryDxe
> -  FILE_GUID                 = f5594d13-ca13-485f-ac3a-80c5074dd8c0
> +  BASE_NAME                 = MemoryDxe
> +  FILE_GUID                 = 69a1a994-ec14-4dd0-ba0d-d1b51b7bab51
>    MODULE_TYPE               = DXE_DRIVER
>    VERSION_STRING            = 1.0
>    ENTRY_POINT               = RedfishResourceEntryPoint
> @@ -23,22 +23,25 @@
>    RedfishClientPkg/RedfishClientPkg.dec
> 
>  [Sources]
> -  ../Common/RedfishMemoryCommon.h
> -  ../Common/RedfishMemoryCommon.c
> -  RedfishMemoryDxe.c
> +  ../Common/MemoryCommon.h
> +  ../Common/MemoryCommon.c
> +  MemoryDxe.c
> 
>  [LibraryClasses]
>    BaseMemoryLib
>    DebugLib
> +  EdkIIRedfishResourceConfigLib
>    RedfishLib
>    RedfishFeatureUtilityLib
> +  RedfishResourceIdentifyLib
>    UefiLib
>    UefiDriverEntryPoint
> 
> +
>  [Protocols]
> -  gEdkIIRedfishConfigHandlerProtocolGuid    ## PRODUCED
> -  gEfiRestJsonStructureProtocolGuid         ## CONSUMED
> -  gEdkIIRedfishResourceConfigProtocolGuid   ## PRODUCED
> +  gEdkIIRedfishConfigHandlerProtocolGuid          ## PRODUCED
> +  gEfiRestJsonStructureProtocolGuid               ## CONSUMED
> +  gEdkIIRedfishResourceConfigProtocolGuid         ## PRODUCED
> 
>  [Pcd]
>    gEfiRedfishClientPkgTokenSpaceGuid.PcdMaxRedfishSchemaStringSize
> diff --git
> a/RedfishClientPkg/Features/Memory/V1_7_1/Common/RedfishMemoryCo
> mmon.h
> b/RedfishClientPkg/Features/Memory/V1_7_1/Common/MemoryCommon.
> h
> similarity index 89%
> rename from
> RedfishClientPkg/Features/Memory/V1_7_1/Common/RedfishMemoryCom
> mon.h
> rename to
> RedfishClientPkg/Features/Memory/V1_7_1/Common/MemoryCommon.h
> index f6fe5111..c857868e 100644
> ---
> a/RedfishClientPkg/Features/Memory/V1_7_1/Common/RedfishMemoryCo
> mmon.h
> +++
> b/RedfishClientPkg/Features/Memory/V1_7_1/Common/MemoryCommon.
> h
> @@ -20,7 +20,7 @@
>  #define RESOURCE_SCHEMA_MINOR    "7"
>  #define RESOURCE_SCHEMA_ERRATA   "1"
>  #define RESOURCE_SCHEMA_VERSION  "v1_7_1"
> -#define REDPATH_ARRAY_PATTERN    L"/Memory/{.*}/"
> +#define REDPATH_ARRAY_PATTERN    L"/Memory/\\{.*\\}/"
>  #define REDPATH_ARRAY_PREFIX     L"/Memory/"
>  #define RESOURCE_SCHEMA_FULL     "x-uefi-redfish-Memory.v1_7_1"
> 
> diff --git a/RedfishClientPkg/Include/RedfishResourceCommon.h
> b/RedfishClientPkg/Include/RedfishResourceCommon.h
> index 95e8004d..c270f92b 100644
> --- a/RedfishClientPkg/Include/RedfishResourceCommon.h
> +++ b/RedfishClientPkg/Include/RedfishResourceCommon.h
> @@ -1,7 +1,7 @@
>  /** @file
>    Redfish feature driver common header file.
> 
> -  (C) Copyright 2020-2021 Hewlett Packard Enterprise Development LP<BR>
> +  (C) Copyright 2020-2022 Hewlett Packard Enterprise Development LP<BR>
> 
>    SPDX-License-Identifier: BSD-2-Clause-Patent
> 
> @@ -11,8 +11,8 @@
>  #define EFI_REDFISH_RESOURCE_COMMON_H_
> 
>  #define MAX_RED_PATH_LEN  128
> -#define IS_EMPTY_STRING(a)  ((a) == NULL || (a)[0] == '\0')
> -#define REDFISH_DEBUG_TRACE  DEBUG_INFO
> +
> +#include <RedfishBase.h>
> 
>  //
>  // Libraries
> @@ -23,14 +23,18 @@
>  #include <Library/PrintLib.h>
>  #include <Library/PcdLib.h>
>  #include <Library/RedfishLib.h>
> +#include <Library/RedfishVersionLib.h>
>  #include <Library/RedfishFeatureUtilityLib.h>
>  #include <Library/RedfishPlatformConfigLib.h>
>  #include <Library/UefiLib.h>
>  #include <Library/UefiBootServicesTableLib.h>
> +#include <Library/RedfishResourceIdentifyLib.h>
> +#include <Library/EdkIIRedfishResourceConfigLib.h>
> 
>  //
>  // Protocols
>  //
> +#include <Protocol/EdkIIRedfishFeature.h>
>  #include <Protocol/EdkIIRedfishConfigHandler.h>
>  #include <Protocol/EdkIIRedfishResourceConfigProtocol.h>
>  #include <Protocol/RestJsonStructure.h>
> @@ -42,9 +46,15 @@ typedef struct
> _REDFISH_RESOURCE_COMMON_PRIVATE {
>    EDKII_REDFISH_RESOURCE_CONFIG_PROTOCOL    RedfishResourceConfig;
>    EDKII_REDFISH_CONFIG_HANDLER_PROTOCOL     ConfigHandler;
>    EFI_EVENT                                 Event;
> -  CHAR8                                     *Uri;
> +  EFI_STRING                                Uri;
>    CHAR8                                     *Json;
>    REDFISH_PAYLOAD                           Payload;
> +  //
> +  //  Below are used for the external resource.
> +  //
> +  EDKII_REDFISH_FEATURE_PROTOCOL            *FeatureProtocol;
> +  RESOURCE_INFORMATION_EXCHANGE             *InformationExchange;
> +  EFI_STRING                                RedfishVersion;
>  } REDFISH_RESOURCE_COMMON_PRIVATE;
> 
>  #define
> REDFISH_RESOURCE_COMMON_PRIVATE_DATA_FROM_CONFIG_PROTOCO
> L(This) \
> @@ -58,6 +68,7 @@ typedef struct
> _REDFISH_RESOURCE_COMMON_PRIVATE {
> 
>    @param[in]   This                Pointer to
> REDFISH_RESOURCE_COMMON_PRIVATE instance.
>    @param[in]   Json                The JSON to consume.
> +  @param[in]   HeaderEtag          The Etag string returned in HTTP header.
> 
>    @retval EFI_SUCCESS              Value is returned successfully.
>    @retval Others                   Some error happened.
> @@ -66,7 +77,8 @@ typedef struct
> _REDFISH_RESOURCE_COMMON_PRIVATE {
>  EFI_STATUS
>  RedfishConsumeResourceCommon (
>    IN     REDFISH_RESOURCE_COMMON_PRIVATE  *Private,
> -  IN     CHAR8                            *Json
> +  IN     CHAR8                            *Json,
> +  IN     CHAR8                            *HeaderEtag OPTIONAL
>    );
> 
>  /**
> @@ -118,4 +130,20 @@ RedfishUpdateResourceCommon (
>    IN     CHAR8                            *Json
>    );
> 
> +/**
> +  Identify resource from given URI.
> +
> +  @param[in]   This                Pointer to
> REDFISH_RESOURCE_COMMON_PRIVATE instance.
> +  @param[in]   Json                The JSON to consume.
> +
> +  @retval EFI_SUCCESS              Value is returned successfully.
> +  @retval Others                   Some error happened.
> +
> +**/
> +EFI_STATUS
> +RedfishIdentifyResourceCommon (
> +  IN     REDFISH_RESOURCE_COMMON_PRIVATE  *Private,
> +  IN     CHAR8                            *Json
> +  );
> +
>  #endif
> diff --git
> a/RedfishClientPkg/Features/Memory/V1_7_1/Common/RedfishMemoryCo
> mmon.c
> b/RedfishClientPkg/Features/Memory/V1_7_1/Common/MemoryCommon.
> c
> similarity index 64%
> rename from
> RedfishClientPkg/Features/Memory/V1_7_1/Common/RedfishMemoryCom
> mon.c
> rename to
> RedfishClientPkg/Features/Memory/V1_7_1/Common/MemoryCommon.c
> index 756c42ba..bf87f185 100644
> ---
> a/RedfishClientPkg/Features/Memory/V1_7_1/Common/RedfishMemoryCo
> mmon.c
> +++
> b/RedfishClientPkg/Features/Memory/V1_7_1/Common/MemoryCommon.
> c
> @@ -1,45 +1,53 @@
>  /** @file
>    Redfish feature driver implementation - common functions
> 
> -  (C) Copyright 2020-2021 Hewlett Packard Enterprise Development LP<BR>
> +  (C) Copyright 2020-2022 Hewlett Packard Enterprise Development LP<BR>
> 
>    SPDX-License-Identifier: BSD-2-Clause-Patent
> 
>  **/
> 
> -#include "RedfishMemoryCommon.h"
> +#include "MemoryCommon.h"
> 
>  CHAR8  MemoryEmptyJson[] = "{\"@odata.id\": \"\", \"@odata.type\":
> \"#Memory.v1_7_1.Memory\", \"Id\": \"\", \"Name\": \"\"}";
> 
>  REDFISH_RESOURCE_COMMON_PRIVATE  *mRedfishResourcePrivate =
> NULL;
> 
> +/**
> +  Consume resource from given URI.
> +
> +  @param[in]   This                Pointer to
> REDFISH_RESOURCE_COMMON_PRIVATE instance.
> +  @param[in]   Json                The JSON to consume.
> +  @param[in]   HeaderEtag          The Etag string returned in HTTP header.
> +
> +  @retval EFI_SUCCESS              Value is returned successfully.
> +  @retval Others                   Some error happened.
> +
> +**/
>  EFI_STATUS
>  RedfishConsumeResourceCommon (
>    IN  REDFISH_RESOURCE_COMMON_PRIVATE  *Private,
> -  IN  CHAR8                            *MemoryJson
> +  IN  CHAR8                            *Json,
> +  IN  CHAR8                            *HeaderEtag OPTIONAL
>    )
>  {
>    EFI_STATUS                    Status;
>    EFI_REDFISH_MEMORY_V1_7_1     *Memory;
>    EFI_REDFISH_MEMORY_V1_7_1_CS  *MemoryCs;
>    EFI_STRING                    ConfigureLang;
> -  CHAR8                         *Arraykey;
> -  CHAR8                         *EtagInDb;
> 
> -  if ((Private == NULL) || IS_EMPTY_STRING (MemoryJson)) {
> +  if ((Private == NULL) || IS_EMPTY_STRING (Json)) {
>      return EFI_INVALID_PARAMETER;
>    }
> 
>    Memory        = NULL;
>    MemoryCs      = NULL;
>    ConfigureLang = NULL;
> -  Arraykey      = NULL;
> -  EtagInDb      = NULL;
> 
>    Status = Private->JsonStructProtocol->ToStructure (
>                                            Private->JsonStructProtocol,
>                                            NULL,
> -                                          MemoryJson,
> +                                          Json,
>                                            (EFI_REST_JSON_STRUCTURE_HEADER **)&Memory
>                                            );
>    if (EFI_ERROR (Status)) {
> @@ -52,24 +60,13 @@ RedfishConsumeResourceCommon (
>    //
>    // Check ETAG to see if we need to consume it
>    //
> -  EtagInDb = GetEtagWithUri (Private->Uri);
> -  if ((EtagInDb != NULL) && (MemoryCs->odata_etag != NULL)) {
> -    if (AsciiStrCmp (EtagInDb, MemoryCs->odata_etag) == 0) {
> -      //
> -      // No change
> -      //
> -      DEBUG ((DEBUG_INFO, "%a, ETAG: [%a] no change, ignore consume
> action\n", __FUNCTION__, EtagInDb));
> -      goto ON_RELEASE;
> -    }
> -  }
> -
> -  //
> -  // Find array key from URI
> -  //
> -  Status = GetArraykeyFromUri (Private->Uri, &Arraykey);
> -  if (EFI_ERROR (Status)) {
> -    ASSERT (FALSE);
> -    return Status;
> +  if (CheckEtag (Private->Uri, HeaderEtag, MemoryCs->odata_etag)) {
> +    //
> +    // No change
> +    //
> +    DEBUG ((DEBUG_INFO, "%a, ETAG: %s has no change, ignore consume
> action\n", __FUNCTION__, Private->Uri));
> +    Status = EFI_ALREADY_STARTED;
> +    goto ON_RELEASE;
>    }
> 
>    //
> @@ -77,9 +74,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->AllocationAlignmentMiB != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "AllocationAlignmentMiB", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "AllocationAlignmentMiB");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >AllocationAlignmentMiB);
>        if (EFI_ERROR (Status)) {
> @@ -88,7 +85,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -97,9 +94,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->AllocationIncrementMiB != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "AllocationIncrementMiB", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "AllocationIncrementMiB");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >AllocationIncrementMiB);
>        if (EFI_ERROR (Status)) {
> @@ -108,18 +105,42 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> +  //
> +  // Handle ALLOWEDSPEEDSMHZ
> +  //
> +  if (MemoryCs->AllowedSpeedsMHz != NULL) {
> +    //
> +    // Find corresponding configure language for collection resource.
> +    //
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "AllowedSpeedsMHz");
> +    if (ConfigureLang != NULL) {
> +      Status = ApplyFeatureSettingsNumericArrayType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs-
> >AllowedSpeedsMHz);
> +      if (EFI_ERROR (Status)) {
> +        DEBUG ((DEBUG_ERROR, "%a, apply setting for %s failed: %r\n",
> __FUNCTION__, ConfigureLang, Status));
> +      }
> +
> +      FreePool (ConfigureLang);
> +    } else {
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
> +    }
> +  }
> +
> +  //
> +  // ASSEMBLY will be handled by feature driver.
> +  //
> +
>    //
>    // Handle BASEMODULETYPE
>    //
>    if (MemoryCs->BaseModuleType != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "BaseModuleType", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "BaseModuleType");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs-
> >BaseModuleType);
>        if (EFI_ERROR (Status)) {
> @@ -128,7 +149,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -137,9 +158,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->BusWidthBits != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "BusWidthBits", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "BusWidthBits");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >BusWidthBits);
>        if (EFI_ERROR (Status)) {
> @@ -148,7 +169,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -157,9 +178,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->CacheSizeMiB != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "CacheSizeMiB", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "CacheSizeMiB");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >CacheSizeMiB);
>        if (EFI_ERROR (Status)) {
> @@ -168,7 +189,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -177,9 +198,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->CapacityMiB != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "CapacityMiB", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "CapacityMiB");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >CapacityMiB);
>        if (EFI_ERROR (Status)) {
> @@ -188,7 +209,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -197,9 +218,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->ConfigurationLocked != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "ConfigurationLocked", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "ConfigurationLocked");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsBooleanType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (BOOLEAN)*MemoryCs-
> >ConfigurationLocked);
>        if (EFI_ERROR (Status)) {
> @@ -208,7 +229,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -217,9 +238,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->DataWidthBits != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "DataWidthBits", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "DataWidthBits");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >DataWidthBits);
>        if (EFI_ERROR (Status)) {
> @@ -228,7 +249,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -237,9 +258,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->DeviceID != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "DeviceID", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "DeviceID");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->DeviceID);
>        if (EFI_ERROR (Status)) {
> @@ -248,7 +269,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -257,9 +278,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->DeviceLocator != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "DeviceLocator", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "DeviceLocator");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->DeviceLocator);
>        if (EFI_ERROR (Status)) {
> @@ -268,7 +289,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -277,9 +298,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->ErrorCorrection != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "ErrorCorrection", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "ErrorCorrection");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->ErrorCorrection);
>        if (EFI_ERROR (Status)) {
> @@ -288,7 +309,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -297,9 +318,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->FirmwareApiVersion != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "FirmwareApiVersion", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "FirmwareApiVersion");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs-
> >FirmwareApiVersion);
>        if (EFI_ERROR (Status)) {
> @@ -308,7 +329,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -317,9 +338,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->FirmwareRevision != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "FirmwareRevision", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "FirmwareRevision");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs-
> >FirmwareRevision);
>        if (EFI_ERROR (Status)) {
> @@ -328,7 +349,27 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
> +    }
> +  }
> +
> +  //
> +  // Handle FUNCTIONCLASSES
> +  //
> +  if (MemoryCs->FunctionClasses != NULL) {
> +    //
> +    // Find corresponding configure language for collection resource.
> +    //
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "FunctionClasses");
> +    if (ConfigureLang != NULL) {
> +      Status = ApplyFeatureSettingsStringArrayType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs-
> >FunctionClasses);
> +      if (EFI_ERROR (Status)) {
> +        DEBUG ((DEBUG_ERROR, "%a, apply setting for %s failed: %r\n",
> __FUNCTION__, ConfigureLang, Status));
> +      }
> +
> +      FreePool (ConfigureLang);
> +    } else {
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -337,9 +378,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->IsRankSpareEnabled != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "IsRankSpareEnabled", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "IsRankSpareEnabled");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsBooleanType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (BOOLEAN)*MemoryCs-
> >IsRankSpareEnabled);
>        if (EFI_ERROR (Status)) {
> @@ -348,7 +389,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -357,9 +398,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->IsSpareDeviceEnabled != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "IsSpareDeviceEnabled", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "IsSpareDeviceEnabled");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsBooleanType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (BOOLEAN)*MemoryCs-
> >IsSpareDeviceEnabled);
>        if (EFI_ERROR (Status)) {
> @@ -368,18 +409,22 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> +  //
> +  // LOCATION is not handled. Defined in
> http://redfish.dmtf.org/schemas/v1/Resource.json
> +  //
> +
>    //
>    // Handle LOGICALSIZEMIB
>    //
>    if (MemoryCs->LogicalSizeMiB != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "LogicalSizeMiB", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "LogicalSizeMiB");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >LogicalSizeMiB);
>        if (EFI_ERROR (Status)) {
> @@ -388,7 +433,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -397,9 +442,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->Manufacturer != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "Manufacturer", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "Manufacturer");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->Manufacturer);
>        if (EFI_ERROR (Status)) {
> @@ -408,7 +453,27 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
> +    }
> +  }
> +
> +  //
> +  // Handle MAXTDPMILLIWATTS
> +  //
> +  if (MemoryCs->MaxTDPMilliWatts != NULL) {
> +    //
> +    // Find corresponding configure language for collection resource.
> +    //
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "MaxTDPMilliWatts");
> +    if (ConfigureLang != NULL) {
> +      Status = ApplyFeatureSettingsNumericArrayType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs-
> >MaxTDPMilliWatts);
> +      if (EFI_ERROR (Status)) {
> +        DEBUG ((DEBUG_ERROR, "%a, apply setting for %s failed: %r\n",
> __FUNCTION__, ConfigureLang, Status));
> +      }
> +
> +      FreePool (ConfigureLang);
> +    } else {
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -417,9 +482,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->MemoryDeviceType != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "MemoryDeviceType", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "MemoryDeviceType");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs-
> >MemoryDeviceType);
>        if (EFI_ERROR (Status)) {
> @@ -428,7 +493,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -445,9 +510,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->MemoryLocation->Channel != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "MemoryLocation/Channel", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "MemoryLocation/Channel");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >MemoryLocation->Channel);
>        if (EFI_ERROR (Status)) {
> @@ -456,7 +521,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -465,9 +530,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->MemoryLocation->MemoryController != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "MemoryLocation/MemoryController",
> Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "MemoryLocation/MemoryController");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >MemoryLocation->MemoryController);
>        if (EFI_ERROR (Status)) {
> @@ -476,7 +541,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -485,9 +550,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->MemoryLocation->Slot != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "MemoryLocation/Slot", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "MemoryLocation/Slot");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >MemoryLocation->Slot);
>        if (EFI_ERROR (Status)) {
> @@ -496,7 +561,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -505,9 +570,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->MemoryLocation->Socket != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "MemoryLocation/Socket", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "MemoryLocation/Socket");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >MemoryLocation->Socket);
>        if (EFI_ERROR (Status)) {
> @@ -516,18 +581,26 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> +  //
> +  // Handle MEMORYMEDIA
> +  //
> +  //
> +  // ****** Warning ******
> +  // Unsupported array type:
> +  //
> +
>    //
>    // Handle MEMORYSUBSYSTEMCONTROLLERMANUFACTURERID
>    //
>    if (MemoryCs->MemorySubsystemControllerManufacturerID != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION,
> "MemorySubsystemControllerManufacturerID", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "MemorySubsystemControllerManufacturerID");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs-
> >MemorySubsystemControllerManufacturerID);
>        if (EFI_ERROR (Status)) {
> @@ -536,7 +609,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -545,9 +618,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->MemorySubsystemControllerProductID != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "MemorySubsystemControllerProductID",
> Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "MemorySubsystemControllerProductID");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs-
> >MemorySubsystemControllerProductID);
>        if (EFI_ERROR (Status)) {
> @@ -556,7 +629,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -565,9 +638,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->MemoryType != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "MemoryType", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "MemoryType");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->MemoryType);
>        if (EFI_ERROR (Status)) {
> @@ -576,18 +649,22 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> +  //
> +  // METRICS will be handled by feature driver.
> +  //
> +
>    //
>    // Handle MODULEMANUFACTURERID
>    //
>    if (MemoryCs->ModuleManufacturerID != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "ModuleManufacturerID", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "ModuleManufacturerID");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs-
> >ModuleManufacturerID);
>        if (EFI_ERROR (Status)) {
> @@ -596,7 +673,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -605,9 +682,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->ModuleProductID != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "ModuleProductID", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "ModuleProductID");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs-
> >ModuleProductID);
>        if (EFI_ERROR (Status)) {
> @@ -616,7 +693,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -625,9 +702,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->NonVolatileSizeMiB != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "NonVolatileSizeMiB", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "NonVolatileSizeMiB");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >NonVolatileSizeMiB);
>        if (EFI_ERROR (Status)) {
> @@ -636,18 +713,26 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> +  //
> +  // Handle OPERATINGMEMORYMODES
> +  //
> +  //
> +  // ****** Warning ******
> +  // Unsupported array type:
> +  //
> +
>    //
>    // Handle OPERATINGSPEEDMHZ
>    //
>    if (MemoryCs->OperatingSpeedMhz != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "OperatingSpeedMhz", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "OperatingSpeedMhz");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >OperatingSpeedMhz);
>        if (EFI_ERROR (Status)) {
> @@ -656,7 +741,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -665,9 +750,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->PartNumber != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "PartNumber", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "PartNumber");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->PartNumber);
>        if (EFI_ERROR (Status)) {
> @@ -676,7 +761,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -685,9 +770,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->PersistentRegionNumberLimit != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "PersistentRegionNumberLimit", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "PersistentRegionNumberLimit");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >PersistentRegionNumberLimit);
>        if (EFI_ERROR (Status)) {
> @@ -696,7 +781,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -705,9 +790,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->PersistentRegionSizeLimitMiB != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "PersistentRegionSizeLimitMiB", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "PersistentRegionSizeLimitMiB");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >PersistentRegionSizeLimitMiB);
>        if (EFI_ERROR (Status)) {
> @@ -716,7 +801,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -725,9 +810,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->PersistentRegionSizeMaxMiB != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "PersistentRegionSizeMaxMiB", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "PersistentRegionSizeMaxMiB");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >PersistentRegionSizeMaxMiB);
>        if (EFI_ERROR (Status)) {
> @@ -736,7 +821,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -753,9 +838,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->PowerManagementPolicy-
> >AveragePowerBudgetMilliWatts != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION,
> "PowerManagementPolicy/AveragePowerBudgetMilliWatts", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "PowerManagementPolicy/AveragePowerBudgetMilliWatts");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >PowerManagementPolicy->AveragePowerBudgetMilliWatts);
>        if (EFI_ERROR (Status)) {
> @@ -764,7 +849,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -773,9 +858,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->PowerManagementPolicy->MaxTDPMilliWatts != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION,
> "PowerManagementPolicy/MaxTDPMilliWatts", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "PowerManagementPolicy/MaxTDPMilliWatts");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >PowerManagementPolicy->MaxTDPMilliWatts);
>        if (EFI_ERROR (Status)) {
> @@ -784,7 +869,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -793,9 +878,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->PowerManagementPolicy->PeakPowerBudgetMilliWatts !=
> NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION,
> "PowerManagementPolicy/PeakPowerBudgetMilliWatts", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "PowerManagementPolicy/PeakPowerBudgetMilliWatts");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >PowerManagementPolicy->PeakPowerBudgetMilliWatts);
>        if (EFI_ERROR (Status)) {
> @@ -804,7 +889,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -813,9 +898,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->PowerManagementPolicy->PolicyEnabled != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "PowerManagementPolicy/PolicyEnabled",
> Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "PowerManagementPolicy/PolicyEnabled");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsBooleanType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (BOOLEAN)*MemoryCs-
> >PowerManagementPolicy->PolicyEnabled);
>        if (EFI_ERROR (Status)) {
> @@ -824,7 +909,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -833,9 +918,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->RankCount != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "RankCount", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "RankCount");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >RankCount);
>        if (EFI_ERROR (Status)) {
> @@ -844,10 +929,18 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> +  //
> +  // Handle REGIONS
> +  //
> +  //
> +  // ****** Warning ******
> +  // Unsupported array type:
> +  //
> +
>    //
>    // Handle SECURITYCAPABILITIES
>    //
> @@ -861,9 +954,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->SecurityCapabilities->ConfigurationLockCapable != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION,
> "SecurityCapabilities/ConfigurationLockCapable", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "SecurityCapabilities/ConfigurationLockCapable");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsBooleanType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (BOOLEAN)*MemoryCs-
> >SecurityCapabilities->ConfigurationLockCapable);
>        if (EFI_ERROR (Status)) {
> @@ -872,7 +965,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -881,9 +974,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->SecurityCapabilities->DataLockCapable != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "SecurityCapabilities/DataLockCapable",
> Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "SecurityCapabilities/DataLockCapable");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsBooleanType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (BOOLEAN)*MemoryCs-
> >SecurityCapabilities->DataLockCapable);
>        if (EFI_ERROR (Status)) {
> @@ -892,7 +985,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -901,9 +994,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->SecurityCapabilities->MaxPassphraseCount != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "SecurityCapabilities/MaxPassphraseCount",
> Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "SecurityCapabilities/MaxPassphraseCount");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >SecurityCapabilities->MaxPassphraseCount);
>        if (EFI_ERROR (Status)) {
> @@ -912,7 +1005,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -921,9 +1014,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->SecurityCapabilities->PassphraseCapable != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "SecurityCapabilities/PassphraseCapable",
> Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "SecurityCapabilities/PassphraseCapable");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsBooleanType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (BOOLEAN)*MemoryCs-
> >SecurityCapabilities->PassphraseCapable);
>        if (EFI_ERROR (Status)) {
> @@ -932,7 +1025,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -941,9 +1034,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->SecurityCapabilities->PassphraseLockLimit != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "SecurityCapabilities/PassphraseLockLimit",
> Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "SecurityCapabilities/PassphraseLockLimit");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >SecurityCapabilities->PassphraseLockLimit);
>        if (EFI_ERROR (Status)) {
> @@ -952,18 +1045,26 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> +  //
> +  // Handle SECURITYCAPABILITIES->SECURITYSTATES
> +  //
> +  //
> +  // ****** Warning ******
> +  // Unsupported array type:
> +  //
> +
>    //
>    // Handle SECURITYSTATE
>    //
>    if (MemoryCs->SecurityState != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "SecurityState", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "SecurityState");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->SecurityState);
>        if (EFI_ERROR (Status)) {
> @@ -972,7 +1073,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -981,9 +1082,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->SerialNumber != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "SerialNumber", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "SerialNumber");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->SerialNumber);
>        if (EFI_ERROR (Status)) {
> @@ -992,7 +1093,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -1001,9 +1102,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->SpareDeviceCount != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "SpareDeviceCount", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "SpareDeviceCount");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >SpareDeviceCount);
>        if (EFI_ERROR (Status)) {
> @@ -1012,7 +1113,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -1021,9 +1122,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->SubsystemDeviceID != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "SubsystemDeviceID", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "SubsystemDeviceID");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs-
> >SubsystemDeviceID);
>        if (EFI_ERROR (Status)) {
> @@ -1032,7 +1133,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -1041,9 +1142,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->SubsystemVendorID != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "SubsystemVendorID", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "SubsystemVendorID");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs-
> >SubsystemVendorID);
>        if (EFI_ERROR (Status)) {
> @@ -1052,7 +1153,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -1061,9 +1162,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->VendorID != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "VendorID", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id, "VendorID");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsStringType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, MemoryCs->VendorID);
>        if (EFI_ERROR (Status)) {
> @@ -1072,7 +1173,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -1081,9 +1182,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->VolatileRegionNumberLimit != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "VolatileRegionNumberLimit", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "VolatileRegionNumberLimit");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >VolatileRegionNumberLimit);
>        if (EFI_ERROR (Status)) {
> @@ -1092,7 +1193,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -1101,9 +1202,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->VolatileRegionSizeLimitMiB != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "VolatileRegionSizeLimitMiB", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "VolatileRegionSizeLimitMiB");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >VolatileRegionSizeLimitMiB);
>        if (EFI_ERROR (Status)) {
> @@ -1112,7 +1213,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -1121,9 +1222,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->VolatileRegionSizeMaxMiB != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "VolatileRegionSizeMaxMiB", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "VolatileRegionSizeMaxMiB");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >VolatileRegionSizeMaxMiB);
>        if (EFI_ERROR (Status)) {
> @@ -1132,7 +1233,7 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> @@ -1141,9 +1242,9 @@ RedfishConsumeResourceCommon (
>    //
>    if (MemoryCs->VolatileSizeMiB != NULL) {
>      //
> -    // Find corresponding redpath for collection resource.
> +    // Find corresponding configure language for collection resource.
>      //
> -    ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, "VolatileSizeMiB", Arraykey);
> +    ConfigureLang = GetConfigureLang (MemoryCs->odata_id,
> "VolatileSizeMiB");
>      if (ConfigureLang != NULL) {
>        Status = ApplyFeatureSettingsNumericType (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, ConfigureLang, (UINTN)*MemoryCs-
> >VolatileSizeMiB);
>        if (EFI_ERROR (Status)) {
> @@ -1152,24 +1253,15 @@ RedfishConsumeResourceCommon (
> 
>        FreePool (ConfigureLang);
>      } else {
> -      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for key:
> %a\n", __FUNCTION__, Arraykey));
> +      DEBUG ((DEBUG_ERROR, "%a, can not get configure language for URI:
> %s\n", __FUNCTION__, Private->Uri));
>      }
>    }
> 
> -  //
>  ON_RELEASE:
> 
>    //
>    // Release resource.
>    //
> -  if (EtagInDb != NULL) {
> -    FreePool (EtagInDb);
> -  }
> -
> -  if (Arraykey != NULL) {
> -    FreePool (Arraykey);
> -  }
> -
>    Private->JsonStructProtocol->DestoryStructure (
>                                   Private->JsonStructProtocol,
>                                   (EFI_REST_JSON_STRUCTURE_HEADER *)Memory
> @@ -1181,7 +1273,7 @@ ON_RELEASE:
>  EFI_STATUS
>  ProvisioningMemoryProperties (
>    IN  EFI_REST_JSON_STRUCTURE_PROTOCOL *JsonStructProtocol,
> -  IN  CHAR8 *IputJson,
> +  IN  CHAR8 *InputJson,
>    IN  CHAR8 *ResourceId, OPTIONAL
>    IN  EFI_STRING                        ConfigureLang,
>    IN  BOOLEAN                           ProvisionMode,
> @@ -1191,28 +1283,29 @@ ProvisioningMemoryProperties (
>    EFI_REDFISH_MEMORY_V1_7_1     *Memory;
>    EFI_REDFISH_MEMORY_V1_7_1_CS  *MemoryCs;
>    EFI_STATUS                    Status;
> +  BOOLEAN                       PropertyChanged;
>    INT64                         *NumericValue;
> -  INT32                         *IntegerValue;
> -  BOOLEAN                       *BooleanValue;
> +  INT64                         *NumericArrayValue;
> +  UINTN                         ArraySize;
>    CHAR8                         *AsciiStringValue;
> -  BOOLEAN                       PropertyChanged;
> -  BOOLEAN                       UnusedProperty;
> +  BOOLEAN                       *BooleanValue;
> +  INT32                         *IntegerValue;
> +  CHAR8                         **AsciiStringArrayValue;
> 
> -  if ((JsonStructProtocol == NULL) || (ResultJson == NULL) ||
> IS_EMPTY_STRING (IputJson) || IS_EMPTY_STRING (ConfigureLang)) {
> +  if ((JsonStructProtocol == NULL) || (ResultJson == NULL) ||
> IS_EMPTY_STRING (InputJson) || IS_EMPTY_STRING (ConfigureLang)) {
>      return EFI_INVALID_PARAMETER;
>    }
> 
> -  DEBUG ((REDFISH_DEBUG_TRACE, "%a provision for %s with: %s\n",
> __FUNCTION__, ConfigureLang, (ProvisionMode ? L"Provision all resource" :
> L"Provision existing resource")));
> +  DEBUG ((REDFISH_DEBUG_TRACE, "%a provision for %s with: %s\n",
> __FUNCTION__, ConfigureLang, (ProvisionMode ? L"Provision resource" :
> L"Update resource")));
> 
>    *ResultJson     = NULL;
>    PropertyChanged = FALSE;
> -  UnusedProperty  = TRUE;
> 
>    Memory = NULL;
>    Status = JsonStructProtocol->ToStructure (
>                                   JsonStructProtocol,
>                                   NULL,
> -                                 IputJson,
> +                                 InputJson,
>                                   (EFI_REST_JSON_STRUCTURE_HEADER **)&Memory
>                                   );
>    if (EFI_ERROR (Status)) {
> @@ -1255,6 +1348,19 @@ ProvisioningMemoryProperties (
>      }
>    }
> 
> +  //
> +  // Handle ALLOWEDSPEEDSMHZ
> +  //
> +  if (PropertyChecker (MemoryCs->AllowedSpeedsMHz, ProvisionMode)) {
> +    NumericArrayValue = GetPropertyNumericArrayValue
> (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION,
> L"AllowedSpeedsMHz", ConfigureLang, &ArraySize);
> +    if (NumericArrayValue != NULL) {
> +      if (ProvisionMode || !CompareRedfishNumericArrayValues (MemoryCs-
> >AllowedSpeedsMHz, NumericArrayValue, ArraySize)) {
> +        AddRedfishNumericArray (&MemoryCs->AllowedSpeedsMHz,
> NumericArrayValue, ArraySize);
> +        PropertyChanged = TRUE;
> +      }
> +    }
> +  }
> +
>    //
>    // Handle BASEMODULETYPE
>    //
> @@ -1318,6 +1424,7 @@ ProvisioningMemoryProperties (
>          if (IntegerValue != NULL) {
>            *IntegerValue                 = (BooleanValue ? 0x01 : 0x00);
>            MemoryCs->ConfigurationLocked = IntegerValue;
> +          PropertyChanged               = TRUE;
>          }
>        }
>      }
> @@ -1401,6 +1508,19 @@ ProvisioningMemoryProperties (
>      }
>    }
> 
> +  //
> +  // Handle FUNCTIONCLASSES
> +  //
> +  if (PropertyChecker (MemoryCs->FunctionClasses, ProvisionMode)) {
> +    AsciiStringArrayValue = GetPropertyStringArrayValue
> (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"FunctionClasses",
> ConfigureLang, &ArraySize);
> +    if (AsciiStringArrayValue != NULL) {
> +      if (ProvisionMode || !CompareRedfishStringArrayValues (MemoryCs-
> >FunctionClasses, AsciiStringArrayValue, ArraySize)) {
> +        AddRedfishCharArray (&MemoryCs->FunctionClasses,
> AsciiStringArrayValue, ArraySize);
> +        PropertyChanged = TRUE;
> +      }
> +    }
> +  }
> +
>    //
>    // Handle ISRANKSPAREENABLED
>    //
> @@ -1412,6 +1532,7 @@ ProvisioningMemoryProperties (
>          if (IntegerValue != NULL) {
>            *IntegerValue                = (BooleanValue ? 0x01 : 0x00);
>            MemoryCs->IsRankSpareEnabled = IntegerValue;
> +          PropertyChanged              = TRUE;
>          }
>        }
>      }
> @@ -1428,6 +1549,7 @@ ProvisioningMemoryProperties (
>          if (IntegerValue != NULL) {
>            *IntegerValue                  = (BooleanValue ? 0x01 : 0x00);
>            MemoryCs->IsSpareDeviceEnabled = IntegerValue;
> +          PropertyChanged                = TRUE;
>          }
>        }
>      }
> @@ -1459,6 +1581,19 @@ ProvisioningMemoryProperties (
>      }
>    }
> 
> +  //
> +  // Handle MAXTDPMILLIWATTS
> +  //
> +  if (PropertyChecker (MemoryCs->MaxTDPMilliWatts, ProvisionMode)) {
> +    NumericArrayValue = GetPropertyNumericArrayValue
> (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION, L"MaxTDPMilliWatts",
> ConfigureLang, &ArraySize);
> +    if (NumericArrayValue != NULL) {
> +      if (ProvisionMode || !CompareRedfishNumericArrayValues (MemoryCs-
> >MaxTDPMilliWatts, NumericArrayValue, ArraySize)) {
> +        AddRedfishNumericArray (&MemoryCs->MaxTDPMilliWatts,
> NumericArrayValue, ArraySize);
> +        PropertyChanged = TRUE;
> +      }
> +    }
> +  }
> +
>    //
>    // Handle MEMORYDEVICETYPE
>    //
> @@ -1475,74 +1610,66 @@ ProvisioningMemoryProperties (
>    //
>    // Handle MEMORYLOCATION
>    //
> -  if (MemoryCs->MemoryLocation == NULL) {
> -    MemoryCs->MemoryLocation = AllocateZeroPool (sizeof
> (RedfishMemory_V1_7_1_MemoryLocation_CS));
> -    ASSERT (MemoryCs->MemoryLocation != NULL);
> -    UnusedProperty = TRUE;
> -  } else {
> -    UnusedProperty = FALSE;
> -  }
> -
> -  //
> -  // Handle MEMORYLOCATION->CHANNEL
> -  //
> -  if (PropertyChecker (MemoryCs->MemoryLocation->Channel,
> ProvisionMode)) {
> -    NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, L"MemoryLocation/Channel",
> ConfigureLang);
> -    if (NumericValue != NULL) {
> -      if (ProvisionMode || (*MemoryCs->MemoryLocation->Channel !=
> *NumericValue)) {
> -        MemoryCs->MemoryLocation->Channel = NumericValue;
> -        PropertyChanged                   = TRUE;
> -        UnusedProperty                    = FALSE;
> +  if (MemoryCs->MemoryLocation != NULL) {
> +    //
> +    // Handle MEMORYLOCATION->CHANNEL
> +    //
> +    if (PropertyChecker (MemoryCs->MemoryLocation->Channel,
> ProvisionMode)) {
> +      NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, L"MemoryLocation/Channel",
> ConfigureLang);
> +      if (NumericValue != NULL) {
> +        if (ProvisionMode || (*MemoryCs->MemoryLocation->Channel !=
> *NumericValue)) {
> +          MemoryCs->MemoryLocation->Channel = NumericValue;
> +          PropertyChanged                   = TRUE;
> +        }
>        }
>      }
> -  }
> 
> -  //
> -  // Handle MEMORYLOCATION->MEMORYCONTROLLER
> -  //
> -  if (PropertyChecker (MemoryCs->MemoryLocation->MemoryController,
> ProvisionMode)) {
> -    NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, L"MemoryLocation/MemoryController",
> ConfigureLang);
> -    if (NumericValue != NULL) {
> -      if (ProvisionMode || (*MemoryCs->MemoryLocation-
> >MemoryController != *NumericValue)) {
> -        MemoryCs->MemoryLocation->MemoryController = NumericValue;
> -        PropertyChanged                            = TRUE;
> -        UnusedProperty                             = FALSE;
> +    //
> +    // Handle MEMORYLOCATION->MEMORYCONTROLLER
> +    //
> +    if (PropertyChecker (MemoryCs->MemoryLocation->MemoryController,
> ProvisionMode)) {
> +      NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, L"MemoryLocation/MemoryController",
> ConfigureLang);
> +      if (NumericValue != NULL) {
> +        if (ProvisionMode || (*MemoryCs->MemoryLocation-
> >MemoryController != *NumericValue)) {
> +          MemoryCs->MemoryLocation->MemoryController = NumericValue;
> +          PropertyChanged                            = TRUE;
> +        }
>        }
>      }
> -  }
> 
> -  //
> -  // Handle MEMORYLOCATION->SLOT
> -  //
> -  if (PropertyChecker (MemoryCs->MemoryLocation->Slot, ProvisionMode))
> {
> -    NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, L"MemoryLocation/Slot", ConfigureLang);
> -    if (NumericValue != NULL) {
> -      if (ProvisionMode || (*MemoryCs->MemoryLocation->Slot !=
> *NumericValue)) {
> -        MemoryCs->MemoryLocation->Slot = NumericValue;
> -        PropertyChanged                = TRUE;
> -        UnusedProperty                 = FALSE;
> +    //
> +    // Handle MEMORYLOCATION->SLOT
> +    //
> +    if (PropertyChecker (MemoryCs->MemoryLocation->Slot,
> ProvisionMode)) {
> +      NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, L"MemoryLocation/Slot", ConfigureLang);
> +      if (NumericValue != NULL) {
> +        if (ProvisionMode || (*MemoryCs->MemoryLocation->Slot !=
> *NumericValue)) {
> +          MemoryCs->MemoryLocation->Slot = NumericValue;
> +          PropertyChanged                = TRUE;
> +        }
>        }
>      }
> -  }
> 
> -  //
> -  // Handle MEMORYLOCATION->SOCKET
> -  //
> -  if (PropertyChecker (MemoryCs->MemoryLocation->Socket,
> ProvisionMode)) {
> -    NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, L"MemoryLocation/Socket", ConfigureLang);
> -    if (NumericValue != NULL) {
> -      if (ProvisionMode || (*MemoryCs->MemoryLocation->Socket !=
> *NumericValue)) {
> -        MemoryCs->MemoryLocation->Socket = NumericValue;
> -        PropertyChanged                  = TRUE;
> -        UnusedProperty                   = FALSE;
> +    //
> +    // Handle MEMORYLOCATION->SOCKET
> +    //
> +    if (PropertyChecker (MemoryCs->MemoryLocation->Socket,
> ProvisionMode)) {
> +      NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, L"MemoryLocation/Socket", ConfigureLang);
> +      if (NumericValue != NULL) {
> +        if (ProvisionMode || (*MemoryCs->MemoryLocation->Socket !=
> *NumericValue)) {
> +          MemoryCs->MemoryLocation->Socket = NumericValue;
> +          PropertyChanged                  = TRUE;
> +        }
>        }
>      }
>    }
> 
> -  if (UnusedProperty) {
> -    FreePool (MemoryCs->MemoryLocation);
> -    MemoryCs->MemoryLocation = NULL;
> -  }
> +  //
> +  // Handle MEMORYMEDIA
> +  //
> +  //// ****** Warning ******
> +  // Unsupported array type:
> +  //
> 
>    //
>    // Handle MEMORYSUBSYSTEMCONTROLLERMANUFACTURERID
> @@ -1622,6 +1749,13 @@ ProvisioningMemoryProperties (
>      }
>    }
> 
> +  //
> +  // Handle OPERATINGMEMORYMODES
> +  //
> +  //// ****** Warning ******
> +  // Unsupported array type:
> +  //
> +
>    //
>    // Handle OPERATINGSPEEDMHZ
>    //
> @@ -1690,79 +1824,64 @@ ProvisioningMemoryProperties (
>    //
>    // Handle POWERMANAGEMENTPOLICY
>    //
> -  if (MemoryCs->PowerManagementPolicy == NULL) {
> -    MemoryCs->PowerManagementPolicy = AllocateZeroPool (sizeof
> (RedfishMemory_V1_7_1_PowerManagementPolicy_CS));
> -    ASSERT (MemoryCs->PowerManagementPolicy != NULL);
> -    UnusedProperty = TRUE;
> -  } else {
> -    UnusedProperty = FALSE;
> -  }
> -
> -  //
> -  // Handle POWERMANAGEMENTPOLICY-
> >AVERAGEPOWERBUDGETMILLIWATTS
> -  //
> -  if (PropertyChecker (MemoryCs->PowerManagementPolicy-
> >AveragePowerBudgetMilliWatts, ProvisionMode)) {
> -    NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION,
> L"PowerManagementPolicy/AveragePowerBudgetMilliWatts",
> ConfigureLang);
> -    if (NumericValue != NULL) {
> -      if (ProvisionMode || (*MemoryCs->PowerManagementPolicy-
> >AveragePowerBudgetMilliWatts != *NumericValue)) {
> -        MemoryCs->PowerManagementPolicy-
> >AveragePowerBudgetMilliWatts = NumericValue;
> -        PropertyChanged                                               = TRUE;
> -        UnusedProperty                                                = FALSE;
> +  if (MemoryCs->PowerManagementPolicy != NULL) {
> +    //
> +    // Handle POWERMANAGEMENTPOLICY-
> >AVERAGEPOWERBUDGETMILLIWATTS
> +    //
> +    if (PropertyChecker (MemoryCs->PowerManagementPolicy-
> >AveragePowerBudgetMilliWatts, ProvisionMode)) {
> +      NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION,
> L"PowerManagementPolicy/AveragePowerBudgetMilliWatts",
> ConfigureLang);
> +      if (NumericValue != NULL) {
> +        if (ProvisionMode || (*MemoryCs->PowerManagementPolicy-
> >AveragePowerBudgetMilliWatts != *NumericValue)) {
> +          MemoryCs->PowerManagementPolicy-
> >AveragePowerBudgetMilliWatts = NumericValue;
> +          PropertyChanged                                               = TRUE;
> +        }
>        }
>      }
> -  }
> 
> -  //
> -  // Handle POWERMANAGEMENTPOLICY->MAXTDPMILLIWATTS
> -  //
> -  if (PropertyChecker (MemoryCs->PowerManagementPolicy-
> >MaxTDPMilliWatts, ProvisionMode)) {
> -    NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION,
> L"PowerManagementPolicy/MaxTDPMilliWatts", ConfigureLang);
> -    if (NumericValue != NULL) {
> -      if (ProvisionMode || (*MemoryCs->PowerManagementPolicy-
> >MaxTDPMilliWatts != *NumericValue)) {
> -        MemoryCs->PowerManagementPolicy->MaxTDPMilliWatts =
> NumericValue;
> -        PropertyChanged                                   = TRUE;
> -        UnusedProperty                                    = FALSE;
> +    //
> +    // Handle POWERMANAGEMENTPOLICY->MAXTDPMILLIWATTS
> +    //
> +    if (PropertyChecker (MemoryCs->PowerManagementPolicy-
> >MaxTDPMilliWatts, ProvisionMode)) {
> +      NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION,
> L"PowerManagementPolicy/MaxTDPMilliWatts", ConfigureLang);
> +      if (NumericValue != NULL) {
> +        if (ProvisionMode || (*MemoryCs->PowerManagementPolicy-
> >MaxTDPMilliWatts != *NumericValue)) {
> +          MemoryCs->PowerManagementPolicy->MaxTDPMilliWatts =
> NumericValue;
> +          PropertyChanged                                   = TRUE;
> +        }
>        }
>      }
> -  }
> 
> -  //
> -  // Handle POWERMANAGEMENTPOLICY-
> >PEAKPOWERBUDGETMILLIWATTS
> -  //
> -  if (PropertyChecker (MemoryCs->PowerManagementPolicy-
> >PeakPowerBudgetMilliWatts, ProvisionMode)) {
> -    NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION,
> L"PowerManagementPolicy/PeakPowerBudgetMilliWatts", ConfigureLang);
> -    if (NumericValue != NULL) {
> -      if (ProvisionMode || (*MemoryCs->PowerManagementPolicy-
> >PeakPowerBudgetMilliWatts != *NumericValue)) {
> -        MemoryCs->PowerManagementPolicy->PeakPowerBudgetMilliWatts =
> NumericValue;
> -        PropertyChanged                                            = TRUE;
> -        UnusedProperty                                             = FALSE;
> +    //
> +    // Handle POWERMANAGEMENTPOLICY-
> >PEAKPOWERBUDGETMILLIWATTS
> +    //
> +    if (PropertyChecker (MemoryCs->PowerManagementPolicy-
> >PeakPowerBudgetMilliWatts, ProvisionMode)) {
> +      NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION,
> L"PowerManagementPolicy/PeakPowerBudgetMilliWatts", ConfigureLang);
> +      if (NumericValue != NULL) {
> +        if (ProvisionMode || (*MemoryCs->PowerManagementPolicy-
> >PeakPowerBudgetMilliWatts != *NumericValue)) {
> +          MemoryCs->PowerManagementPolicy->PeakPowerBudgetMilliWatts
> = NumericValue;
> +          PropertyChanged                                            = TRUE;
> +        }
>        }
>      }
> -  }
> 
> -  //
> -  // Handle POWERMANAGEMENTPOLICY->POLICYENABLED
> -  //
> -  if (PropertyChecker (MemoryCs->PowerManagementPolicy-
> >PolicyEnabled, ProvisionMode)) {
> -    BooleanValue = GetPropertyBooleanValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, L"PowerManagementPolicy/PolicyEnabled",
> ConfigureLang);
> -    if (BooleanValue != NULL) {
> -      if (ProvisionMode || (*MemoryCs->PowerManagementPolicy-
> >PolicyEnabled != *BooleanValue)) {
> -        IntegerValue = AllocatePool (sizeof (*IntegerValue));
> -        if (IntegerValue != NULL) {
> -          *IntegerValue                                  = (BooleanValue ? 0x01 : 0x00);
> -          MemoryCs->PowerManagementPolicy->PolicyEnabled = IntegerValue;
> -          PropertyChanged                                = TRUE;
> -          UnusedProperty                                 = FALSE;
> +    //
> +    // Handle POWERMANAGEMENTPOLICY->POLICYENABLED
> +    //
> +    if (PropertyChecker (MemoryCs->PowerManagementPolicy-
> >PolicyEnabled, ProvisionMode)) {
> +      BooleanValue = GetPropertyBooleanValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, L"PowerManagementPolicy/PolicyEnabled",
> ConfigureLang);
> +      if (BooleanValue != NULL) {
> +        if (ProvisionMode || (*MemoryCs->PowerManagementPolicy-
> >PolicyEnabled != *BooleanValue)) {
> +          IntegerValue = AllocatePool (sizeof (*IntegerValue));
> +          if (IntegerValue != NULL) {
> +            *IntegerValue                                  = (BooleanValue ? 0x01 : 0x00);
> +            MemoryCs->PowerManagementPolicy->PolicyEnabled =
> IntegerValue;
> +            PropertyChanged                                = TRUE;
> +          }
>          }
>        }
>      }
>    }
> 
> -  if (UnusedProperty) {
> -    FreePool (MemoryCs->PowerManagementPolicy);
> -    MemoryCs->PowerManagementPolicy = NULL;
> -  }
> -
>    //
>    // Handle RANKCOUNT
>    //
> @@ -1777,101 +1896,99 @@ ProvisioningMemoryProperties (
>    }
> 
>    //
> -  // Handle SECURITYCAPABILITIES
> +  // Handle REGIONS
> +  //
> +  //// ****** Warning ******
> +  // Unsupported array type:
>    //
> -  if (MemoryCs->SecurityCapabilities == NULL) {
> -    MemoryCs->SecurityCapabilities = AllocateZeroPool (sizeof
> (RedfishMemory_V1_7_1_SecurityCapabilities_CS));
> -    ASSERT (MemoryCs->SecurityCapabilities != NULL);
> -    UnusedProperty = TRUE;
> -  } else {
> -    UnusedProperty = FALSE;
> -  }
> 
>    //
> -  // Handle SECURITYCAPABILITIES->CONFIGURATIONLOCKCAPABLE
> +  // Handle SECURITYCAPABILITIES
>    //
> -  if (PropertyChecker (MemoryCs->SecurityCapabilities-
> >ConfigurationLockCapable, ProvisionMode)) {
> -    BooleanValue = GetPropertyBooleanValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION,
> L"SecurityCapabilities/ConfigurationLockCapable", ConfigureLang);
> -    if (BooleanValue != NULL) {
> -      if (ProvisionMode || (*MemoryCs->SecurityCapabilities-
> >ConfigurationLockCapable != *BooleanValue)) {
> -        IntegerValue = AllocatePool (sizeof (*IntegerValue));
> -        if (IntegerValue != NULL) {
> -          *IntegerValue                                            = (BooleanValue ? 0x01 : 0x00);
> -          MemoryCs->SecurityCapabilities->ConfigurationLockCapable =
> IntegerValue;
> -          PropertyChanged                                          = TRUE;
> -          UnusedProperty                                           = FALSE;
> +  if (MemoryCs->SecurityCapabilities != NULL) {
> +    //
> +    // Handle SECURITYCAPABILITIES->CONFIGURATIONLOCKCAPABLE
> +    //
> +    if (PropertyChecker (MemoryCs->SecurityCapabilities-
> >ConfigurationLockCapable, ProvisionMode)) {
> +      BooleanValue = GetPropertyBooleanValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION,
> L"SecurityCapabilities/ConfigurationLockCapable", ConfigureLang);
> +      if (BooleanValue != NULL) {
> +        if (ProvisionMode || (*MemoryCs->SecurityCapabilities-
> >ConfigurationLockCapable != *BooleanValue)) {
> +          IntegerValue = AllocatePool (sizeof (*IntegerValue));
> +          if (IntegerValue != NULL) {
> +            *IntegerValue                                            = (BooleanValue ? 0x01 : 0x00);
> +            MemoryCs->SecurityCapabilities->ConfigurationLockCapable =
> IntegerValue;
> +            PropertyChanged                                          = TRUE;
> +          }
>          }
>        }
>      }
> -  }
> 
> -  //
> -  // Handle SECURITYCAPABILITIES->DATALOCKCAPABLE
> -  //
> -  if (PropertyChecker (MemoryCs->SecurityCapabilities->DataLockCapable,
> ProvisionMode)) {
> -    BooleanValue = GetPropertyBooleanValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, L"SecurityCapabilities/DataLockCapable",
> ConfigureLang);
> -    if (BooleanValue != NULL) {
> -      if (ProvisionMode || (*MemoryCs->SecurityCapabilities-
> >DataLockCapable != *BooleanValue)) {
> -        IntegerValue = AllocatePool (sizeof (*IntegerValue));
> -        if (IntegerValue != NULL) {
> -          *IntegerValue                                   = (BooleanValue ? 0x01 : 0x00);
> -          MemoryCs->SecurityCapabilities->DataLockCapable = IntegerValue;
> -          PropertyChanged                                 = TRUE;
> -          UnusedProperty                                  = FALSE;
> +    //
> +    // Handle SECURITYCAPABILITIES->DATALOCKCAPABLE
> +    //
> +    if (PropertyChecker (MemoryCs->SecurityCapabilities->DataLockCapable,
> ProvisionMode)) {
> +      BooleanValue = GetPropertyBooleanValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, L"SecurityCapabilities/DataLockCapable",
> ConfigureLang);
> +      if (BooleanValue != NULL) {
> +        if (ProvisionMode || (*MemoryCs->SecurityCapabilities-
> >DataLockCapable != *BooleanValue)) {
> +          IntegerValue = AllocatePool (sizeof (*IntegerValue));
> +          if (IntegerValue != NULL) {
> +            *IntegerValue                                   = (BooleanValue ? 0x01 : 0x00);
> +            MemoryCs->SecurityCapabilities->DataLockCapable = IntegerValue;
> +            PropertyChanged                                 = TRUE;
> +          }
>          }
>        }
>      }
> -  }
> 
> -  //
> -  // Handle SECURITYCAPABILITIES->MAXPASSPHRASECOUNT
> -  //
> -  if (PropertyChecker (MemoryCs->SecurityCapabilities-
> >MaxPassphraseCount, ProvisionMode)) {
> -    NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION,
> L"SecurityCapabilities/MaxPassphraseCount", ConfigureLang);
> -    if (NumericValue != NULL) {
> -      if (ProvisionMode || (*MemoryCs->SecurityCapabilities-
> >MaxPassphraseCount != *NumericValue)) {
> -        MemoryCs->SecurityCapabilities->MaxPassphraseCount =
> NumericValue;
> -        PropertyChanged                                    = TRUE;
> -        UnusedProperty                                     = FALSE;
> +    //
> +    // Handle SECURITYCAPABILITIES->MAXPASSPHRASECOUNT
> +    //
> +    if (PropertyChecker (MemoryCs->SecurityCapabilities-
> >MaxPassphraseCount, ProvisionMode)) {
> +      NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION,
> L"SecurityCapabilities/MaxPassphraseCount", ConfigureLang);
> +      if (NumericValue != NULL) {
> +        if (ProvisionMode || (*MemoryCs->SecurityCapabilities-
> >MaxPassphraseCount != *NumericValue)) {
> +          MemoryCs->SecurityCapabilities->MaxPassphraseCount =
> NumericValue;
> +          PropertyChanged                                    = TRUE;
> +        }
>        }
>      }
> -  }
> 
> -  //
> -  // Handle SECURITYCAPABILITIES->PASSPHRASECAPABLE
> -  //
> -  if (PropertyChecker (MemoryCs->SecurityCapabilities->PassphraseCapable,
> ProvisionMode)) {
> -    BooleanValue = GetPropertyBooleanValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, L"SecurityCapabilities/PassphraseCapable",
> ConfigureLang);
> -    if (BooleanValue != NULL) {
> -      if (ProvisionMode || (*MemoryCs->SecurityCapabilities-
> >PassphraseCapable != *BooleanValue)) {
> -        IntegerValue = AllocatePool (sizeof (*IntegerValue));
> -        if (IntegerValue != NULL) {
> -          *IntegerValue                                     = (BooleanValue ? 0x01 : 0x00);
> -          MemoryCs->SecurityCapabilities->PassphraseCapable = IntegerValue;
> -          PropertyChanged                                   = TRUE;
> -          UnusedProperty                                    = FALSE;
> +    //
> +    // Handle SECURITYCAPABILITIES->PASSPHRASECAPABLE
> +    //
> +    if (PropertyChecker (MemoryCs->SecurityCapabilities-
> >PassphraseCapable, ProvisionMode)) {
> +      BooleanValue = GetPropertyBooleanValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, L"SecurityCapabilities/PassphraseCapable",
> ConfigureLang);
> +      if (BooleanValue != NULL) {
> +        if (ProvisionMode || (*MemoryCs->SecurityCapabilities-
> >PassphraseCapable != *BooleanValue)) {
> +          IntegerValue = AllocatePool (sizeof (*IntegerValue));
> +          if (IntegerValue != NULL) {
> +            *IntegerValue                                     = (BooleanValue ? 0x01 : 0x00);
> +            MemoryCs->SecurityCapabilities->PassphraseCapable = IntegerValue;
> +            PropertyChanged                                   = TRUE;
> +          }
>          }
>        }
>      }
> -  }
> 
> -  //
> -  // Handle SECURITYCAPABILITIES->PASSPHRASELOCKLIMIT
> -  //
> -  if (PropertyChecker (MemoryCs->SecurityCapabilities-
> >PassphraseLockLimit, ProvisionMode)) {
> -    NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, L"SecurityCapabilities/PassphraseLockLimit",
> ConfigureLang);
> -    if (NumericValue != NULL) {
> -      if (ProvisionMode || (*MemoryCs->SecurityCapabilities-
> >PassphraseLockLimit != *NumericValue)) {
> -        MemoryCs->SecurityCapabilities->PassphraseLockLimit = NumericValue;
> -        PropertyChanged                                     = TRUE;
> -        UnusedProperty                                      = FALSE;
> +    //
> +    // Handle SECURITYCAPABILITIES->PASSPHRASELOCKLIMIT
> +    //
> +    if (PropertyChecker (MemoryCs->SecurityCapabilities-
> >PassphraseLockLimit, ProvisionMode)) {
> +      NumericValue = GetPropertyNumericValue (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, L"SecurityCapabilities/PassphraseLockLimit",
> ConfigureLang);
> +      if (NumericValue != NULL) {
> +        if (ProvisionMode || (*MemoryCs->SecurityCapabilities-
> >PassphraseLockLimit != *NumericValue)) {
> +          MemoryCs->SecurityCapabilities->PassphraseLockLimit =
> NumericValue;
> +          PropertyChanged                                     = TRUE;
> +        }
>        }
>      }
> -  }
> 
> -  if (UnusedProperty) {
> -    FreePool (MemoryCs->SecurityCapabilities);
> -    MemoryCs->SecurityCapabilities = NULL;
> +    //
> +    // Handle SECURITYCAPABILITIES->SECURITYSTATES
> +    //
> +    //// ****** Warning ******
> +    // Unsupported array type:
> +    //
>    }
> 
>    //
> @@ -2035,18 +2152,17 @@ ProvisioningMemoryResource (
>    IN  EFI_STRING                       ConfigureLang
>    )
>  {
> -  CHAR8       *MemoryJson;
> +  CHAR8       *Json;
>    EFI_STATUS  Status;
> -  CHAR8       *NewResourceLocation;
> -  CHAR8       *NewKey;
> +  EFI_STRING  NewResourceLocation;
>    CHAR8       *EtagStr;
>    CHAR8       ResourceId[16];
> -  CHAR8       NewUri[255];
> 
>    if (IS_EMPTY_STRING (ConfigureLang) || (Private == NULL)) {
>      return EFI_INVALID_PARAMETER;
>    }
> 
> +  EtagStr = NULL;
>    AsciiSPrint (ResourceId, sizeof (ResourceId), "%d", Index);
> 
>    Status = ProvisioningMemoryProperties (
> @@ -2055,55 +2171,45 @@ ProvisioningMemoryResource (
>               ResourceId,
>               ConfigureLang,
>               TRUE,
> -             &MemoryJson
> +             &Json
>               );
>    if (EFI_ERROR (Status)) {
>      DEBUG ((DEBUG_ERROR, "%a, provisioning resource for %s failed: %r\n",
> __FUNCTION__, ConfigureLang, Status));
>      return Status;
>    }
> 
> -  Status = CreatePayloadToPostResource (Private->RedfishService, Private-
> >Payload, MemoryJson, &NewResourceLocation, &EtagStr);
> +  Status = CreatePayloadToPostResource (Private->RedfishService, Private-
> >Payload, Json, &NewResourceLocation, &EtagStr);
>    if (EFI_ERROR (Status)) {
> -    DEBUG ((DEBUG_ERROR, "%a, post memory resource for %s failed: %r\n",
> __FUNCTION__, ConfigureLang, Status));
> +    DEBUG ((DEBUG_ERROR, "%a, post Memory resource for %s failed: %r\n",
> __FUNCTION__, ConfigureLang, Status));
>      goto RELEASE_RESOURCE;
>    }
> 
> -  NewUri[0] = '\0';
> +  ASSERT (NewResourceLocation != NULL);
> 
>    //
>    // Keep location of new resource.
>    //
>    if (NewResourceLocation != NULL) {
> -    //
> -    // Find key
> -    //
> -    NewKey = AsciiStrStr (NewResourceLocation, RESOURCE_SCHEMA);
> -    if (NewKey != NULL) {
> -      NewKey += 6;
> -      //
> -      // skip '/'
> -      //
> -      if (NewKey[0] == '/') {
> -        NewKey++;
> -      }
> -
> -      SetConfigureLangWithkey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, NewKey, Index);
> -      AsciiSPrint (NewUri, sizeof (NewUri), "%a[%a]", Private->Uri, NewKey);
> -      FreePool (NewResourceLocation);
> -    }
> +    RedfisSetRedfishUri (ConfigureLang, NewResourceLocation);
>    }
> 
>    //
>    // Handle Etag
>    //
>    if (EtagStr != NULL) {
> -    SetEtagWithUri (EtagStr, NewUri);
> +    SetEtagWithUri (EtagStr, NewResourceLocation);
>      FreePool (EtagStr);
>    }
> 
>  RELEASE_RESOURCE:
> 
> -  FreePool (MemoryJson);
> +  if (NewResourceLocation != NULL) {
> +    FreePool (NewResourceLocation);
> +  }
> +
> +  if (Json != NULL) {
> +    FreePool (Json);
> +  }
> 
>    return Status;
>  }
> @@ -2123,12 +2229,18 @@ ProvisioningMemoryResources (
> 
>    Status = RedfishFeatureGetUnifiedArrayTypeConfigureLang
> (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION,
> REDPATH_ARRAY_PATTERN, &UnifiedConfigureLangList);
>    if (EFI_ERROR (Status) || (UnifiedConfigureLangList.Count == 0)) {
> -    DEBUG ((DEBUG_ERROR, "%a, No HII question found with redpath: %s:
> %r\n", __FUNCTION__, REDPATH_ARRAY_PATTERN, Status));
> +    DEBUG ((DEBUG_ERROR, "%a, No HII question found with configure
> language: %s: %r\n", __FUNCTION__, REDPATH_ARRAY_PATTERN, Status));
>      return EFI_NOT_FOUND;
>    }
> 
> +  //
> +  // Set the configuration language in the
> RESOURCE_INFORMATION_EXCHANGE.
> +  // This information is sent back to the parent resource (e.g. the collection
> driver).
> +  //
> +  EdkIIRedfishResourceSetConfigureLang (&UnifiedConfigureLangList);
> +
>    for (Index = 0; Index < UnifiedConfigureLangList.Count; Index++) {
> -    DEBUG ((DEBUG_INFO, "[%d] create memory resource from: %s\n",
> UnifiedConfigureLangList.List[Index].Index,
> UnifiedConfigureLangList.List[Index].ConfigureLang));
> +    DEBUG ((DEBUG_INFO, "[%d] create Memory resource from: %s\n",
> UnifiedConfigureLangList.List[Index].Index,
> UnifiedConfigureLangList.List[Index].ConfigureLang));
>      ProvisioningMemoryResource (Private,
> UnifiedConfigureLangList.List[Index].Index,
> UnifiedConfigureLangList.List[Index].ConfigureLang);
>      FreePool (UnifiedConfigureLangList.List[Index].ConfigureLang);
>    }
> @@ -2141,14 +2253,70 @@ ProvisioningMemoryExistResource (
>    IN  REDFISH_RESOURCE_COMMON_PRIVATE  *Private
>    )
>  {
> +  EFI_STATUS  Status;
> +  EFI_STRING  ConfigureLang;
> +  CHAR8       *EtagStr;
> +  CHAR8       *Json;
> +
>    if (Private == NULL) {
>      return EFI_INVALID_PARAMETER;
>    }
> 
> -  Private->Json = JsonDumpString (RedfishJsonInPayload (Private->Payload),
> EDKII_JSON_COMPACT);
> -  ASSERT (Private->Json != NULL);
> +  EtagStr       = NULL;
> +  Json          = NULL;
> +  ConfigureLang = NULL;
> +
> +  ConfigureLang = RedfishGetConfigLanguage (Private->Uri);
> +  if (ConfigureLang == NULL) {
> +    return EFI_NOT_FOUND;
> +  }
> +
> +  Status = ProvisioningMemoryProperties (
> +             Private->JsonStructProtocol,
> +             MemoryEmptyJson,
> +             NULL,
> +             ConfigureLang,
> +             TRUE,
> +             &Json
> +             );
> +  if (EFI_ERROR (Status)) {
> +    if (Status == EFI_NOT_FOUND) {
> +      DEBUG ((REDFISH_DEBUG_TRACE, "%a, provisioning existing resource
> for %s ignored. Nothing changed\n", __FUNCTION__, ConfigureLang));
> +    } else {
> +      DEBUG ((DEBUG_ERROR, "%a, provisioning existing resource for %s
> failed: %r\n", __FUNCTION__, ConfigureLang, Status));
> +    }
> +
> +    goto ON_RELEASE;
> +  }
> +
> +  DEBUG ((REDFISH_DEBUG_TRACE, "%a, provisioning existing resource for
> %s\n", __FUNCTION__, ConfigureLang));
> +  //
> +  // PUT back to instance
> +  //
> +  Status = CreatePayloadToPatchResource (Private->RedfishService, Private-
> >Payload, Json, &EtagStr);
> +  if (EFI_ERROR (Status)) {
> +    DEBUG ((DEBUG_ERROR, "%a, patch resource for %s failed: %r\n",
> __FUNCTION__, ConfigureLang, Status));
> +  }
> 
> -  return RedfishUpdateResourceCommon (Private, Private->Json);
> +  //
> +  // Handle Etag
> +  //
> +  if (EtagStr != NULL) {
> +    SetEtagWithUri (EtagStr, Private->Uri);
> +    FreePool (EtagStr);
> +  }
> +
> +ON_RELEASE:
> +
> +  if (Json != NULL) {
> +    FreePool (Json);
> +  }
> +
> +  if (ConfigureLang != NULL) {
> +    FreePool (ConfigureLang);
> +  }
> +
> +  return Status;
>  }
> 
>  /**
> @@ -2196,9 +2364,6 @@ RedfishCheckResourceCommon (
>    EFI_STRING  *ConfigureLangList;
>    UINTN       Count;
>    EFI_STRING  Property;
> -  CHAR8       *PropertyAscii;
> -  UINTN       BuffSize;
> -  CHAR8       *Match;
> 
>    if ((Private == NULL) || IS_EMPTY_STRING (Json)) {
>      return EFI_INVALID_PARAMETER;
> @@ -2216,27 +2381,16 @@ RedfishCheckResourceCommon (
> 
>    Status = EFI_SUCCESS;
>    for (Index = 0; Index < Count; Index++) {
> -    Property = GetPropertyFromConfigureLang (ConfigureLangList[Index]);
> +    Property = GetPropertyFromConfigureLang (Private->Uri,
> ConfigureLangList[Index]);
>      if (Property == NULL) {
>        continue;
>      }
> 
> -    DEBUG ((DEBUG_INFO, "[%d] check resource from: %s\n", Index,
> Property));
> -
> -    BuffSize      = StrLen (Property) + 1;
> -    PropertyAscii = AllocatePool (BuffSize);
> -    UnicodeStrToAsciiStrS (Property, PropertyAscii, BuffSize);
> -
> -    //
> -    // check to see if it is partial match.
> -    //
> -    Match = AsciiStrStr (Json, PropertyAscii);
> -    if ((Match == NULL) || (AsciiStrnCmp (Match, PropertyAscii, AsciiStrLen
> (PropertyAscii)) != 0)) {
> +    DEBUG ((DEBUG_INFO, "%a, [%d] check attribute for: %s\n",
> __FUNCTION__, Index, Property));
> +    if (!MatchPropertyWithJsonContext (Property, Json)) {
> +      DEBUG ((DEBUG_INFO, "%a, property is missing: %s\n", __FUNCTION__,
> Property));
>        Status = EFI_NOT_FOUND;
> -      DEBUG ((DEBUG_ERROR, "%a, property %a is missing\n",
> __FUNCTION__, PropertyAscii));
>      }
> -
> -    FreePool (PropertyAscii);
>    }
> 
>    FreePool (ConfigureLangList);
> @@ -2257,40 +2411,34 @@ RedfishCheckResourceCommon (
>  EFI_STATUS
>  RedfishUpdateResourceCommon (
>    IN     REDFISH_RESOURCE_COMMON_PRIVATE  *Private,
> -  IN     CHAR8                            *Json
> +  IN     CHAR8                            *InputJson
>    )
>  {
>    EFI_STATUS  Status;
> -  CHAR8       *MemoryJson;
> -  CHAR8       *ArrayKey;
> +  CHAR8       *Json;
>    EFI_STRING  ConfigureLang;
>    CHAR8       *EtagStr;
> 
> -  if ((Private == NULL) || IS_EMPTY_STRING (Json)) {
> +  if ((Private == NULL) || IS_EMPTY_STRING (InputJson)) {
>      return EFI_INVALID_PARAMETER;
>    }
> 
> -  MemoryJson    = NULL;
> +  EtagStr       = NULL;
> +  Json          = NULL;
>    ConfigureLang = NULL;
> -  ArrayKey      = NULL;
> 
> -  Status = GetArraykeyFromUri (Private->Uri, &ArrayKey);
> -  if (EFI_ERROR (Status)) {
> -    return Status;
> -  }
> -
> -  ConfigureLang = GetConfigureLangByKey (RESOURCE_SCHEMA,
> RESOURCE_SCHEMA_VERSION, NULL, ArrayKey);
> +  ConfigureLang = RedfishGetConfigLanguage (Private->Uri);
>    if (ConfigureLang == NULL) {
>      return EFI_NOT_FOUND;
>    }
> 
>    Status = ProvisioningMemoryProperties (
>               Private->JsonStructProtocol,
> -             Json,
> +             InputJson,
>               NULL,
>               ConfigureLang,
>               FALSE,
> -             &MemoryJson
> +             &Json
>               );
>    if (EFI_ERROR (Status)) {
>      if (Status == EFI_NOT_FOUND) {
> @@ -2306,9 +2454,9 @@ RedfishUpdateResourceCommon (
>    //
>    // PUT back to instance
>    //
> -  Status = CreatePayloadToPatchResource (Private->RedfishService, Private-
> >Payload, MemoryJson, &EtagStr);
> +  Status = CreatePayloadToPatchResource (Private->RedfishService, Private-
> >Payload, Json, &EtagStr);
>    if (EFI_ERROR (Status)) {
> -    DEBUG ((DEBUG_ERROR, "%a, post memory resource for %s failed: %r\n",
> __FUNCTION__, ConfigureLang, Status));
> +    DEBUG ((DEBUG_ERROR, "%a, patch resource for %s failed: %r\n",
> __FUNCTION__, ConfigureLang, Status));
>    }
> 
>    //
> @@ -2321,17 +2469,83 @@ RedfishUpdateResourceCommon (
> 
>  ON_RELEASE:
> 
> -  if (MemoryJson != NULL) {
> -    FreePool (MemoryJson);
> +  if (Json != NULL) {
> +    FreePool (Json);
>    }
> 
>    if (ConfigureLang != NULL) {
>      FreePool (ConfigureLang);
>    }
> 
> -  if (ArrayKey != NULL) {
> -    FreePool (ArrayKey);
> +  return Status;
> +}
> +
> +/**
> +  Identify resource from given URI.
> +
> +  @param[in]   This                Pointer to
> REDFISH_RESOURCE_COMMON_PRIVATE instance.
> +  @param[in]   Json                The JSON to consume.
> +
> +  @retval EFI_SUCCESS              Value is returned successfully.
> +  @retval Others                   Some error happened.
> +
> +**/
> +EFI_STATUS
> +RedfishIdentifyResourceCommon (
> +  IN     REDFISH_RESOURCE_COMMON_PRIVATE  *Private,
> +  IN     CHAR8                            *Json
> +  )
> +{
> +  BOOLEAN                                      Supported;
> +  EFI_STATUS                                   Status;
> +  EFI_STRING                                   EndOfChar;
> +  REDFISH_FEATURE_ARRAY_TYPE_CONFIG_LANG_LIST  ConfigLangList;
> +
> +  Supported = RedfishIdentifyResource (Private->Uri, Private->Json);
> +  if (Supported) {
> +    Status = RedfishFeatureGetUnifiedArrayTypeConfigureLang
> (RESOURCE_SCHEMA, RESOURCE_SCHEMA_VERSION,
> REDPATH_ARRAY_PATTERN, &ConfigLangList);
> +    if (EFI_ERROR (Status)) {
> +      DEBUG ((DEBUG_ERROR, "%a,
> BiosConfigToRedfishGetConfigureLangRegex failed: %r\n", __FUNCTION__,
> Status));
> +      return Status;
> +    }
> +
> +    if (ConfigLangList.Count == 0) {
> +      return EFI_SUCCESS;
> +    }
> +
> +    EndOfChar = StrStr (ConfigLangList.List[0].ConfigureLang, L"}");
> +    if (EndOfChar == NULL) {
> +      ASSERT (FALSE);
> +      return EFI_DEVICE_ERROR;
> +    }
> +
> +    // EndOfChar = StrStr (ConfigLangList.List[0].ConfigureLang, L"}");
> +    Status = IsRedpathArray (ConfigLangList.List[0].ConfigureLang, NULL,
> &EndOfChar);
> +    if (EFI_ERROR (Status) && (Status != EFI_NOT_FOUND)) {
> +      ASSERT (FALSE);
> +      return EFI_DEVICE_ERROR;
> +    }
> +
> +    if (Status != EFI_SUCCESS) {
> +      //
> +      // This is not the collection redpath.
> +      //
> +      GetRedpathNodeByIndex (ConfigLangList.List[0].ConfigureLang, 0,
> &EndOfChar);
> +    }
> +
> +    *(++EndOfChar) = '\0';
> +    //
> +    // Keep URI and ConfigLang mapping
> +    //
> +    RedfisSetRedfishUri (ConfigLangList.List[0].ConfigureLang, Private->Uri);
> +    //
> +    // Set the configuration language in the
> RESOURCE_INFORMATION_EXCHANGE.
> +    // This information is sent back to the parent resource (e.g. the collection
> driver).
> +    //
> +    EdkIIRedfishResourceSetConfigureLang (&ConfigLangList);
> +    DestroyConfiglanguageList (&ConfigLangList);
> +    return EFI_SUCCESS;
>    }
> 
> -  return Status;
> +  return EFI_UNSUPPORTED;
>  }
> diff --git
> a/RedfishClientPkg/Features/Memory/V1_7_1/Dxe/RedfishMemoryDxe.c
> b/RedfishClientPkg/Features/Memory/V1_7_1/Dxe/MemoryDxe.c
> similarity index 75%
> rename from
> RedfishClientPkg/Features/Memory/V1_7_1/Dxe/RedfishMemoryDxe.c
> rename to RedfishClientPkg/Features/Memory/V1_7_1/Dxe/MemoryDxe.c
> index 90e3a20f..07058888 100644
> ---
> a/RedfishClientPkg/Features/Memory/V1_7_1/Dxe/RedfishMemoryDxe.c
> +++ b/RedfishClientPkg/Features/Memory/V1_7_1/Dxe/MemoryDxe.c
> @@ -1,16 +1,18 @@
>  /** @file
>    Redfish feature driver implementation - Memory
> 
> -  (C) Copyright 2020-2021 Hewlett Packard Enterprise Development LP<BR>
> +  (C) Copyright 2020-2022 Hewlett Packard Enterprise Development LP<BR>
> 
>    SPDX-License-Identifier: BSD-2-Clause-Patent
> 
>  **/
> 
> -#include "../Common/RedfishMemoryCommon.h"
> +#include "../Common/MemoryCommon.h"
> 
>  extern REDFISH_RESOURCE_COMMON_PRIVATE
> *mRedfishResourcePrivate;
> 
> +EFI_HANDLE  medfishResourceConfigProtocolHandle;
> +
>  /**
>    Provising redfish resource by given URI.
> 
> @@ -26,7 +28,7 @@ extern REDFISH_RESOURCE_COMMON_PRIVATE
> *mRedfishResourcePrivate;
>  EFI_STATUS
>  RedfishResourceProvisioningResource (
>    IN     EDKII_REDFISH_RESOURCE_CONFIG_PROTOCOL  *This,
> -  IN     CHAR8                                   *Uri,
> +  IN     EFI_STRING                              Uri,
>    IN     BOOLEAN                                 PostMode
>    )
>  {
> @@ -38,9 +40,7 @@ RedfishResourceProvisioningResource (
>      return EFI_INVALID_PARAMETER;
>    }
> 
> -  if (!PostMode) {
> -    return EFI_UNSUPPORTED;
> -  }
> +  DEBUG ((DEBUG_INFO, "%a, provisioning in %s mode\n", __FUNCTION__,
> (PostMode ? L"POST" : L"PATCH")));
> 
>    Private =
> REDFISH_RESOURCE_COMMON_PRIVATE_DATA_FROM_RESOURCE_PROTO
> COL (This);
> 
> @@ -48,9 +48,9 @@ RedfishResourceProvisioningResource (
>      return EFI_NOT_READY;
>    }
> 
> -  Status = GetResourceByPath (Private->RedfishService, Uri, &Response);
> +  Status = GetResourceByUri (Private->RedfishService, Uri, &Response);
>    if (EFI_ERROR (Status)) {
> -    DEBUG ((DEBUG_ERROR, "%a, get resource from: %a failed\n",
> __FUNCTION__, Uri));
> +    DEBUG ((DEBUG_ERROR, "%a, get resource from: %s failed\n",
> __FUNCTION__, Uri));
>      return Status;
>    }
> 
> @@ -58,7 +58,7 @@ RedfishResourceProvisioningResource (
>    Private->Payload = Response.Payload;
>    ASSERT (Private->Payload != NULL);
> 
> -  Status = RedfishProvisioningResourceCommon (Private, FALSE);
> +  Status = RedfishProvisioningResourceCommon (Private, !PostMode);
> 
>    //
>    // Release resource
> @@ -89,12 +89,13 @@ RedfishResourceProvisioningResource (
>  EFI_STATUS
>  RedfishResourceConsumeResource (
>    IN     EDKII_REDFISH_RESOURCE_CONFIG_PROTOCOL  *This,
> -  IN     CHAR8                                   *Uri
> +  IN     EFI_STRING                              Uri
>    )
>  {
>    REDFISH_RESOURCE_COMMON_PRIVATE  *Private;
>    EFI_STATUS                       Status;
>    REDFISH_RESPONSE                 Response;
> +  CHAR8                            *Etag;
> 
>    if ((This == NULL) || IS_EMPTY_STRING (Uri)) {
>      return EFI_INVALID_PARAMETER;
> @@ -106,9 +107,9 @@ RedfishResourceConsumeResource (
>      return EFI_NOT_READY;
>    }
> 
> -  Status = GetResourceByPath (Private->RedfishService, Uri, &Response);
> +  Status = GetResourceByUri (Private->RedfishService, Uri, &Response);
>    if (EFI_ERROR (Status)) {
> -    DEBUG ((DEBUG_ERROR, "%a, get resource from: %a failed\n",
> __FUNCTION__, Uri));
> +    DEBUG ((DEBUG_ERROR, "%a, get resource from: %s failed\n",
> __FUNCTION__, Uri));
>      return Status;
>    }
> 
> @@ -119,9 +120,27 @@ RedfishResourceConsumeResource (
>    Private->Json = JsonDumpString (RedfishJsonInPayload (Private->Payload),
> EDKII_JSON_COMPACT);
>    ASSERT (Private->Json != NULL);
> 
> -  Status = RedfishConsumeResourceCommon (Private, Private->Json);
> +  //
> +  // Find etag in HTTP response header
> +  //
> +  Etag   = NULL;
> +  Status = GetEtagAndLocation (&Response, &Etag, NULL);
>    if (EFI_ERROR (Status)) {
> -    DEBUG ((DEBUG_ERROR, "%a, failed to consume resource from: %a %r\n",
> __FUNCTION__, Uri, Status));
> +    DEBUG ((DEBUG_ERROR, "%a, failed to get ETag from HTTP header\n",
> __FUNCTION__));
> +  }
> +
> +  Status = RedfishConsumeResourceCommon (Private, Private->Json, Etag);
> +  if (EFI_ERROR (Status)) {
> +    if (Status != EFI_ALREADY_STARTED) {
> +      DEBUG ((DEBUG_ERROR, "%a, failed to consume resource from: %s:
> %r\n", __FUNCTION__, Uri, Status));
> +    }
> +  } else {
> +    //
> +    // Keep etag after consuming pending settings.
> +    //
> +    if (Etag != NULL) {
> +      SetEtagWithUri (Etag, Private->Uri);
> +    }
>    }
> 
>    //
> @@ -193,7 +212,7 @@ RedfishResourceGetInfo (
>  EFI_STATUS
>  RedfishResourceUpdate (
>    IN     EDKII_REDFISH_RESOURCE_CONFIG_PROTOCOL  *This,
> -  IN     CHAR8                                   *Uri
> +  IN     EFI_STRING                              Uri
>    )
>  {
>    REDFISH_RESOURCE_COMMON_PRIVATE  *Private;
> @@ -210,9 +229,9 @@ RedfishResourceUpdate (
>      return EFI_NOT_READY;
>    }
> 
> -  Status = GetResourceByPath (Private->RedfishService, Uri, &Response);
> +  Status = GetResourceByUri (Private->RedfishService, Uri, &Response);
>    if (EFI_ERROR (Status)) {
> -    DEBUG ((DEBUG_ERROR, "%a, get resource from: %a failed\n",
> __FUNCTION__, Uri));
> +    DEBUG ((DEBUG_ERROR, "%a, get resource from: %s failed\n",
> __FUNCTION__, Uri));
>      return Status;
>    }
> 
> @@ -225,7 +244,7 @@ RedfishResourceUpdate (
> 
>    Status = RedfishUpdateResourceCommon (Private, Private->Json);
>    if (EFI_ERROR (Status)) {
> -    DEBUG ((DEBUG_ERROR, "%a, failed to update resource from: %a %r\n",
> __FUNCTION__, Uri, Status));
> +    DEBUG ((DEBUG_ERROR, "%a, failed to update resource from: %s: %r\n",
> __FUNCTION__, Uri, Status));
>    }
> 
>    //
> @@ -262,7 +281,7 @@ RedfishResourceUpdate (
>  EFI_STATUS
>  RedfishResourceCheck (
>    IN     EDKII_REDFISH_RESOURCE_CONFIG_PROTOCOL  *This,
> -  IN     CHAR8                                   *Uri
> +  IN     EFI_STRING                              Uri
>    )
>  {
>    REDFISH_RESOURCE_COMMON_PRIVATE  *Private;
> @@ -279,9 +298,9 @@ RedfishResourceCheck (
>      return EFI_NOT_READY;
>    }
> 
> -  Status = GetResourceByPath (Private->RedfishService, Uri, &Response);
> +  Status = GetResourceByUri (Private->RedfishService, Uri, &Response);
>    if (EFI_ERROR (Status)) {
> -    DEBUG ((DEBUG_ERROR, "%a, get resource from: %a failed\n",
> __FUNCTION__, Uri));
> +    DEBUG ((DEBUG_ERROR, "%a, get resource from: %s failed\n",
> __FUNCTION__, Uri));
>      return Status;
>    }
> 
> @@ -294,7 +313,77 @@ RedfishResourceCheck (
> 
>    Status = RedfishCheckResourceCommon (Private, Private->Json);
>    if (EFI_ERROR (Status)) {
> -    DEBUG ((DEBUG_ERROR, "%a, failed to check resource from: %a %r\n",
> __FUNCTION__, Uri, Status));
> +    DEBUG ((DEBUG_ERROR, "%a, failed to check resource from: %s: %r\n",
> __FUNCTION__, Uri, Status));
> +  }
> +
> +  //
> +  // Release resource
> +  //
> +  if (Private->Payload != NULL) {
> +    RedfishFreeResponse (
> +      Response.StatusCode,
> +      Response.HeaderCount,
> +      Response.Headers,
> +      Response.Payload
> +      );
> +    Private->Payload = NULL;
> +  }
> +
> +  if (Private->Json != NULL) {
> +    FreePool (Private->Json);
> +    Private->Json = NULL;
> +  }
> +
> +  return Status;
> +}
> +
> +/**
> +  Identify resource on given URI.
> +
> +  @param[in]   This                Pointer to
> EDKII_REDFISH_RESOURCE_CONFIG_PROTOCOL instance.
> +  @param[in]   Uri                 The target URI to consume.
> +
> +  @retval EFI_SUCCESS              This is target resource which we want to
> handle.
> +  @retval EFI_UNSUPPORTED          This is not the target resource.
> +  @retval Others                   Some error happened.
> +
> +**/
> +EFI_STATUS
> +RedfishResourceIdentify (
> +  IN     EDKII_REDFISH_RESOURCE_CONFIG_PROTOCOL  *This,
> +  IN     EFI_STRING                              Uri
> +  )
> +{
> +  REDFISH_RESOURCE_COMMON_PRIVATE  *Private;
> +  EFI_STATUS                       Status;
> +  REDFISH_RESPONSE                 Response;
> +
> +  if ((This == NULL) || IS_EMPTY_STRING (Uri)) {
> +    return EFI_INVALID_PARAMETER;
> +  }
> +
> +  Private =
> REDFISH_RESOURCE_COMMON_PRIVATE_DATA_FROM_RESOURCE_PROTO
> COL (This);
> +
> +  if (Private->RedfishService == NULL) {
> +    return EFI_NOT_READY;
> +  }
> +
> +  Status = GetResourceByUri (Private->RedfishService, Uri, &Response);
> +  if (EFI_ERROR (Status)) {
> +    DEBUG ((DEBUG_ERROR, "%a, get resource from: %s failed\n",
> __FUNCTION__, Uri));
> +    return Status;
> +  }
> +
> +  Private->Uri     = Uri;
> +  Private->Payload = Response.Payload;
> +  ASSERT (Private->Payload != NULL);
> +
> +  Private->Json = JsonDumpString (RedfishJsonInPayload (Private->Payload),
> EDKII_JSON_COMPACT);
> +  ASSERT (Private->Json != NULL);
> +
> +  Status = RedfishIdentifyResourceCommon (Private, Private->Json);
> +  if (EFI_ERROR (Status)) {
> +    DEBUG ((DEBUG_ERROR, "%a, identify %s failed: %r\n", __FUNCTION__,
> Uri, Status));
>    }
> 
>    //
> @@ -323,6 +412,7 @@ EDKII_REDFISH_RESOURCE_CONFIG_PROTOCOL
> mRedfishResourceConfig = {
>    RedfishResourceConsumeResource,
>    RedfishResourceUpdate,
>    RedfishResourceCheck,
> +  RedfishResourceIdentify,
>    RedfishResourceGetInfo
>  };
> 
> @@ -523,6 +613,8 @@ RedfishResourceEntryPoint (
>      return EFI_ALREADY_STARTED;
>    }
> 
> +  medfishResourceConfigProtocolHandle = ImageHandle;
> +
>    mRedfishResourcePrivate = AllocateZeroPool (sizeof
> (REDFISH_RESOURCE_COMMON_PRIVATE));
>    CopyMem (&mRedfishResourcePrivate->ConfigHandler,
> &mRedfishConfigHandler, sizeof
> (EDKII_REDFISH_CONFIG_HANDLER_PROTOCOL));
>    CopyMem (&mRedfishResourcePrivate->RedfishResourceConfig,
> &mRedfishResourceConfig, sizeof
> (EDKII_REDFISH_RESOURCE_CONFIG_PROTOCOL));
> diff --git a/RedfishClientPkg/RedfishClient.fdf.inc
> b/RedfishClientPkg/RedfishClient.fdf.inc
> index 3619d2a2..5c4b9670 100644
> --- a/RedfishClientPkg/RedfishClient.fdf.inc
> +++ b/RedfishClientPkg/RedfishClient.fdf.inc
> @@ -14,7 +14,7 @@
>    INF RedfishClientPkg/RedfishFeatureCoreDxe/RedfishFeatureCoreDxe.inf
>    INF RedfishClientPkg/RedfishETagDxe/RedfishETagDxe.inf
>    INF
> RedfishClientPkg/RedfishConfigLangMapDxe/RedfishConfigLangMapDxe.inf
> -  INF
> RedfishClientPkg/Features/Memory/V1_7_1/Dxe/RedfishMemoryDxe.inf
> +  INF RedfishClientPkg/Features/Memory/V1_7_1/Dxe/MemoryDxe.inf
>    INF
> RedfishClientPkg/Features/MemoryCollectionDxe/MemoryCollectionDxe.inf
> 
>    !include RedfishClientPkg/RedfishJsonStructureDxe.fdf.inc
> --
> 2.17.1

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-05-10 14:06 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-05-10 13:04 [edk2-redfish-client][PATCH 2/5] RedfishClientPkg: Rename Memory feature driver Nickle Wang
2023-05-10 14:06 ` Chang, Abner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox