2016-09-02 05:30:30

by Ashok Raj Nagarajan

Subject: [PATCH] ath10k: fix reporting channel survey data

When user requests for survey dump data, driver is providing wrong survey
information. This information we sent is the survey data that we have
collected during previous user request.

This issue occurs because we request survey dump for wrong channel. With
this change, we correctly display the correct and current survey
information to userspace.

Fixes: fa7937e3d5c2 ("ath10k: update bss channel survey information")
Signed-off-by: Ashok Raj Nagarajan <[email protected]>
---
drivers/net/wireless/ath/ath10k/mac.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c
index 0bbd0a0..1fa3c17 100644
--- a/drivers/net/wireless/ath/ath10k/mac.c
+++ b/drivers/net/wireless/ath/ath10k/mac.c
@@ -6538,7 +6538,7 @@ static int ath10k_get_survey(struct ieee80211_hw *hw, int idx,
goto exit;
}

- ath10k_mac_update_bss_chan_survey(ar, survey->channel);
+ ath10k_mac_update_bss_chan_survey(ar, &sband->channels[idx]);

spin_lock_bh(&ar->data_lock);
memcpy(survey, ar_survey, sizeof(*survey));
--
1.9.1

2016-09-09 12:11:19

by Kalle Valo

Subject: Re: ath10k: fix reporting channel survey data

Ashok Raj Nagarajan <[email protected]> wrote:
> When user requests for survey dump data, driver is providing wrong survey
> information. This information we sent is the survey data that we have
> collected during previous user request.
>
> This issue occurs because we request survey dump for wrong channel. With
> this change, we correctly display the correct and current survey
> information to userspace.
>
> Fixes: fa7937e3d5c2 ("ath10k: update bss channel survey information")
> Signed-off-by: Ashok Raj Nagarajan <[email protected]>

Thanks, 1 patch applied to ath-next branch of ath.git:

77eb3d693182 ath10k: fix reporting channel survey data

--
Sent by pwcli
https://patchwork.kernel.org/patch/9310467/