Re: [PATCH 1/2] ath9k_hw: Fix incorrect macversion and macrev checks

[Felix Fietkau <nbd@openwrt.org>]

On 2010-12-20 6:37 PM, Senthil Balasubramanian wrote:
> There are few places where we are checking for macversion and revsions
> before RTC is powered ON. However we are reading the macversion and
> revisions only after RTC is powered ON and so both macversion and
> revisions are actully zero and this leads to incorrect srev checks.
> 
> fix this by reading the macversion and revisisons even before we start
> using them. There is no reason why should we delay reading this info
> until RTC is powered on as this is just a register information.
> 
> Cc: Stable Kernel <stable@kernel.org>
> Signed-off-by: Senthil Balasubramanian <senthilkumar@atheros.com>
> ---
>  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 ddda76f..c9b7f1e 100644
> --- a/drivers/net/wireless/ath/ath9k/hw.c
> +++ b/drivers/net/wireless/ath/ath9k/hw.c
> @@ -1078,13 +1078,13 @@ static bool ath9k_hw_set_reset_power_on(struct ath_hw *ah)
>  		return false;
>  	}
>  
> -	ath9k_hw_read_revisions(ah);
> -
>  	return ath9k_hw_set_reset(ah, ATH9K_RESET_WARM);
>  }
>  
>  static bool ath9k_hw_set_reset_reg(struct ath_hw *ah, u32 type)
>  {
> +	ath9k_hw_read_revisions(ah);
> +
>  	if (AR_SREV_9300_20_OR_LATER(ah)) {
>  		REG_WRITE(ah, AR_WA, ah->WARegVal);
>  		udelay(10);
Why re-read the revision on every reset?

- Felix