From: Toke Høiland-Jørgensen Date: Mon, 4 Apr 2022 21:01:08 +0000 (+0200) Subject: mac80211: Improve confusing comment around tx_info clearing X-Git-Tag: howlett/maple/20220722_2~466^2~193^2~90 X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=e5c95ca094cfe59a4013d711f87ff034cba30f80;p=users%2Fjedix%2Flinux-maple.git 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 Link: https://lore.kernel.org/r/20220404210108.2684907-1-toke@toke.dk Signed-off-by: Johannes Berg --- 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)