diff options
| author | Johannes Berg <johannes.berg@intel.com> | 2018-11-09 13:13:15 +0300 |
|---|---|---|
| committer | Johannes Berg <johannes.berg@intel.com> | 2018-11-09 13:13:15 +0300 |
| commit | 6af8354f1db95a01a1ca0638868367d7fa7b6324 (patch) | |
| tree | d08191395c04e0dfcc44da12d9b36ecfd7aec974 /net/mac80211 | |
| parent | 140d905b256165ae22c12837c046665a8d0aa599 (diff) | |
| download | linux-6af8354f1db95a01a1ca0638868367d7fa7b6324.tar.xz | |
mac80211: sta_info: avoid tidstats variable shadowing
We have a pointer called 'tidstats' that shadows a bool function
argument with the same name, but we actually only use it once so
just remove the pointer.
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/mac80211')
| -rw-r--r-- | net/mac80211/sta_info.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/net/mac80211/sta_info.c b/net/mac80211/sta_info.c index fb8c2252ac0e..11b7ae691db0 100644 --- a/net/mac80211/sta_info.c +++ b/net/mac80211/sta_info.c @@ -2253,11 +2253,8 @@ void sta_set_sinfo(struct sta_info *sta, struct station_info *sinfo, } if (tidstats && !cfg80211_sinfo_alloc_tid_stats(sinfo, GFP_KERNEL)) { - for (i = 0; i < IEEE80211_NUM_TIDS + 1; i++) { - struct cfg80211_tid_stats *tidstats = &sinfo->pertid[i]; - - sta_set_tidstats(sta, tidstats, i); - } + for (i = 0; i < IEEE80211_NUM_TIDS + 1; i++) + sta_set_tidstats(sta, &sinfo->pertid[i], i); } if (ieee80211_vif_is_mesh(&sdata->vif)) { |