Branch: refs/heads/master Home: https://github.com/tianocore/edk2 Commit: 917a7e3f347adb0721ff161a68309f51f621eec5 https://github.com/tianocore/edk2/commit/917a7e3f347adb0721ff161a68309f51f621eec5 Author: Michael Kubacki <michael.kuba...@microsoft.com> 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 <quic_llind...@quicinc.com> Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org> Signed-off-by: Michael Kubacki <michael.kuba...@microsoft.com> Reviewed-by: Ard Biesheuvel <a...@kernel.org> Reviewed-by: Leif Lindholm <quic_llind...@quicinc.com> Commit: b4036b52b18b215973a1d388c37b5affd55679cc https://github.com/tianocore/edk2/commit/b4036b52b18b215973a1d388c37b5affd55679cc Author: Michael Kubacki <michael.kuba...@microsoft.com> 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 <ray...@intel.com> Signed-off-by: Michael Kubacki <michael.kuba...@microsoft.com> Reviewed-by: Michael D Kinney <michael.d.kin...@intel.com> Commit: 1096a9b04bbf7bf2319210060bfc1afaac943235 https://github.com/tianocore/edk2/commit/1096a9b04bbf7bf2319210060bfc1afaac943235 Author: Michael Kubacki <michael.kuba...@microsoft.com> 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 <dandan...@intel.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: Liming Gao <gaolim...@byosoft.com.cn> Cc: Liming Gao <gaolim...@byosoft.com.cn> Cc: Ray Ni <ray...@intel.com> Signed-off-by: Michael Kubacki <michael.kuba...@microsoft.com> Reviewed-by: Hao A Wu <hao.a...@intel.com> Reviewed-by: Liming Gao <gaolim...@byosoft.com.cn> Commit: c403de7bd48ed4b19965dfdda36dff33886021a3 https://github.com/tianocore/edk2/commit/c403de7bd48ed4b19965dfdda36dff33886021a3 Author: Michael Kubacki <michael.kuba...@microsoft.com> 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 <abner.ch...@amd.com> Cc: Nickle Wang <nic...@csie.io> Signed-off-by: Michael Kubacki <michael.kuba...@microsoft.com> Reviewed-by: Abner Chang <abner.ch...@amd.com> Reviewed-by: Nickle Wang <nic...@csie.io> Commit: e495b1009a8ed118ab49f5020a77afc9b6d8f3f9 https://github.com/tianocore/edk2/commit/e495b1009a8ed118ab49f5020a77afc9b6d8f3f9 Author: Michael Kubacki <michael.kuba...@microsoft.com> 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 <jiewen....@intel.com> Cc: Jian J Wang <jian.j.w...@intel.com> Cc: Qi Zhang <qi1.zh...@intel.com> Cc: Rahul Kumar <rahul1.ku...@intel.com> Signed-off-by: Michael Kubacki <michael.kuba...@microsoft.com> Reviewed-by: Jiewen Yao <jiewen....@intel.com> Commit: e8a537d28d37c092bd03093064264071f2938ca8 https://github.com/tianocore/edk2/commit/e8a537d28d37c092bd03093064264071f2938ca8 Author: Michael Kubacki <michael.kuba...@microsoft.com> 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 <sami.muja...@arm.com> Cc: Alexei Fedorov <alexei.fedo...@arm.com> Signed-off-by: Michael Kubacki <michael.kuba...@microsoft.com> Reviewed-by: Sami Mujawar <sami.muja...@arm.com> Commit: 8fdb4de628120a8736ee47522d1941e3ad3bdf8a https://github.com/tianocore/edk2/commit/8fdb4de628120a8736ee47522d1941e3ad3bdf8a Author: Michael Kubacki <michael.kuba...@microsoft.com> 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 <maciej.rab...@linux.intel.com> Cc: Jiaxin Wu <jiaxin...@intel.com> Cc: Siyuan Fu <siyuan...@intel.com> Signed-off-by: Michael Kubacki <michael.kuba...@microsoft.com> Reviewed-by: Maciej Rabeda <maciej.rab...@linux.intel.com> Commit: cdb80a281fa3fd3457faf1dc65d5a443e8d512d3 https://github.com/tianocore/edk2/commit/cdb80a281fa3fd3457faf1dc65d5a443e8d512d3 Author: Michael Kubacki <michael.kuba...@microsoft.com> 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 <ardb+tianoc...@kernel.org> Cc: Jiewen Yao <jiewen....@intel.com> Cc: Jordan Justen <jordan.l.jus...@intel.com> Cc: Gerd Hoffmann <kra...@redhat.com> Cc: David Woodhouse <dw...@infradead.org> Signed-off-by: Michael Kubacki <michael.kuba...@microsoft.com> Reviewed-by: Ard Biesheuvel <a...@kernel.org> Acked-by: Gerd Hoffmann <kra...@redhat.com> Compare: https://github.com/tianocore/edk2/compare/7b8f69d7e106...cdb80a281fa3 _______________________________________________ edk2-commits mailing list edk2-commits@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-commits