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, gree...@candelatech.com wrote:
> From: Ben Greear <gree...@candelatech.com>
> 
> 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 <gree...@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 */
> 


-- 
Ben Greear <gree...@candelatech.com>
Candela Technologies Inc  http://www.candelatech.com

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to