diff options
author | Johannes Berg <johannes@sipsolutions.net> | 2008-10-27 18:56:10 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-10-27 20:06:16 -0400 |
commit | 0c68ae2605dbcf67414d8d1f19af93be44b355fb (patch) | |
tree | 19175d53668f5358d346929e5db01d8fe41cb2aa /net/mac80211/ht.c | |
parent | dd45c9cf687682c9ce256ab14bd8914db77410bb (diff) |
mac80211: convert to %pM away from print_mac
Also remove a few stray DECLARE_MAC_BUF that were no longer
used at all.
Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/mac80211/ht.c')
-rw-r--r-- | net/mac80211/ht.c | 42 |
1 files changed, 16 insertions, 26 deletions
diff --git a/net/mac80211/ht.c b/net/mac80211/ht.c index dc7d9a3d70d5..b854483cf23f 100644 --- a/net/mac80211/ht.c +++ b/net/mac80211/ht.c | |||
@@ -241,7 +241,6 @@ void ieee80211_sta_stop_rx_ba_session(struct ieee80211_sub_if_data *sdata, u8 *r | |||
241 | struct ieee80211_hw *hw = &local->hw; | 241 | struct ieee80211_hw *hw = &local->hw; |
242 | struct sta_info *sta; | 242 | struct sta_info *sta; |
243 | int ret, i; | 243 | int ret, i; |
244 | DECLARE_MAC_BUF(mac); | ||
245 | 244 | ||
246 | rcu_read_lock(); | 245 | rcu_read_lock(); |
247 | 246 | ||
@@ -269,8 +268,8 @@ void ieee80211_sta_stop_rx_ba_session(struct ieee80211_sub_if_data *sdata, u8 *r | |||
269 | BUG_ON(!local->ops->ampdu_action); | 268 | BUG_ON(!local->ops->ampdu_action); |
270 | 269 | ||
271 | #ifdef CONFIG_MAC80211_HT_DEBUG | 270 | #ifdef CONFIG_MAC80211_HT_DEBUG |
272 | printk(KERN_DEBUG "Rx BA session stop requested for %s tid %u\n", | 271 | printk(KERN_DEBUG "Rx BA session stop requested for %pM tid %u\n", |
273 | print_mac(mac, ra), tid); | 272 | ra, tid); |
274 | #endif /* CONFIG_MAC80211_HT_DEBUG */ | 273 | #endif /* CONFIG_MAC80211_HT_DEBUG */ |
275 | 274 | ||
276 | ret = local->ops->ampdu_action(hw, IEEE80211_AMPDU_RX_STOP, | 275 | ret = local->ops->ampdu_action(hw, IEEE80211_AMPDU_RX_STOP, |
@@ -383,14 +382,13 @@ int ieee80211_start_tx_ba_session(struct ieee80211_hw *hw, u8 *ra, u16 tid) | |||
383 | u16 start_seq_num; | 382 | u16 start_seq_num; |
384 | u8 *state; | 383 | u8 *state; |
385 | int ret; | 384 | int ret; |
386 | DECLARE_MAC_BUF(mac); | ||
387 | 385 | ||
388 | if (tid >= STA_TID_NUM) | 386 | if (tid >= STA_TID_NUM) |
389 | return -EINVAL; | 387 | return -EINVAL; |
390 | 388 | ||
391 | #ifdef CONFIG_MAC80211_HT_DEBUG | 389 | #ifdef CONFIG_MAC80211_HT_DEBUG |
392 | printk(KERN_DEBUG "Open BA session requested for %s tid %u\n", | 390 | printk(KERN_DEBUG "Open BA session requested for %pM tid %u\n", |
393 | print_mac(mac, ra), tid); | 391 | ra, tid); |
394 | #endif /* CONFIG_MAC80211_HT_DEBUG */ | 392 | #endif /* CONFIG_MAC80211_HT_DEBUG */ |
395 | 393 | ||
396 | rcu_read_lock(); | 394 | rcu_read_lock(); |
@@ -524,7 +522,6 @@ int ieee80211_stop_tx_ba_session(struct ieee80211_hw *hw, | |||
524 | struct sta_info *sta; | 522 | struct sta_info *sta; |
525 | u8 *state; | 523 | u8 *state; |
526 | int ret = 0; | 524 | int ret = 0; |
527 | DECLARE_MAC_BUF(mac); | ||
528 | 525 | ||
529 | if (tid >= STA_TID_NUM) | 526 | if (tid >= STA_TID_NUM) |
530 | return -EINVAL; | 527 | return -EINVAL; |
@@ -546,8 +543,8 @@ int ieee80211_stop_tx_ba_session(struct ieee80211_hw *hw, | |||
546 | } | 543 | } |
547 | 544 | ||
548 | #ifdef CONFIG_MAC80211_HT_DEBUG | 545 | #ifdef CONFIG_MAC80211_HT_DEBUG |
549 | printk(KERN_DEBUG "Tx BA session stop requested for %s tid %u\n", | 546 | printk(KERN_DEBUG "Tx BA session stop requested for %pM tid %u\n", |
550 | print_mac(mac, ra), tid); | 547 | ra, tid); |
551 | #endif /* CONFIG_MAC80211_HT_DEBUG */ | 548 | #endif /* CONFIG_MAC80211_HT_DEBUG */ |
552 | 549 | ||
553 | ieee80211_stop_queue(hw, sta->tid_to_tx_q[tid]); | 550 | ieee80211_stop_queue(hw, sta->tid_to_tx_q[tid]); |
@@ -579,7 +576,6 @@ void ieee80211_start_tx_ba_cb(struct ieee80211_hw *hw, u8 *ra, u16 tid) | |||
579 | struct ieee80211_local *local = hw_to_local(hw); | 576 | struct ieee80211_local *local = hw_to_local(hw); |
580 | struct sta_info *sta; | 577 | struct sta_info *sta; |
581 | u8 *state; | 578 | u8 *state; |
582 | DECLARE_MAC_BUF(mac); | ||
583 | 579 | ||
584 | if (tid >= STA_TID_NUM) { | 580 | if (tid >= STA_TID_NUM) { |
585 | #ifdef CONFIG_MAC80211_HT_DEBUG | 581 | #ifdef CONFIG_MAC80211_HT_DEBUG |
@@ -594,8 +590,7 @@ void ieee80211_start_tx_ba_cb(struct ieee80211_hw *hw, u8 *ra, u16 tid) | |||
594 | if (!sta) { | 590 | if (!sta) { |
595 | rcu_read_unlock(); | 591 | rcu_read_unlock(); |
596 | #ifdef CONFIG_MAC80211_HT_DEBUG | 592 | #ifdef CONFIG_MAC80211_HT_DEBUG |
597 | printk(KERN_DEBUG "Could not find station: %s\n", | 593 | printk(KERN_DEBUG "Could not find station: %pM\n", ra); |
598 | print_mac(mac, ra)); | ||
599 | #endif | 594 | #endif |
600 | return; | 595 | return; |
601 | } | 596 | } |
@@ -634,7 +629,6 @@ void ieee80211_stop_tx_ba_cb(struct ieee80211_hw *hw, u8 *ra, u8 tid) | |||
634 | struct sta_info *sta; | 629 | struct sta_info *sta; |
635 | u8 *state; | 630 | u8 *state; |
636 | int agg_queue; | 631 | int agg_queue; |
637 | DECLARE_MAC_BUF(mac); | ||
638 | 632 | ||
639 | if (tid >= STA_TID_NUM) { | 633 | if (tid >= STA_TID_NUM) { |
640 | #ifdef CONFIG_MAC80211_HT_DEBUG | 634 | #ifdef CONFIG_MAC80211_HT_DEBUG |
@@ -645,16 +639,15 @@ void ieee80211_stop_tx_ba_cb(struct ieee80211_hw *hw, u8 *ra, u8 tid) | |||
645 | } | 639 | } |
646 | 640 | ||
647 | #ifdef CONFIG_MAC80211_HT_DEBUG | 641 | #ifdef CONFIG_MAC80211_HT_DEBUG |
648 | printk(KERN_DEBUG "Stopping Tx BA session for %s tid %d\n", | 642 | printk(KERN_DEBUG "Stopping Tx BA session for %pM tid %d\n", |
649 | print_mac(mac, ra), tid); | 643 | ra, tid); |
650 | #endif /* CONFIG_MAC80211_HT_DEBUG */ | 644 | #endif /* CONFIG_MAC80211_HT_DEBUG */ |
651 | 645 | ||
652 | rcu_read_lock(); | 646 | rcu_read_lock(); |
653 | sta = sta_info_get(local, ra); | 647 | sta = sta_info_get(local, ra); |
654 | if (!sta) { | 648 | if (!sta) { |
655 | #ifdef CONFIG_MAC80211_HT_DEBUG | 649 | #ifdef CONFIG_MAC80211_HT_DEBUG |
656 | printk(KERN_DEBUG "Could not find station: %s\n", | 650 | printk(KERN_DEBUG "Could not find station: %pM\n", ra); |
657 | print_mac(mac, ra)); | ||
658 | #endif | 651 | #endif |
659 | rcu_read_unlock(); | 652 | rcu_read_unlock(); |
660 | return; | 653 | return; |
@@ -783,7 +776,6 @@ void ieee80211_process_addba_request(struct ieee80211_local *local, | |||
783 | u16 capab, tid, timeout, ba_policy, buf_size, start_seq_num, status; | 776 | u16 capab, tid, timeout, ba_policy, buf_size, start_seq_num, status; |
784 | u8 dialog_token; | 777 | u8 dialog_token; |
785 | int ret = -EOPNOTSUPP; | 778 | int ret = -EOPNOTSUPP; |
786 | DECLARE_MAC_BUF(mac); | ||
787 | 779 | ||
788 | /* extract session parameters from addba request frame */ | 780 | /* extract session parameters from addba request frame */ |
789 | dialog_token = mgmt->u.action.u.addba_req.dialog_token; | 781 | dialog_token = mgmt->u.action.u.addba_req.dialog_token; |
@@ -808,8 +800,8 @@ void ieee80211_process_addba_request(struct ieee80211_local *local, | |||
808 | #ifdef CONFIG_MAC80211_HT_DEBUG | 800 | #ifdef CONFIG_MAC80211_HT_DEBUG |
809 | if (net_ratelimit()) | 801 | if (net_ratelimit()) |
810 | printk(KERN_DEBUG "AddBA Req with bad params from " | 802 | printk(KERN_DEBUG "AddBA Req with bad params from " |
811 | "%s on tid %u. policy %d, buffer size %d\n", | 803 | "%pM on tid %u. policy %d, buffer size %d\n", |
812 | print_mac(mac, mgmt->sa), tid, ba_policy, | 804 | mgmt->sa, tid, ba_policy, |
813 | buf_size); | 805 | buf_size); |
814 | #endif /* CONFIG_MAC80211_HT_DEBUG */ | 806 | #endif /* CONFIG_MAC80211_HT_DEBUG */ |
815 | goto end_no_lock; | 807 | goto end_no_lock; |
@@ -831,8 +823,8 @@ void ieee80211_process_addba_request(struct ieee80211_local *local, | |||
831 | #ifdef CONFIG_MAC80211_HT_DEBUG | 823 | #ifdef CONFIG_MAC80211_HT_DEBUG |
832 | if (net_ratelimit()) | 824 | if (net_ratelimit()) |
833 | printk(KERN_DEBUG "unexpected AddBA Req from " | 825 | printk(KERN_DEBUG "unexpected AddBA Req from " |
834 | "%s on tid %u\n", | 826 | "%pM on tid %u\n", |
835 | print_mac(mac, mgmt->sa), tid); | 827 | mgmt->sa, tid); |
836 | #endif /* CONFIG_MAC80211_HT_DEBUG */ | 828 | #endif /* CONFIG_MAC80211_HT_DEBUG */ |
837 | goto end; | 829 | goto end; |
838 | } | 830 | } |
@@ -964,7 +956,6 @@ void ieee80211_process_delba(struct ieee80211_sub_if_data *sdata, | |||
964 | struct ieee80211_local *local = sdata->local; | 956 | struct ieee80211_local *local = sdata->local; |
965 | u16 tid, params; | 957 | u16 tid, params; |
966 | u16 initiator; | 958 | u16 initiator; |
967 | DECLARE_MAC_BUF(mac); | ||
968 | 959 | ||
969 | params = le16_to_cpu(mgmt->u.action.u.delba.params); | 960 | params = le16_to_cpu(mgmt->u.action.u.delba.params); |
970 | tid = (params & IEEE80211_DELBA_PARAM_TID_MASK) >> 12; | 961 | tid = (params & IEEE80211_DELBA_PARAM_TID_MASK) >> 12; |
@@ -972,9 +963,8 @@ void ieee80211_process_delba(struct ieee80211_sub_if_data *sdata, | |||
972 | 963 | ||
973 | #ifdef CONFIG_MAC80211_HT_DEBUG | 964 | #ifdef CONFIG_MAC80211_HT_DEBUG |
974 | if (net_ratelimit()) | 965 | if (net_ratelimit()) |
975 | printk(KERN_DEBUG "delba from %s (%s) tid %d reason code %d\n", | 966 | printk(KERN_DEBUG "delba from %pM (%s) tid %d reason code %d\n", |
976 | print_mac(mac, mgmt->sa), | 967 | mgmt->sa, initiator ? "initiator" : "recipient", tid, |
977 | initiator ? "initiator" : "recipient", tid, | ||
978 | mgmt->u.action.u.delba.reason_code); | 968 | mgmt->u.action.u.delba.reason_code); |
979 | #endif /* CONFIG_MAC80211_HT_DEBUG */ | 969 | #endif /* CONFIG_MAC80211_HT_DEBUG */ |
980 | 970 | ||