Sorry, ...this and previous patch should not have gone to LKML.
Will send it over to ath10k list where it was supposed to go.
Thanks,
Ben
On 05/15/2014 11:31 AM, greearb@xxxxxxxxxxxxxxx wrote:
> From: Ben Greear <greearb@xxxxxxxxxxxxxxx>
>
> When the driver cannot provide proper rssi, mark
> status with RX_FLAG_NO_SIGNAL_VAL so that stack
> properly ignores it.
>
> Signed-off-by: Ben Greear <greearb@xxxxxxxxxxxxxxx>
> ---
>
> Patch is against my tree, hopefully will apply OK against upstream
> but have not tested that yet.
>
> drivers/net/wireless/ath/ath10k/htt_rx.c | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/net/wireless/ath/ath10k/htt_rx.c b/drivers/net/wireless/ath/ath10k/htt_rx.c
> index 0c83ffb..d8ec8dd 100644
> --- a/drivers/net/wireless/ath/ath10k/htt_rx.c
> +++ b/drivers/net/wireless/ath/ath10k/htt_rx.c
> @@ -1216,11 +1216,12 @@ static void ath10k_htt_rx_handler(struct ath10k_htt *htt,
> mpdu_ranges = htt_rx_ind_get_mpdu_ranges(rx);
>
> /* Fill this once, while this is per-ppdu */
> - if (rx->ppdu.info0 & HTT_RX_INDICATION_INFO0_START_VALID) {
> - memset(rx_status, 0, sizeof(*rx_status));
> + memset(rx_status, 0, sizeof(*rx_status));
> + if (rx->ppdu.info0 & HTT_RX_INDICATION_INFO0_START_VALID)
> rx_status->signal = ATH10K_DEFAULT_NOISE_FLOOR +
> rx->ppdu.combined_rssi;
> - }
> + else
> + rx_status->flag |= RX_FLAG_NO_SIGNAL_VAL;
>
> if (rx->ppdu.info0 & HTT_RX_INDICATION_INFO0_END_VALID) {
> /* TSF available only in 32-bit */
>
--
Ben Greear <greearb@xxxxxxxxxxxxxxx>
Candela Technologies Inc http://www.candelatech.com
--
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
