diff options
author | Johannes Berg <johannes.berg@intel.com> | 2012-06-22 05:29:50 -0400 |
---|---|---|
committer | Johannes Berg <johannes.berg@intel.com> | 2012-06-24 05:32:29 -0400 |
commit | bdcbd8e0e3ffdad32b14b6373e67bfcf5fd3f002 (patch) | |
tree | b636f2229570dc95edef997272bd0656cf19224d /net/mac80211/ht.c | |
parent | d3b2fb53c7f82903880769d406c11c7e619b11a4 (diff) |
mac80211: clean up debugging
There are a few things that make the logging and
debugging in mac80211 less useful than it should
be right now:
* a lot of messages should be pr_info, not pr_debug
* wholesale use of pr_debug makes it require *both*
Kconfig and dynamic configuration
* there are still a lot of ifdefs
* the style is very inconsistent, sometimes the
sdata->name is printed in front
Clean up everything, introducing new macros and
separating out the station MLME debugging into
a new Kconfig symbol.
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/mac80211/ht.c')
-rw-r--r-- | net/mac80211/ht.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/net/mac80211/ht.c b/net/mac80211/ht.c index 6f8615c54b22..4b4538d63925 100644 --- a/net/mac80211/ht.c +++ b/net/mac80211/ht.c | |||
@@ -305,12 +305,10 @@ void ieee80211_process_delba(struct ieee80211_sub_if_data *sdata, | |||
305 | tid = (params & IEEE80211_DELBA_PARAM_TID_MASK) >> 12; | 305 | tid = (params & IEEE80211_DELBA_PARAM_TID_MASK) >> 12; |
306 | initiator = (params & IEEE80211_DELBA_PARAM_INITIATOR_MASK) >> 11; | 306 | initiator = (params & IEEE80211_DELBA_PARAM_INITIATOR_MASK) >> 11; |
307 | 307 | ||
308 | #ifdef CONFIG_MAC80211_HT_DEBUG | 308 | ht_dbg_ratelimited(sdata, "delba from %pM (%s) tid %d reason code %d\n", |
309 | net_dbg_ratelimited("delba from %pM (%s) tid %d reason code %d\n", | 309 | mgmt->sa, initiator ? "initiator" : "recipient", |
310 | mgmt->sa, initiator ? "initiator" : "recipient", | 310 | tid, |
311 | tid, | 311 | le16_to_cpu(mgmt->u.action.u.delba.reason_code)); |
312 | le16_to_cpu(mgmt->u.action.u.delba.reason_code)); | ||
313 | #endif /* CONFIG_MAC80211_HT_DEBUG */ | ||
314 | 312 | ||
315 | if (initiator == WLAN_BACK_INITIATOR) | 313 | if (initiator == WLAN_BACK_INITIATOR) |
316 | __ieee80211_stop_rx_ba_session(sta, tid, WLAN_BACK_INITIATOR, 0, | 314 | __ieee80211_stop_rx_ba_session(sta, tid, WLAN_BACK_INITIATOR, 0, |