[edk2,edk2-staging,19/20] IntelUndiPkg/XGigUndiDxe: drop separate debug macros for DBG_LVL

Message ID 20181115023353.20159-20-ard.biesheuvel@linaro.org
State New
Headers show
Series
  • IntelUndiPkg/XGigUndiDxe: fix GCC / ARM build issues
Related show

Commit Message

Ard Biesheuvel Nov. 15, 2018, 2:33 a.m.
The DBG_LVL bit tests are already done at build time, so there is no
point in having separate definitions, since they only result in
unused variable warnings.

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>

---
 IntelUndiPkg/XGigUndiDxe/Xgbe.h | 21 --------------------
 1 file changed, 21 deletions(-)

-- 
2.17.1

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Patch

diff --git a/IntelUndiPkg/XGigUndiDxe/Xgbe.h b/IntelUndiPkg/XGigUndiDxe/Xgbe.h
index 2a904de5bf2f..ecfcc1032e8b 100644
--- a/IntelUndiPkg/XGigUndiDxe/Xgbe.h
+++ b/IntelUndiPkg/XGigUndiDxe/Xgbe.h
@@ -1112,7 +1112,6 @@  WaitForEnter (
   );
 
 // This is the Macro Section
-#if DBG_LVL
 /** When specific debug level is currently set this macro
    prints debug message.
 
@@ -1137,26 +1136,6 @@  WaitForEnter (
   if ((DBG_LVL & Lvl) != 0) { \
     WaitForEnter (); \
   }
-#else /* NOT DBG_LVL */
-
-// Comment out the debug stuff
-/** When DBG_LVL is not defined leave occurences of DEBUGPRINT blank
-
-   @param[in]   Lvl   Debug level
-   @param[in]   Msg   Debug message
-
-   @return   None
-**/
-#define DEBUGPRINT(Lvl, Msg)
-
-/** When DBG_LVL is not defined leave occurences of DEBUGWAIT blank
-
-   @param[in]   Lvl   Debug level
-
-   @return   None
-**/
-#define DEBUGWAIT(Lvl)
-#endif /* DBG_LVL */
 
 /** Delays code execution for specified time in milliseconds