On 1/30/25 14:19, Remi Pommarel wrote:
On Thu, Jan 30, 2025 at 12:21:54PM +0530, Aditya Kumar Singh wrote:
On 1/29/25 22:25, Remi Pommarel wrote:
Currently in ath12k_mac_op_sta_statistics() there is the following
logic:
if (!arsta->txrate.legacy && !arsta->txrate.nss)
return;
Because ath12k_sta_statistics is used to report many info to iw wlan0 link,
if it return for empty legacy and nss of arsta->txrate, then the other
stats after it will not be set.
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 both ath10k with commit 1cd6ba8ae33e ("ath10k:
remove return for NL80211_STA_INFO_TX_BITRATE") and ath11k as well with
commit 1d795645e1ee ("ath11k: remove return for empty tx bitrate in
mac_op_sta_statistics").
Tested-on: QCN9274 hw2.0 PCI WLAN.WBE.1.3.1-00173-QCAHKSWPL_SILICONZ-1
MISSING_BLANK_LINE
'Tested-on:' tag missing blank line after it.
You missed v1 comment? :)
Yes sorry I think your mail never reached me, did you CC me ? Do you
need a respin ?
No problem. No need of respin just because of this. I think Jeff can fix
in pending?
It's strange that the v1 reply appears to be delivered from my mail box
and it is sent to the list. I see even Jeff replying to that. But none
of those are captured in patchwork. Not sure.
Anyways, v1 comment was regarding this blank line and one suggestion
that in future submissions, please use base commit tag.
--
Aditya
