ath9k_hw: add a helper for Power Amplifier calibration for AR9002
authorLuis R. Rodriguez <lrodriguez@atheros.com>
Thu, 15 Apr 2010 21:38:51 +0000 (17:38 -0400)
committerJohn W. Linville <linville@tuxdriver.com>
Fri, 16 Apr 2010 19:43:28 +0000 (15:43 -0400)
The code can be simplified and shared between two locations if we bring
this into a helper. During reset we do not account for the skip count.

Signed-off-by: Luis R. Rodriguez <lrodriguez@atheros.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ath/ath9k/calib.c

index eed2c76f56c64e558340de2f6a00ed4b846dcb91..26bc3819bdde76ac611ac59dda7d376683acebe7 100644 (file)
@@ -949,6 +949,21 @@ static inline void ath9k_hw_9285_pa_cal(struct ath_hw *ah, bool is_reset)
 
 }
 
+static void ar9002_hw_pa_cal(struct ath_hw *ah, bool is_reset)
+{
+       if (AR_SREV_9271(ah)) {
+               if (is_reset || !ah->pacal_info.skipcount)
+                       ath9k_hw_9271_pa_cal(ah, is_reset);
+               else
+                       ah->pacal_info.skipcount--;
+       } else if (AR_SREV_9285_11_OR_LATER(ah)) {
+               if (is_reset || !ah->pacal_info.skipcount)
+                       ath9k_hw_9285_pa_cal(ah, is_reset);
+               else
+                       ah->pacal_info.skipcount--;
+       }
+}
+
 bool ath9k_hw_calibrate(struct ath_hw *ah, struct ath9k_channel *chan,
                        u8 rxchainmask, bool longcal)
 {
@@ -973,17 +988,7 @@ bool ath9k_hw_calibrate(struct ath_hw *ah, struct ath9k_channel *chan,
        /* Do NF cal only at longer intervals */
        if (longcal) {
                /* Do periodic PAOffset Cal */
-               if (AR_SREV_9271(ah)) {
-                       if (!ah->pacal_info.skipcount)
-                               ath9k_hw_9271_pa_cal(ah, false);
-                       else
-                               ah->pacal_info.skipcount--;
-               } else if (AR_SREV_9285_11_OR_LATER(ah)) {
-                       if (!ah->pacal_info.skipcount)
-                               ath9k_hw_9285_pa_cal(ah, false);
-                       else
-                               ah->pacal_info.skipcount--;
-               }
+               ar9002_hw_pa_cal(ah, false);
 
                if (OLC_FOR_AR9280_20_LATER || OLC_FOR_AR9287_10_LATER)
                        ath9k_olc_temp_compensation(ah);
@@ -1142,10 +1147,7 @@ bool ath9k_hw_init_cal(struct ath_hw *ah, struct ath9k_channel *chan)
        }
 
        /* Do PA Calibration */
-       if (AR_SREV_9271(ah))
-               ath9k_hw_9271_pa_cal(ah, true);
-       else if (AR_SREV_9285_11_OR_LATER(ah))
-               ath9k_hw_9285_pa_cal(ah, true);
+       ar9002_hw_pa_cal(ah, true);
 
        /* Do NF Calibration after DC offset and other calibrations */
        REG_WRITE(ah, AR_PHY_AGC_CONTROL,