From: Michael Kubacki <[email protected]> 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. V3 Changes: Updated R-b tags. 1. ArmPlatformPkg/Drivers/NorFlashDxe/NorFlashBlockIoDxe.c: - Repeated BufferSizeInBytes argument per Leif's feedback 2. MdeModulePkg/Bus/Pci/NvmExpressPei/NvmExpressPei.c: - Fixed an additional error on Line 267 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 <[email protected]> Cc: Alexei Fedorov <[email protected]> Cc: Ard Biesheuvel <[email protected]> Cc: Dandan Bi <[email protected]> Cc: David Woodhouse <[email protected]> Cc: Gerd Hoffmann <[email protected]> Cc: Guomin Jiang <[email protected]> Cc: Hao A Wu <[email protected]> Cc: Jian J Wang <[email protected]> Cc: Jiaxin Wu <[email protected]> Cc: Jiewen Yao <[email protected]> Cc: Jordan Justen <[email protected]> Cc: Leif Lindholm <[email protected]> Cc: Liming Gao <[email protected]> Cc: Maciej Rabeda <[email protected]> Cc: Nickle Wang <[email protected]> Cc: Qi Zhang <[email protected]> Cc: Rahul Kumar <[email protected]> Cc: Ray Ni <[email protected]> Cc: Sami Mujawar <[email protected]> Cc: Siyuan Fu <[email protected]> Signed-off-by: Michael Kubacki <[email protected]> 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/NvmExpressPei/NvmExpressPei.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 +- 15 files changed, 24 insertions(+), 33 deletions(-) -- 2.28.0.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#93104): https://edk2.groups.io/g/devel/message/93104 Mute This Topic: https://groups.io/mt/93429907/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
