From deba54761ab1e2796a7e219f371f035a84c3a830 Mon Sep 17 00:00:00 2001 From: Michael D Kinney Date: Sun, 5 Dec 2021 14:33:56 -0800 Subject: [PATCH] PcAtChipsetPkg: Change complex DEBUG_CODE() to DEBUG_CODE_BEGIN/END() REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3767 Update use of DEBUG_CODE(Expression) if Expression is a complex code block with if/while/for/case statements that use {}. Cc: Andrew Fish Cc: Leif Lindholm Cc: Michael Kubacki Signed-off-by: Michael D Kinney Reviewed-by: Ray Ni --- PcAtChipsetPkg/HpetTimerDxe/HpetTimer.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/PcAtChipsetPkg/HpetTimerDxe/HpetTimer.c b/PcAtChipsetPkg/HpetTimerDxe/HpetTimer.c index cbe986ebfd..3adc6d2365 100644 --- a/PcAtChipsetPkg/HpetTimerDxe/HpetTimer.c +++ b/PcAtChipsetPkg/HpetTimerDxe/HpetTimer.c @@ -800,7 +800,7 @@ TimerDriverInitialize ( // // Dump HPET Configuration Information // - DEBUG_CODE ( + DEBUG_CODE_BEGIN (); DEBUG ((DEBUG_INFO, "HPET Base Address = 0x%08x\n", PcdGet32 (PcdHpetBaseAddress))); DEBUG ((DEBUG_INFO, " HPET_GENERAL_CAPABILITIES_ID = 0x%016lx\n", mHpetGeneralCapabilities)); DEBUG ((DEBUG_INFO, " HPET_GENERAL_CONFIGURATION = 0x%016lx\n", mHpetGeneralConfiguration.Uint64)); @@ -812,7 +812,7 @@ TimerDriverInitialize ( DEBUG ((DEBUG_INFO, " HPET_TIMER%d_COMPARATOR = 0x%016lx\n", TimerIndex, HpetRead (HPET_TIMER_COMPARATOR_OFFSET + TimerIndex * HPET_TIMER_STRIDE))); DEBUG ((DEBUG_INFO, " HPET_TIMER%d_MSI_ROUTE = 0x%016lx\n", TimerIndex, HpetRead (HPET_TIMER_MSI_ROUTE_OFFSET + TimerIndex * HPET_TIMER_STRIDE))); } - ); + DEBUG_CODE_END (); // // Capture the current HPET main counter value. @@ -958,7 +958,7 @@ TimerDriverInitialize ( // // Show state of enabled HPET timer // - DEBUG_CODE ( + DEBUG_CODE_BEGIN (); if (mTimerConfiguration.Bits.MsiInterruptCapability != 0 && FeaturePcdGet (PcdHpetMsiEnable)) { DEBUG ((DEBUG_INFO, "HPET Interrupt Mode MSI\n")); } else { @@ -977,7 +977,7 @@ TimerDriverInitialize ( // Wait for a few timer interrupts to fire before continuing // while (mNumTicks < 10); - ); + DEBUG_CODE_END (); // // Install the Timer Architectural Protocol onto a new handle -- 2.39.2