Chun-Yeow Yeoh <yeohchunyeow@xxxxxxxxx> writes:
> Fix the MAC address of wmi_peer_stats so that it is
> printed correctly. This is tested and verified using
> firmware version 999.999.0.636.
>
> Based on the verification, maximum only 3 peer statistics including
> self STA able to be printed out.
>
> Signed-off-by: Chun-Yeow Yeoh <yeohchunyeow@xxxxxxxxx>
> ---
> v2: offset the stats to ignore the first peer (Chun-Yeow)
I think something is wrong still. I tested this with 999.999.0.636 on
x86 32 bit laptop in AP mode, connected STA 00:24:d7:9b:0b:7c to it and
still MAC address is wrong:
PHY errors drops 0
MPDU errors (FCS, MIC, ENC) 0
ath10k PEER stats
=================
Peer MAC address 9b:d7:24:00:00:00
Peer RSSI 36
Peer TX rate 0
Without your patch it's also wrong:
ath10k PEER stats
=================
Peer MAC address 00:00:00:00:02:00
Peer RSSI 3
Peer TX rate 0
Peer MAC address 00:00:00:00:00:24
Peer RSSI 31755
Peer TX rate 0
Any ideas what's happening here?
> --- a/drivers/net/wireless/ath/ath10k/debug.c
> +++ b/drivers/net/wireless/ath/ath10k/debug.c
> @@ -242,6 +242,10 @@ void ath10k_debug_read_target_stats(struct ath10k *ar,
> }
> }
>
> + /* The first peer is self MAC address, ignore this */
> + num_peer_stats--;
> + tmp += sizeof(struct wmi_peer_stats);
Should we show "self peer" separately? Does it provide any useful
information?
--
Kalle Valo
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
