On 15 May 2014 20:34, <greearb at candelatech.com> wrote:
> From: Ben Greear <greearb at candelatech.com>
>
> When the driver cannot provide proper rssi, mark
> status with RX_FLAG_NO_SIGNAL_VAL so that stack
> properly ignores it.
>
I think we should skip this one while we know rssi/rates.
They are correct for all packets between START_VALID and END_VALID flags.
BR
Janusz
> Signed-off-by: Ben Greear <greearb at candelatech.com>
> ---
>
> 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 */
> --
> 1.7.11.7
>
>
> _______________________________________________
> ath10k mailing list
> ath10k at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/ath10k