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@xxxxxxxxxx>
> Signed-off-by: Senthil Balasubramanian <senthilkumar@xxxxxxxxxxx>
> ---
> 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
--
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
