[AMD Official Use Only - General]

Reviewed-by: Abner Chang <abner.ch...@amd.com>

> -----Original Message-----
> From: Nickle Wang <nick...@nvidia.com>
> Sent: Wednesday, July 5, 2023 2:57 PM
> To: devel@edk2.groups.io
> Cc: Chang, Abner <abner.ch...@amd.com>; Igor Kulchytskyy
> <ig...@ami.com>; Nick Ramirez <nrami...@nvidia.com>
> Subject: [PATCH v2 2/3] RedfishPkg/RedfishPlatformConfigDxe: hide debug
> message
>
> Caution: This message originated from an External Source. Use proper caution
> when opening attachments, clicking links, or responding.
>
>
> Change debug message level of showing ordered list op-code
> to REDFISH_PLATFORM_CONFIG_DEBUG.
>
> Signed-off-by: Nickle Wang <nick...@nvidia.com>
> Cc: Abner Chang <abner.ch...@amd.com>
> Cc: Igor Kulchytskyy <ig...@ami.com>
> Cc: Nick Ramirez <nrami...@nvidia.com>
> ---
>  .../RedfishPlatformConfigDxe.h                |  2 +-
>  .../RedfishPlatformConfigDxe.c                | 24 +++++++++----------
>  2 files changed, 13 insertions(+), 13 deletions(-)
>
> diff --git
> a/RedfishPkg/RedfishPlatformConfigDxe/RedfishPlatformConfigDxe.h
> b/RedfishPkg/RedfishPlatformConfigDxe/RedfishPlatformConfigDxe.h
> index c86bc6e9ce2d..38adea04c5bf 100644
> --- a/RedfishPkg/RedfishPlatformConfigDxe/RedfishPlatformConfigDxe.h
> +++ b/RedfishPkg/RedfishPlatformConfigDxe/RedfishPlatformConfigDxe.h
> @@ -75,7 +75,7 @@ typedef struct {
>  #define REGULAR_EXPRESSION_INCLUDE_ALL   L".*"
>  #define CONFIGURE_LANGUAGE_PREFIX        "x-uefi-redfish-"
>  #define REDFISH_PLATFORM_CONFIG_VERSION  0x00010000
> -#define REDFISH_PLATFORM_CONFIG_DEBUG    DEBUG_VERBOSE
> +#define REDFISH_PLATFORM_CONFIG_DEBUG    DEBUG_MANAGEABILITY
>  #define REDFISH_MENU_PATH_SIZE           8
>
>  /**
> diff --git a/RedfishPkg/RedfishPlatformConfigDxe/RedfishPlatformConfigDxe.c
> b/RedfishPkg/RedfishPlatformConfigDxe/RedfishPlatformConfigDxe.c
> index 462f269f6a3f..f2a8e77d9b00 100644
> --- a/RedfishPkg/RedfishPlatformConfigDxe/RedfishPlatformConfigDxe.c
> +++ b/RedfishPkg/RedfishPlatformConfigDxe/RedfishPlatformConfigDxe.c
> @@ -955,12 +955,12 @@ DumpOrderedListValue (
>      return;
>    }
>
> -  DEBUG ((DEBUG_ERROR, "Value.Type= 0x%x\n", OrderedListStatement-
> >Value.Type));
> -  DEBUG ((DEBUG_ERROR, "Value.BufferValueType= 0x%x\n",
> OrderedListStatement->Value.BufferValueType));
> -  DEBUG ((DEBUG_ERROR, "Value.BufferLen= 0x%x\n",
> OrderedListStatement->Value.BufferLen));
> -  DEBUG ((DEBUG_ERROR, "Value.Buffer= 0x%x\n", OrderedListStatement-
> >Value.Buffer));
> -  DEBUG ((DEBUG_ERROR, "Value.MaxContainers= 0x%x\n",
> OrderedListStatement->ExtraData.OrderListData.MaxContainers));
> -  DEBUG ((DEBUG_ERROR, "StorageWidth= 0x%x\n", OrderedListStatement-
> >StorageWidth));
> +  DEBUG ((REDFISH_PLATFORM_CONFIG_DEBUG, "Value.Type= 0x%x\n",
> OrderedListStatement->Value.Type));
> +  DEBUG ((REDFISH_PLATFORM_CONFIG_DEBUG, "Value.BufferValueType=
> 0x%x\n", OrderedListStatement->Value.BufferValueType));
> +  DEBUG ((REDFISH_PLATFORM_CONFIG_DEBUG, "Value.BufferLen=
> 0x%x\n", OrderedListStatement->Value.BufferLen));
> +  DEBUG ((REDFISH_PLATFORM_CONFIG_DEBUG, "Value.Buffer= 0x%x\n",
> OrderedListStatement->Value.Buffer));
> +  DEBUG ((REDFISH_PLATFORM_CONFIG_DEBUG, "Value.MaxContainers=
> 0x%x\n", OrderedListStatement->ExtraData.OrderListData.MaxContainers));
> +  DEBUG ((REDFISH_PLATFORM_CONFIG_DEBUG, "StorageWidth= 0x%x\n",
> OrderedListStatement->StorageWidth));
>
>    if (OrderedListStatement->Value.Buffer == NULL) {
>      return;
> @@ -977,7 +977,7 @@ DumpOrderedListValue (
>        Value8 = (UINT8 *)OrderedListStatement->Value.Buffer;
>        Count  = OrderedListStatement->StorageWidth / sizeof (UINT8);
>        for (Index = 0; Index < Count; Index++) {
> -        DEBUG ((DEBUG_ERROR, "%d ", Value8[Index]));
> +        DEBUG ((REDFISH_PLATFORM_CONFIG_DEBUG, "%d ", Value8[Index]));
>        }
>
>        break;
> @@ -985,7 +985,7 @@ DumpOrderedListValue (
>        Value16 = (UINT16 *)OrderedListStatement->Value.Buffer;
>        Count   = OrderedListStatement->StorageWidth / sizeof (UINT16);
>        for (Index = 0; Index < Count; Index++) {
> -        DEBUG ((DEBUG_ERROR, "%d ", Value16[Index]));
> +        DEBUG ((REDFISH_PLATFORM_CONFIG_DEBUG, "%d ",
> Value16[Index]));
>        }
>
>        break;
> @@ -993,7 +993,7 @@ DumpOrderedListValue (
>        Value32 = (UINT32 *)OrderedListStatement->Value.Buffer;
>        Count   = OrderedListStatement->StorageWidth / sizeof (UINT32);
>        for (Index = 0; Index < Count; Index++) {
> -        DEBUG ((DEBUG_ERROR, "%d ", Value32[Index]));
> +        DEBUG ((REDFISH_PLATFORM_CONFIG_DEBUG, "%d ",
> Value32[Index]));
>        }
>
>        break;
> @@ -1001,7 +1001,7 @@ DumpOrderedListValue (
>        Value64 = (UINT64 *)OrderedListStatement->Value.Buffer;
>        Count   = OrderedListStatement->StorageWidth / sizeof (UINT64);
>        for (Index = 0; Index < Count; Index++) {
> -        DEBUG ((DEBUG_ERROR, "%d ", Value64[Index]));
> +        DEBUG ((REDFISH_PLATFORM_CONFIG_DEBUG, "%d ",
> Value64[Index]));
>        }
>
>        break;
> @@ -1009,13 +1009,13 @@ DumpOrderedListValue (
>        Value8 = (UINT8 *)OrderedListStatement->Value.Buffer;
>        Count  = OrderedListStatement->StorageWidth / sizeof (UINT8);
>        for (Index = 0; Index < Count; Index++) {
> -        DEBUG ((DEBUG_ERROR, "%d ", Value8[Index]));
> +        DEBUG ((REDFISH_PLATFORM_CONFIG_DEBUG, "%d ", Value8[Index]));
>        }
>
>        break;
>    }
>
> -  DEBUG ((DEBUG_ERROR, "\n"));
> +  DEBUG ((REDFISH_PLATFORM_CONFIG_DEBUG, "\n"));
>  }
>
>  /**
> --
> 2.17.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#106695): https://edk2.groups.io/g/devel/message/106695
Mute This Topic: https://groups.io/mt/99960363/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to