From: Michael Kubacki <michael.kuba...@microsoft.com> After noticing a few occurrences of DEBUG macros with a mismatched number of arguments, I wrote a script to automatically detect this.
This patch series is the result of matches found by the script. In some cases it is not obvious what was originally intended so I have attempted to capture what I believe was the original author's intention. V2 Changes: These additional errors were found while finishing testing of the script. 1. Add DynamicTablesPkg/AcpiPpttLibArm patch for the change in - DynamicTablesPkg/Library/Acpi/Arm/AcpiPpttLibArm/PpttGenerator.c 2. Update MdeModulePkg patch to include the change in - MdeModulePkg/Universal/CapsulePei/UefiCapsule.c 3. Add following to the RedfishPkg fix patch: - RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c - RedfishPkg/RedfishHostInterfaceDxe/RedfishHostInterfaceDxe.c 4. Add NetworkPkg patch for the change in - NetworkPkg/TcpDxe/SockInterface.c 5. Add OvmfPkg patch for the change in - OvmfPkg/Csm/LegacyBootManagerLib/LegacyBm.c Cc: Abner Chang <abner.ch...@amd.com> Cc: Alexei Fedorov <alexei.fedo...@arm.com> Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org> Cc: Dandan Bi <dandan...@intel.com> Cc: David Woodhouse <dw...@infradead.org> Cc: Gerd Hoffmann <kra...@redhat.com> Cc: Guomin Jiang <guomin.ji...@intel.com> Cc: Hao A Wu <hao.a...@intel.com> Cc: Jian J Wang <jian.j.w...@intel.com> Cc: Jiaxin Wu <jiaxin...@intel.com> Cc: Jiewen Yao <jiewen....@intel.com> Cc: Jordan Justen <jordan.l.jus...@intel.com> Cc: Leif Lindholm <quic_llind...@quicinc.com> Cc: Liming Gao <gaolim...@byosoft.com.cn> Cc: Maciej Rabeda <maciej.rab...@linux.intel.com> Cc: Nickle Wang <nickle.w...@hpe.com> Cc: Qi Zhang <qi1.zh...@intel.com> Cc: Rahul Kumar <rahul1.ku...@intel.com> Cc: Ray Ni <ray...@intel.com> Cc: Sami Mujawar <sami.muja...@arm.com> Cc: Siyuan Fu <siyuan...@intel.com> Signed-off-by: Michael Kubacki <michael.kuba...@microsoft.com> Michael Kubacki (8): ArmPlatformPkg/NorFlashDxe: Remove unused debug print specifier FatPkg/FatPei: Remove extraneous debug message argument MdeModulePkg: Fix imbalanced debug macros RedfishPkg/RedfishRestExDxe: Remove extra debug macro argument SecurityPkg/SmmTcg2PhysicalPresenceLib: Add missing debug print specifier DynamicTablesPkg/AcpiPpttLibArm: Fix debug macro arguments NetworkPkg/TcpDxe: Fix debug macro arguments OvmfPkg/LegacyBootManagerLib: Fix debug macro arguments ArmPlatformPkg/Drivers/NorFlashDxe/NorFlashBlockIoDxe.c | 4 ++-- DynamicTablesPkg/Library/Acpi/Arm/AcpiPpttLibArm/PpttGenerator.c | 3 +-- FatPkg/FatPei/Gpt.c | 2 +- MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHci.c | 2 +- MdeModulePkg/Bus/Usb/UsbBusDxe/UsbEnumer.c | 8 ++++---- MdeModulePkg/Core/Dxe/Image/Image.c | 2 +- MdeModulePkg/Library/DisplayUpdateProgressLibGraphics/DisplayUpdateProgressLibGraphics.c | 2 +- MdeModulePkg/Universal/CapsulePei/UefiCapsule.c | 2 +- NetworkPkg/TcpDxe/SockInterface.c | 12 ++---------- OvmfPkg/Csm/LegacyBootManagerLib/LegacyBm.c | 6 +++--- RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c | 6 +++--- RedfishPkg/RedfishHostInterfaceDxe/RedfishHostInterfaceDxe.c | 2 +- RedfishPkg/RedfishRestExDxe/RedfishRestExProtocol.c | 2 +- SecurityPkg/Library/SmmTcg2PhysicalPresenceLib/MmTcg2PhysicalPresenceLibCommon.c | 2 +- 14 files changed, 23 insertions(+), 32 deletions(-) -- 2.28.0.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#92786): https://edk2.groups.io/g/devel/message/92786 Mute This Topic: https://groups.io/mt/93242243/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-