On Fri, 2018-03-23 at 19:11 +0530, akolli@xxxxxxxxxxxxxx wrote: > > > > + > > > + sinfo->expected_throughput = > > > + > > > ewma_sta_txrate_read(&arsta->ave_sta_txrate); > > > + sinfo->filled |= > > > BIT(NL80211_STA_INFO_EXPECTED_THROUGHPUT); > > > +} > Yes. This looks redundant, I will remove this line, > sinfo->filled |= BIT(NL80211_STA_INFO_EXPECTED_THROUGHPUT); You don't need any of the changes here in ath10k_sta_statistics(). johannes