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

Reply via email to