Search Linux Wireless

Re: [PATCH 4/4] ath9k: enable ani for 9100

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Nov 21, 2011 at 6:19 AM, Nikolay Martynov <mar.kolya@xxxxxxxxx> wrote:
> ---
>  drivers/net/wireless/ath/ath9k/hw.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/wireless/ath/ath9k/hw.c b/drivers/net/wireless/ath/ath9k/hw.c
> index d6a2568..79c4a0f 100644
> --- a/drivers/net/wireless/ath/ath9k/hw.c
> +++ b/drivers/net/wireless/ath/ath9k/hw.c
> @@ -610,8 +610,8 @@ static int __ath9k_hw_init(struct ath_hw *ah)
>        if (!AR_SREV_9300_20_OR_LATER(ah))
>                ah->ani_function &= ~ATH9K_ANI_MRC_CCK;
>
> -        //disable ANI for 9100 and 9340
> -        if (AR_SREV_9100(ah) || AR_SREV_9340(ah))
> +        //disable ANI for 9340

please change the comment style to
/*
*
*/

> +        if (AR_SREV_9340(ah))
>                 ah->config.enable_ani = false;

please mention in the commit log why we are enabling ANI for AR9100,
if there is some improvement performance in noisy environment, please
do mention it.
i assume if it is disabled we may not  have ANI support for AR9100
(or) the hardware code changes are missing.

>
>        ath9k_hw_init_mode_regs(ah);
> --
> 1.7.4.1
>
> --
> 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
>



-- 
shafi
--
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


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux