Commit 4279425c authored by Nikolay Martynov's avatar Nikolay Martynov Committed by John W. Linville

ath9k: use config.enable_ani to check if ani should be performed

  Currently in ath9k code there is an attempt which is meant to
disable ANI for ar9100 and ar9340. But it doesn't really achieve
this. All it does is disable ANI init and setup (i.e. calls to
ath9k_hw_ani_setup and ath9k_hw_ani_init). Since ath9k_hw_ani_setup is
not called ah->config.ani_poll_interval is never initialized (i.e. it
is always zero) and ath_ani_calibrate always executes ANI procedures
(over uninitialized ANI parameters).
  Moreover, ath_ani_calibrate is being called each 1ms because
common->ani.timer is set to zero interval because
ah->config.ani_poll_interval==0 (and thus smallest value of all
intervals). Normally it should not be called this often.
  This patch changes the code so config.enable_ani is used to check if
ANI should be performed.
  config.enable_ani is initialized to true by default. This patch sets
it to false for ar9100 and ar9340.
Signed-off-by: default avatarNikolay Martynov <mar.kolya@gmail.com>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 86951359
...@@ -808,7 +808,8 @@ void ath9k_htc_ani_work(struct work_struct *work) ...@@ -808,7 +808,8 @@ void ath9k_htc_ani_work(struct work_struct *work)
} }
/* Verify whether we must check ANI */ /* Verify whether we must check ANI */
if ((timestamp - common->ani.checkani_timer) >= ATH_ANI_POLLINTERVAL) { if (ah->config.enable_ani &&
(timestamp - common->ani.checkani_timer) >= ATH_ANI_POLLINTERVAL) {
aniflag = true; aniflag = true;
common->ani.checkani_timer = timestamp; common->ani.checkani_timer = timestamp;
} }
...@@ -838,7 +839,7 @@ void ath9k_htc_ani_work(struct work_struct *work) ...@@ -838,7 +839,7 @@ void ath9k_htc_ani_work(struct work_struct *work)
* short calibration and long calibration. * short calibration and long calibration.
*/ */
cal_interval = ATH_LONG_CALINTERVAL; cal_interval = ATH_LONG_CALINTERVAL;
if (priv->ah->config.enable_ani) if (ah->config.enable_ani)
cal_interval = min(cal_interval, (u32)ATH_ANI_POLLINTERVAL); cal_interval = min(cal_interval, (u32)ATH_ANI_POLLINTERVAL);
if (!common->ani.caldone) if (!common->ani.caldone)
cal_interval = min(cal_interval, (u32)short_cal_interval); cal_interval = min(cal_interval, (u32)short_cal_interval);
......
...@@ -504,7 +504,7 @@ static int ath9k_hw_post_init(struct ath_hw *ah) ...@@ -504,7 +504,7 @@ static int ath9k_hw_post_init(struct ath_hw *ah)
return ecode; return ecode;
} }
if (!AR_SREV_9100(ah) && !AR_SREV_9340(ah)) { if (ah->config.enable_ani) {
ath9k_hw_ani_setup(ah); ath9k_hw_ani_setup(ah);
ath9k_hw_ani_init(ah); ath9k_hw_ani_init(ah);
} }
...@@ -610,6 +610,10 @@ static int __ath9k_hw_init(struct ath_hw *ah) ...@@ -610,6 +610,10 @@ static int __ath9k_hw_init(struct ath_hw *ah)
if (!AR_SREV_9300_20_OR_LATER(ah)) if (!AR_SREV_9300_20_OR_LATER(ah))
ah->ani_function &= ~ATH9K_ANI_MRC_CCK; ah->ani_function &= ~ATH9K_ANI_MRC_CCK;
/* disable ANI for 9100 and 9340 */
if (AR_SREV_9100(ah) || AR_SREV_9340(ah))
ah->config.enable_ani = false;
ath9k_hw_init_mode_regs(ah); ath9k_hw_init_mode_regs(ah);
if (!ah->is_pciexpress) if (!ah->is_pciexpress)
......
...@@ -582,8 +582,9 @@ void ath_ani_calibrate(unsigned long data) ...@@ -582,8 +582,9 @@ void ath_ani_calibrate(unsigned long data)
} }
/* Verify whether we must check ANI */ /* Verify whether we must check ANI */
if ((timestamp - common->ani.checkani_timer) >= if (sc->sc_ah->config.enable_ani
ah->config.ani_poll_interval) { && (timestamp - common->ani.checkani_timer) >=
ah->config.ani_poll_interval) {
aniflag = true; aniflag = true;
common->ani.checkani_timer = timestamp; common->ani.checkani_timer = timestamp;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment