Re: [PATCH] wifi: ath11k: fix rssi station dump for IPQ5018 and QCN6122

George Moussalem <[email protected]>
Newsgroups org.infradead.lists.ath11k,org.kernel.vger.linux-kernel,org.kernel.vger.linux-wireless
Message-ID <SN7PR19MB6736E8D170938359295F975A9DD92@SN7PR19MB6736.namprd19.prod.outlook.com>
Hi all,

On 11/25/25 18:28, Christian Marangi wrote:
> Commit 031ffa6c2cd3 ("wifi: ath11k: fix rssi station dump not updated in
> QCN9074") didn't account for IPQ5018 and QCN6122 WiFi card that are
> based on QCN9074.
> 
> Update the .mpdu_info_get_peerid to use the QCN9074 variant to correctly
> receive consistent RSSI station data.

Is there anything preventing this fix to be reviewed/merged?
I've also tested this and can confirm it's working flawlessly, so:

Tested-by: George Moussalem <[email protected]>

> 
> Reported-by: Scott Mercer <[email protected]>
> Suggested-by: Scott Mercer <[email protected]>
> Tested-by: Scott Mercer <[email protected]>
> Fixes: 031ffa6c2cd3 ("wifi: ath11k: fix rssi station dump not updated in QCN9074")
> Signed-off-by: Christian Marangi <[email protected]>
> ---
>  drivers/net/wireless/ath/ath11k/hw.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/wireless/ath/ath11k/hw.c b/drivers/net/wireless/ath/ath11k/hw.c
> index caa6dc12a790..05d7acbbab56 100644
> --- a/drivers/net/wireless/ath/ath11k/hw.c
> +++ b/drivers/net/wireless/ath/ath11k/hw.c
> @@ -1175,7 +1175,7 @@ const struct ath11k_hw_ops ipq5018_ops = {
>  	.rx_desc_get_attention = ath11k_hw_qcn9074_rx_desc_get_attention,
>  	.reo_setup = ath11k_hw_ipq5018_reo_setup,
>  	.rx_desc_get_msdu_payload = ath11k_hw_qcn9074_rx_desc_get_msdu_payload,
> -	.mpdu_info_get_peerid = ath11k_hw_ipq8074_mpdu_info_get_peerid,
> +	.mpdu_info_get_peerid = ath11k_hw_qcn9074_mpdu_info_get_peerid,
>  	.rx_desc_mac_addr2_valid = ath11k_hw_ipq9074_rx_desc_mac_addr2_valid,
>  	.rx_desc_mpdu_start_addr2 = ath11k_hw_ipq9074_rx_desc_mpdu_start_addr2,
>  	.get_ring_selector = ath11k_hw_ipq8074_get_tcl_ring_selector,


Best regards,
George
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.