Commit 41bc5440 authored by Ryder Lee's avatar Ryder Lee Committed by Felix Fietkau

wifi: mt76: mt7915: drop return in mt7915_sta_statistics

Avoid blocking other statistics sinfo->filled when txrate is
unavailable.
Signed-off-by: default avatarRyder Lee <ryder.lee@mediatek.com>
Signed-off-by: default avatarFelix Fietkau <nbd@nbd.name>
parent e890c3cf
...@@ -1019,21 +1019,20 @@ static void mt7915_sta_statistics(struct ieee80211_hw *hw, ...@@ -1019,21 +1019,20 @@ static void mt7915_sta_statistics(struct ieee80211_hw *hw,
sinfo->filled |= BIT_ULL(NL80211_STA_INFO_RX_BITRATE); sinfo->filled |= BIT_ULL(NL80211_STA_INFO_RX_BITRATE);
} }
if (!txrate->legacy && !txrate->flags) if (txrate->legacy || txrate->flags) {
return; if (txrate->legacy) {
sinfo->txrate.legacy = txrate->legacy;
if (txrate->legacy) { } else {
sinfo->txrate.legacy = txrate->legacy; sinfo->txrate.mcs = txrate->mcs;
} else { sinfo->txrate.nss = txrate->nss;
sinfo->txrate.mcs = txrate->mcs; sinfo->txrate.bw = txrate->bw;
sinfo->txrate.nss = txrate->nss; sinfo->txrate.he_gi = txrate->he_gi;
sinfo->txrate.bw = txrate->bw; sinfo->txrate.he_dcm = txrate->he_dcm;
sinfo->txrate.he_gi = txrate->he_gi; sinfo->txrate.he_ru_alloc = txrate->he_ru_alloc;
sinfo->txrate.he_dcm = txrate->he_dcm; }
sinfo->txrate.he_ru_alloc = txrate->he_ru_alloc; sinfo->txrate.flags = txrate->flags;
sinfo->filled |= BIT_ULL(NL80211_STA_INFO_TX_BITRATE);
} }
sinfo->txrate.flags = txrate->flags;
sinfo->filled |= BIT_ULL(NL80211_STA_INFO_TX_BITRATE);
/* offloading flows bypass networking stack, so driver counts and /* offloading flows bypass networking stack, so driver counts and
* reports sta statistics via NL80211_STA_INFO when WED is active. * reports sta statistics via NL80211_STA_INFO when WED is active.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment