rtlwifi: add CONFIG_RTLWIFI_DEBUG to remove all of the debug logging code
authorJoe Perches <joe@perches.com>
Thu, 5 Jan 2012 16:29:07 +0000 (08:29 -0800)
committerJohn W. Linville <linville@tuxdriver.com>
Tue, 24 Jan 2012 19:07:41 +0000 (14:07 -0500)
It does seem odd though to have a DBG_EMERG and not always
emit it.

What might also be useful for any embedded use is to add
CONFIG_RTLWIFI_DEBUG to conditionally remove all of the
debug logging code to reduce the largish object size.

This reduces the object size by about 1/3 (250KB)
when CONFIG_RTLWIFI_DEBUG is not set.

$ size drivers/net/wireless/rtlwifi/built-in.o*
   text    data     bss     dec     hex filename
 368722   55333   94224  518279   7e887 drivers/net/wireless/rtlwifi/built-in.o.new
 586904   55333  127216  769453   bbdad drivers/net/wireless/rtlwifi/built-in.o.old

Signed-off-by: Joe Perches <joe@perches.com>
Acked-by: Larry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/rtlwifi/Kconfig
drivers/net/wireless/rtlwifi/debug.h

index d6c42e69bdbd467225d4db746f0e0f82d860fd68..44b9c0a57702dcdb37584ef4f099e8e6d093a461 100644 (file)
@@ -49,6 +49,11 @@ config RTLWIFI
        depends on RTL8192CE || RTL8192CU || RTL8192SE || RTL8192DE
        default m
 
+config RTLWIFI_DEBUG
+       tristate "Additional debugging output"
+       depends on RTL8192CE || RTL8192CU || RTL8192SE || RTL8192DE
+       default y
+
 config RTL8192C_COMMON
        tristate
        depends on RTL8192CE || RTL8192CU
index b024c23b6523ee8c7caf4cc4a9491cc91029d9fc..fd5600c6942833cf390e4b7cb5698fbc82bfb4ef 100644 (file)
@@ -156,6 +156,8 @@ enum dbgp_flag_e {
        DBGP_TYPE_MAX
 };
 
+#ifdef CONFIG_RTLWIFI_DEBUG
+
 #define RT_ASSERT(_exp, fmt, ...)                                      \
 do {                                                                   \
        if (!(_exp)) {                                                  \
@@ -195,5 +197,37 @@ do {                                                                       \
        }                                                               \
 } while (0)
 
+#else
+
+struct rtl_priv;
+
+__printf(2, 3)
+static inline void RT_ASSERT(int exp, const char *fmt, ...)
+{
+}
+
+__printf(4, 5)
+static inline void RT_TRACE(struct rtl_priv *rtlpriv,
+                           int comp, int level,
+                           const char *fmt, ...)
+{
+}
+
+__printf(4, 5)
+static inline void RTPRINT(struct rtl_priv *rtlpriv,
+                          int dbgtype, int dbgflag,
+                          const char *fmt, ...)
+{
+}
+
+static inline void RT_PRINT_DATA(struct rtl_priv *rtlpriv,
+                                int comp, int level,
+                                const char *titlestring,
+                                const void *hexdata, size_t hexdatalen)
+{
+}
+
+#endif
+
 void rtl_dbgp_flag_init(struct ieee80211_hw *hw);
 #endif