Wen Gong <wgong@xxxxxxxxxxxxxx> wrote:
> 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@xxxxxxxxxxxxxx>
> Signed-off-by: Kalle Valo <kvalo@xxxxxxxxxxxxxx>
Patch applied to ath-next branch of ath.git, thanks.
1d795645e1ee ath11k: remove return for empty tx bitrate in mac_op_sta_statistics
--
https://patchwork.kernel.org/project/linux-wireless/patch/20211011084957.31024-1-wgong@xxxxxxxxxxxxxx/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
