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+tianocore@kernel.org>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Jordan Justen <jordan.l.justen@intel.com>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Cc: David Woodhouse <dwmw2@infradead.org>
Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
Reviewed-by: Ard Biesheuvel <ardb@kernel.org>
Acked-by: Gerd Hoffmann <kraxel@redhat.com>

diff --git a/OvmfPkg/Csm/LegacyBootManagerLib/LegacyBm.c b/OvmfPkg/Csm/LegacyBootManagerLib/LegacyBm.c
index 032aacba68de404a2f835f3edc69609a66cd5262..2e92bce726a9b39e4e2c26114493d38ac8659033 100644 (file)
--- a/OvmfPkg/Csm/LegacyBootManagerLib/LegacyBm.c
+++ b/OvmfPkg/Csm/LegacyBootManagerLib/LegacyBm.c
@@ -1159,8 +1159,8 @@ LegacyBmPrintBbsTable (
   UINT16  Index;\r
 \r
   DEBUG ((DEBUG_INFO, "\n"));\r
-  DEBUG ((DEBUG_INFO, " NO  Prio bb/dd/ff cl/sc Type Stat segm:offs\n"));\r
-  DEBUG ((DEBUG_INFO, "=============================================\n"));\r
+  DEBUG ((DEBUG_INFO, " NO  Prio bb/dd/ff cl/sc Type Stat segm:offs mseg dseg\n"));\r
+  DEBUG ((DEBUG_INFO, "======================================================\n"));\r
   for (Index = 0; Index < BbsCount; Index++) {\r
     if (!LegacyBmValidBbsEntry (&LocalBbsTable[Index])) {\r
       continue;\r
@@ -1168,7 +1168,7 @@ LegacyBmPrintBbsTable (
 \r
     DEBUG (\r
       (DEBUG_INFO,\r
-       " %02x: %04x %02x/%02x/%02x %02x/%02x %04x %04x %04x:%04x\n",\r
+       " %02x: %04x %02x/%02x/%02x %02x/%02x %04x %04x %04x:%04x %04x %04x\n",\r
        (UINTN)Index,\r
        (UINTN)LocalBbsTable[Index].BootPriority,\r
        (UINTN)LocalBbsTable[Index].Bus,\r