Branch: refs/heads/master
Home: https://github.com/tianocore/edk2
Commit: 917a7e3f347adb0721ff161a68309f51f621eec5
https://github.com/tianocore/edk2/commit/917a7e3f347adb0721ff161a68309f51f621eec5
Author: Michael Kubacki <[email protected]>
Date: 2022-09-09 (Fri, 09 Sep 2022)
Changed paths:
M ArmPlatformPkg/Drivers/NorFlashDxe/NorFlashBlockIoDxe.c
Log Message:
-----------
ArmPlatformPkg/NorFlashDxe: Remove unused debug print specifier
These debug messages are repeated in both NorFlashBlockIoReadBlocks()
and NorFlashBlockIoWriteBlocks():
"NorFlashBlockIoWriteBlocks(MediaId=0x%x, Lba=%ld, BufferSize=0x%x"
"bytes (%d kB), BufferPtr @ 0x%08x)\n"
Although this requires 5 arguments, only 4 are provided. The kilobyte
value was never given.
This change removes that specifier so the 4 arguments match the 4
specifiers in the debug macro.
Cc: Leif Lindholm <[email protected]>
Cc: Ard Biesheuvel <[email protected]>
Signed-off-by: Michael Kubacki <[email protected]>
Reviewed-by: Ard Biesheuvel <[email protected]>
Reviewed-by: Leif Lindholm <[email protected]>
Commit: b4036b52b18b215973a1d388c37b5affd55679cc
https://github.com/tianocore/edk2/commit/b4036b52b18b215973a1d388c37b5affd55679cc
Author: Michael Kubacki <[email protected]>
Date: 2022-09-09 (Fri, 09 Sep 2022)
Changed paths:
M FatPkg/FatPei/Gpt.c
Log Message:
-----------
FatPkg/FatPei: Remove extraneous debug message argument
This debug macro should take one argument based on the number of
print specifiers defined. However, two arguments are given.
It looks like the code may have been refactored such that the
second argument was moved to a new print and this argument was
not removed. In any case, it should not be there now.
Cc: Ray Ni <[email protected]>
Signed-off-by: Michael Kubacki <[email protected]>
Reviewed-by: Michael D Kinney <[email protected]>
Commit: 1096a9b04bbf7bf2319210060bfc1afaac943235
https://github.com/tianocore/edk2/commit/1096a9b04bbf7bf2319210060bfc1afaac943235
Author: Michael Kubacki <[email protected]>
Date: 2022-09-09 (Fri, 09 Sep 2022)
Changed paths:
M MdeModulePkg/Bus/Pci/NvmExpressPei/NvmExpressPei.c
M MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdMmcPciHci.c
M MdeModulePkg/Bus/Usb/UsbBusDxe/UsbEnumer.c
M MdeModulePkg/Core/Dxe/Image/Image.c
M
MdeModulePkg/Library/DisplayUpdateProgressLibGraphics/DisplayUpdateProgressLibGraphics.c
M MdeModulePkg/Universal/CapsulePei/UefiCapsule.c
Log Message:
-----------
MdeModulePkg: Fix imbalanced debug macros
Updates debug macros in the package that have an imbalanced number
of print specifiers to arguments. These changes try to preserve
what was likely intended by the author. In cases information was
missing due to the bug, the specifier may be removed since it was
not previously accurately printing the expected value.
Cc: Dandan Bi <[email protected]>
Cc: Guomin Jiang <[email protected]>
Cc: Hao A Wu <[email protected]>
Cc: Jian J Wang <[email protected]>
Cc: Liming Gao <[email protected]>
Cc: Liming Gao <[email protected]>
Cc: Ray Ni <[email protected]>
Signed-off-by: Michael Kubacki <[email protected]>
Reviewed-by: Hao A Wu <[email protected]>
Reviewed-by: Liming Gao <[email protected]>
Commit: c403de7bd48ed4b19965dfdda36dff33886021a3
https://github.com/tianocore/edk2/commit/c403de7bd48ed4b19965dfdda36dff33886021a3
Author: Michael Kubacki <[email protected]>
Date: 2022-09-09 (Fri, 09 Sep 2022)
Changed paths:
M RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c
M RedfishPkg/RedfishHostInterfaceDxe/RedfishHostInterfaceDxe.c
M RedfishPkg/RedfishRestExDxe/RedfishRestExProtocol.c
Log Message:
-----------
RedfishPkg/RedfishRestExDxe: Remove extra debug macro argument
The debug macro argument in this change is removed since it does
have a corresponding print specifier in the debug message string.
Cc: Abner Chang <[email protected]>
Cc: Nickle Wang <[email protected]>
Signed-off-by: Michael Kubacki <[email protected]>
Reviewed-by: Abner Chang <[email protected]>
Reviewed-by: Nickle Wang <[email protected]>
Commit: e495b1009a8ed118ab49f5020a77afc9b6d8f3f9
https://github.com/tianocore/edk2/commit/e495b1009a8ed118ab49f5020a77afc9b6d8f3f9
Author: Michael Kubacki <[email protected]>
Date: 2022-09-09 (Fri, 09 Sep 2022)
Changed paths:
M
SecurityPkg/Library/SmmTcg2PhysicalPresenceLib/MmTcg2PhysicalPresenceLibCommon.c
Log Message:
-----------
SecurityPkg/SmmTcg2PhysicalPresenceLib: Add missing debug print specifier
The debug macro modified in this change was missing a print specifier
for a debug message argument given.
Cc: Jiewen Yao <[email protected]>
Cc: Jian J Wang <[email protected]>
Cc: Qi Zhang <[email protected]>
Cc: Rahul Kumar <[email protected]>
Signed-off-by: Michael Kubacki <[email protected]>
Reviewed-by: Jiewen Yao <[email protected]>
Commit: e8a537d28d37c092bd03093064264071f2938ca8
https://github.com/tianocore/edk2/commit/e8a537d28d37c092bd03093064264071f2938ca8
Author: Michael Kubacki <[email protected]>
Date: 2022-09-09 (Fri, 09 Sep 2022)
Changed paths:
M DynamicTablesPkg/Library/Acpi/Arm/AcpiPpttLibArm/PpttGenerator.c
Log Message:
-----------
DynamicTablesPkg/AcpiPpttLibArm: Fix debug macro arguments
Cc: Sami Mujawar <[email protected]>
Cc: Alexei Fedorov <[email protected]>
Signed-off-by: Michael Kubacki <[email protected]>
Reviewed-by: Sami Mujawar <[email protected]>
Commit: 8fdb4de628120a8736ee47522d1941e3ad3bdf8a
https://github.com/tianocore/edk2/commit/8fdb4de628120a8736ee47522d1941e3ad3bdf8a
Author: Michael Kubacki <[email protected]>
Date: 2022-09-09 (Fri, 09 Sep 2022)
Changed paths:
M NetworkPkg/TcpDxe/SockInterface.c
Log Message:
-----------
NetworkPkg/TcpDxe: Fix debug macro arguments
Removes Status argument that is not needed from DEBUG macros.
Cc: Maciej Rabeda <[email protected]>
Cc: Jiaxin Wu <[email protected]>
Cc: Siyuan Fu <[email protected]>
Signed-off-by: Michael Kubacki <[email protected]>
Reviewed-by: Maciej Rabeda <[email protected]>
Commit: cdb80a281fa3fd3457faf1dc65d5a443e8d512d3
https://github.com/tianocore/edk2/commit/cdb80a281fa3fd3457faf1dc65d5a443e8d512d3
Author: Michael Kubacki <[email protected]>
Date: 2022-09-09 (Fri, 09 Sep 2022)
Changed paths:
M OvmfPkg/Csm/LegacyBootManagerLib/LegacyBm.c
Log Message:
-----------
OvmfPkg/LegacyBootManagerLib: Fix debug macro arguments
The DEBUG macro updated in this patch previously contained 11 print
specifiers in the debug string but passeed 13 arguments. This change
attempts to update the macro to the author's intention so the number
of specifiers match the number of arguments.
Cc: Ard Biesheuvel <[email protected]>
Cc: Jiewen Yao <[email protected]>
Cc: Jordan Justen <[email protected]>
Cc: Gerd Hoffmann <[email protected]>
Cc: David Woodhouse <[email protected]>
Signed-off-by: Michael Kubacki <[email protected]>
Reviewed-by: Ard Biesheuvel <[email protected]>
Acked-by: Gerd Hoffmann <[email protected]>
Compare: https://github.com/tianocore/edk2/compare/7b8f69d7e106...cdb80a281fa3
_______________________________________________
edk2-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-commits
