Re: [edk2-devel] [PATCH v2 1/7] RedfishPkg/PlatformHostInterfaceBmcUsbNicLib: Fix DEBUG macro args

["Chang, Abner via groups.io" <abner.chang=amd.com@groups.io>]

[AMD Official Use Only - General]

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

> -----Original Message-----
> From: mikuback@linux.microsoft.com <mikuback@linux.microsoft.com>
> Sent: Thursday, September 14, 2023 1:06 AM
> To: devel@edk2.groups.io
> Cc: Chang, Abner <Abner.Chang@amd.com>; Nickle Wang
> <nicklew@nvidia.com>; Igor Kulchytskyy <igork@ami.com>
> Subject: [PATCH v2 1/7] RedfishPkg/PlatformHostInterfaceBmcUsbNicLib: Fix
> DEBUG macro args
>
> Caution: This message originated from an External Source. Use proper caution
> when opening attachments, clicking links, or responding.
>
>
> From: Michael Kubacki <michael.kubacki@microsoft.com>
>
> Some macros added have a mismatched number of  print specifiers to
> arguments.
>
> Cc: Abner Chang <abner.chang@amd.com>
> Cc: Nickle Wang <nicklew@nvidia.com>
> Cc: Igor Kulchytskyy <igork@ami.com>
> Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
> ---
>
> RedfishPkg/Library/PlatformHostInterfaceBmcUsbNicLib/PlatformHostInterfa
> ceBmcUsbNicLib.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git
> a/RedfishPkg/Library/PlatformHostInterfaceBmcUsbNicLib/PlatformHostInter
> faceBmcUsbNicLib.c
> b/RedfishPkg/Library/PlatformHostInterfaceBmcUsbNicLib/PlatformHostInter
> faceBmcUsbNicLib.c
> index d18d83b93808..95900579118b 100644
> ---
> a/RedfishPkg/Library/PlatformHostInterfaceBmcUsbNicLib/PlatformHostInter
> faceBmcUsbNicLib.c
> +++
> b/RedfishPkg/Library/PlatformHostInterfaceBmcUsbNicLib/PlatformHostInter
> faceBmcUsbNicLib.c
> @@ -65,10 +65,10 @@ ProbeRedfishCredentialBootstrap (
>         (ResponseData.CompletionCode ==
> REDFISH_IPMI_COMP_CODE_BOOTSTRAP_CREDENTIAL_DISABLED)
>        ))
>    {
> -    DEBUG ((DEBUG_REDFISH_HOST_INTERFACE, "    Redfish Credentail
> Bootstrapping is supported\n", __func__));
> +    DEBUG ((DEBUG_REDFISH_HOST_INTERFACE, "    Redfish Credential
> Bootstrapping is supported\n"));
>      ReturnBool = TRUE;
>    } else {
> -    DEBUG ((DEBUG_REDFISH_HOST_INTERFACE, "    Redfish Credentail
> Bootstrapping is not supported\n", __func__));
> +    DEBUG ((DEBUG_REDFISH_HOST_INTERFACE, "    Redfish Credential
> Bootstrapping is not supported\n"));
>      ReturnBool = FALSE;
>    }
>
> @@ -645,7 +645,7 @@ HostInterfaceIpmiCheckMacAddress (
>                                                              &ResponseDataSize
>                                                              );
>        if (EFI_ERROR (Status)) {
> -        DEBUG ((DEBUG_ERROR, "    - Fails to send command.\n", ChannelNum));
> +        DEBUG ((DEBUG_ERROR, "    - Channel %d fails to send command.\n",
> ChannelNum));
>          continue;
>        }
>
> @@ -1084,7 +1084,7 @@ CheckBmcUsbNicOnHandles (
>                      (VOID **)&DevicePath
>                      );
>      if (EFI_ERROR (Status)) {
> -      DEBUG ((DEBUG_ERROR, "    Failed to locate SNP on %d handle.\n",
> __func__, Index));
> +      DEBUG ((DEBUG_ERROR, "    Failed to locate SNP on %d handle.\n",
> Index));
>        continue;
>      }
>
> --
> 2.42.0.windows.2



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#108631): https://edk2.groups.io/g/devel/message/108631
Mute This Topic: https://groups.io/mt/101341647/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-