diff options
author | Toke Høiland-Jørgensen <toke@toke.dk> | 2022-04-04 23:01:08 +0200 |
---|---|---|
committer | Johannes Berg <johannes.berg@intel.com> | 2022-04-11 15:12:25 +0200 |
commit | e5c95ca094cfe59a4013d711f87ff034cba30f80 (patch) | |
tree | c106faaa3a5c4a4ac322e6eee78c1c5302556b84 /include | |
parent | mac80211: protect ieee80211_assign_beacon with next_beacon check (diff) | |
download | linux-e5c95ca094cfe59a4013d711f87ff034cba30f80.tar.xz linux-e5c95ca094cfe59a4013d711f87ff034cba30f80.zip |
mac80211: Improve confusing comment around tx_info clearing
The comment above the ieee80211_tx_info_clear_status() helper was somewhat
confusing as to which fields it was or wasn't clearing. So replace it by
something that is hopefully more, well, clear.
Signed-off-by: Toke Høiland-Jørgensen <toke@toke.dk>
Link: https://lore.kernel.org/r/20220404210108.2684907-1-toke@toke.dk
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'include')
-rw-r--r-- | include/net/mac80211.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/include/net/mac80211.h b/include/net/mac80211.h index 382ebb862ea8..db992f71604d 100644 --- a/include/net/mac80211.h +++ b/include/net/mac80211.h @@ -1201,9 +1201,9 @@ static inline struct ieee80211_rx_status *IEEE80211_SKB_RXCB(struct sk_buff *skb * in the TX status but the rate control information (it does clear * the count since you need to fill that in anyway). * - * NOTE: You can only use this function if you do NOT use - * info->driver_data! Use info->rate_driver_data - * instead if you need only the less space that allows. + * NOTE: While the rates array is kept intact, this will wipe all of the + * driver_data fields in info, so it's up to the driver to restore + * any fields it needs after calling this helper. */ static inline void ieee80211_tx_info_clear_status(struct ieee80211_tx_info *info) |