diff options
| author | Johannes Berg <johannes.berg@intel.com> | 2022-06-01 14:16:00 +0200 |
|---|---|---|
| committer | Johannes Berg <johannes.berg@intel.com> | 2022-06-20 12:55:38 +0200 |
| commit | e5c0ee01fedf2df42426ac4b3c45b4c5cd49de4a (patch) | |
| tree | 27999aa42efc69dfc8860d4cbb21d8fddcc24f6f /net/mac80211/tx.c | |
| parent | 37a7d0dae3111c431b81959ca13cf5effdf9e929 (diff) | |
| download | linux-e5c0ee01fedf2df42426ac4b3c45b4c5cd49de4a.tar.gz linux-e5c0ee01fedf2df42426ac4b3c45b4c5cd49de4a.tar.bz2 linux-e5c0ee01fedf2df42426ac4b3c45b4c5cd49de4a.zip | |
wifi: mac80211: status: look up band only where needed
For MLD, we might eventually not really know the band on status,
but some code assumes it's there. Move the sband lookup deep to
the code that actually needs it, to make it clear where exactly
it's needed and for what purposes.
For rate control, at least initially we won't support it in MLO,
so that won't be an issue.
For TX monitoring, we may have to elide the rate and/or rely on
ieee80211_tx_status_ext() for rate information.
This also simplifies the function prototypes.
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/mac80211/tx.c')
| -rw-r--r-- | net/mac80211/tx.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/net/mac80211/tx.c b/net/mac80211/tx.c index 8530363b2666..f2229b120bdd 100644 --- a/net/mac80211/tx.c +++ b/net/mac80211/tx.c @@ -5228,7 +5228,6 @@ struct sk_buff *ieee80211_beacon_get_tim(struct ieee80211_hw *hw, struct ieee80211_mutable_offsets offs = {}; struct sk_buff *bcn = __ieee80211_beacon_get(hw, vif, &offs, false); struct sk_buff *copy; - struct ieee80211_supported_band *sband; int shift; if (!bcn) @@ -5250,12 +5249,7 @@ struct sk_buff *ieee80211_beacon_get_tim(struct ieee80211_hw *hw, return bcn; shift = ieee80211_vif_get_shift(vif); - sband = ieee80211_get_sband(vif_to_sdata(vif)); - if (!sband) - return bcn; - - ieee80211_tx_monitor(hw_to_local(hw), copy, sband, 1, shift, false, - NULL); + ieee80211_tx_monitor(hw_to_local(hw), copy, 1, shift, false, NULL); return bcn; } |