ath11k: remove return for empty tx bitrate in mac_op_sta_statistics
authorWen Gong <wgong@codeaurora.org>
Mon, 11 Oct 2021 08:49:57 +0000 (04:49 -0400)
committerKalle Valo <kvalo@codeaurora.org>
Mon, 15 Nov 2021 09:23:26 +0000 (11:23 +0200)
commit1d795645e1eef97fe5d409e3dd5747a942f00e08
tree055e7157697a952dd304dad521570cc79092eae6
parent78406044bdd0cc8987bc082b76867c63ab1c6af8
ath11k: remove return for empty tx bitrate in mac_op_sta_statistics

Currently in ath11k_mac_op_sta_statistics() there is the following
logic:

    if (!arsta->txrate.legacy && !arsta->txrate.nss)
        return;

Unfortunately if this condition is true then the function returns without
setting parameters that follow the txrate. To address this issue remove the
return and instead invert the logic to set the txrate logic if
(arsta->txrate.legacy || arsta->txrate.nss).

The same was done also in ath10k in commit 1cd6ba8ae33e ("ath10k: remove return
for NL80211_STA_INFO_TX_BITRATE").

Tested-on: QCA6390 hw2.0 PCI WLAN.HST.1.0.1-01740-QCAHSTSWPLZ_V2_TO_X86-1

Signed-off-by: Wen Gong <wgong@codeaurora.org>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20211011084957.31024-1-wgong@codeaurora.org
drivers/net/wireless/ath/ath11k/mac.c