Re: [edk2-devel] [edk2-platforms][PATCH v2 0/3] Silicon/Intel: Fix invalid DEBUG() macros

Re: [edk2-devel] [edk2-platforms][PATCH v2 0/3] Silicon/Intel: Fix invalid DEBUG() macros

[Michael Kubacki]

Following up on this series. I addressed Nate's v1 feedback.

Thanks,
Michael

On 10/12/2022 10:16 AM, Michael Kubacki wrote:
> From: Michael Kubacki <michael.kubacki@microsoft.com>
> 
> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4095
> 
> Fixes several debug macros in Silicon/Intel that have a mismatched
> number of print specifiers to arguments.
> 
> The original author's intention is not always 100% obvious. Though,
> this series is relatively straightforward.
> 
> v2 changes:
>    - Applied debug macro suggestions in Nate's feedback
> 
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Heng Luo <heng.luo@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Sai Chaganty <rangasai.v.chaganty@intel.com>
> Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
> 
> Michael Kubacki (3):
>    CoffeelakeSiliconPkg: Fix invalid debug macros
>    KabylakeSiliconPkg: Fix invalid debug macros
>    TigerlakeSiliconPkg: Fix invalid debug macros
> 
>   Silicon/Intel/CoffeelakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/CpuPrintPolicy.c                                       |  2 +-
>   Silicon/Intel/CoffeelakeSiliconPkg/Pch/Library/PeiDxeSmmGbeMdiLib/GbeMdiLib.c                                         |  2 +-
>   Silicon/Intel/CoffeelakeSiliconPkg/Pch/Library/PeiOcWdtLib/PeiOcWdtLib.c                                              |  4 ++--
>   Silicon/Intel/CoffeelakeSiliconPkg/Pch/Library/Private/PeiDxeSmmPchPciExpressHelpersLib/PchPciExpressHelpersLibrary.c |  2 +-
>   Silicon/Intel/KabylakeSiliconPkg/Cpu/Library/PeiCpuPolicyLib/CpuPrintPolicy.c                                         | 19 +++++++++++--------
>   Silicon/Intel/KabylakeSiliconPkg/Pch/Library/PeiOcWdtLib/PeiOcWdtLib.c                                                |  4 ++--
>   Silicon/Intel/TigerlakeSiliconPkg/IpBlock/Gbe/LibraryPrivate/PeiDxeSmmGbeMdiLib/GbeMdiLib.c                           |  2 +-
>   Silicon/Intel/TigerlakeSiliconPkg/IpBlock/PcieRp/LibraryPrivate/PciExpressHelpersLibrary/PciExpressHelpersLibrary.c   |  2 +-
>   8 files changed, 20 insertions(+), 17 deletions(-)
>