ath9k_hw: fix a noise floor calibration related race condition
authorFelix Fietkau <nbd@openwrt.org>
Fri, 30 Jul 2010 22:12:01 +0000 (00:12 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Wed, 4 Aug 2010 19:27:37 +0000 (15:27 -0400)
On AR5008-AR9002, other forms of calibration must not be started while
the noise floor calibration is running, as this can create invalid
readings which were sometimes not even recoverable by any further
calibration attempts.

This patch also ensures that the result of noise floor measurements
are processed faster and also allows the result of the initial
calibration on reset to make it into the NF history buffer

Signed-off-by: Felix Fietkau <nbd@openwrt.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ath/ath9k/ar9002_calib.c
drivers/net/wireless/ath/ath9k/calib.c
drivers/net/wireless/ath/ath9k/calib.h
drivers/net/wireless/ath/ath9k/hw.h

index d28a8d37f01f5e6091fc0a4f82b8517376ad1486..fe7418aefc4a994b00109e98bac4edc7d0e67a6e 100644 (file)
@@ -687,8 +687,13 @@ static bool ar9002_hw_calibrate(struct ath_hw *ah,
 {
        bool iscaldone = true;
        struct ath9k_cal_list *currCal = ah->cal_list_curr;
+       bool nfcal, nfcal_pending = false;
 
-       if (currCal &&
+       nfcal = !!(REG_READ(ah, AR_PHY_AGC_CONTROL) & AR_PHY_AGC_CONTROL_NF);
+       if (ah->caldata)
+               nfcal_pending = ah->caldata->nfcal_pending;
+
+       if (currCal && !nfcal &&
            (currCal->calState == CAL_RUNNING ||
             currCal->calState == CAL_WAITING)) {
                iscaldone = ar9002_hw_per_calibration(ah, chan,
@@ -704,7 +709,7 @@ static bool ar9002_hw_calibrate(struct ath_hw *ah,
        }
 
        /* Do NF cal only at longer intervals */
-       if (longcal) {
+       if (longcal || nfcal_pending) {
                /* Do periodic PAOffset Cal */
                ar9002_hw_pa_cal(ah, false);
                ar9002_hw_olc_temp_compensation(ah);
@@ -713,16 +718,18 @@ static bool ar9002_hw_calibrate(struct ath_hw *ah,
                 * Get the value from the previous NF cal and update
                 * history buffer.
                 */
-               ath9k_hw_getnf(ah, chan);
-
-               /*
-                * Load the NF from history buffer of the current channel.
-                * NF is slow time-variant, so it is OK to use a historical
-                * value.
-                */
-               ath9k_hw_loadnf(ah, ah->curchan);
+               if (ath9k_hw_getnf(ah, chan)) {
+                       /*
+                        * Load the NF from history buffer of the current
+                        * channel.
+                        * NF is slow time-variant, so it is OK to use a
+                        * historical value.
+                        */
+                       ath9k_hw_loadnf(ah, ah->curchan);
+               }
 
-               ath9k_hw_start_nfcal(ah, false);
+               if (longcal)
+                       ath9k_hw_start_nfcal(ah, false);
        }
 
        return iscaldone;
@@ -872,6 +879,9 @@ static bool ar9002_hw_init_cal(struct ath_hw *ah, struct ath9k_channel *chan)
        /* Do NF Calibration after DC offset and other calibrations */
        ath9k_hw_start_nfcal(ah, true);
 
+       if (ah->caldata)
+               ah->caldata->nfcal_pending = true;
+
        ah->cal_list = ah->cal_list_last = ah->cal_list_curr = NULL;
 
        /* Enable IQ, ADC Gain and ADC DC offset CALs */
index 18b5c0dcc1fc571b2c38ef09a4fc78ba71d07ada..45208690c0ec3d5cf747d0ac18218ddeedaf20f8 100644 (file)
@@ -156,6 +156,9 @@ EXPORT_SYMBOL(ath9k_hw_reset_calvalid);
 
 void ath9k_hw_start_nfcal(struct ath_hw *ah, bool update)
 {
+       if (ah->caldata)
+               ah->caldata->nfcal_pending = true;
+
        REG_SET_BIT(ah, AR_PHY_AGC_CONTROL,
                    AR_PHY_AGC_CONTROL_ENABLE_NF);
 
@@ -288,8 +291,7 @@ static void ath9k_hw_nf_sanitize(struct ath_hw *ah, s16 *nf)
        }
 }
 
-int16_t ath9k_hw_getnf(struct ath_hw *ah,
-                      struct ath9k_channel *chan)
+bool ath9k_hw_getnf(struct ath_hw *ah, struct ath9k_channel *chan)
 {
        struct ath_common *common = ath9k_hw_common(ah);
        int16_t nf, nfThresh;
@@ -299,7 +301,7 @@ int16_t ath9k_hw_getnf(struct ath_hw *ah,
        struct ath9k_hw_cal_data *caldata = ah->caldata;
 
        if (!caldata)
-               return ath9k_hw_get_default_nf(ah, chan);
+               return false;
 
        chan->channelFlags &= (~CHANNEL_CW_INT);
        if (REG_READ(ah, AR_PHY_AGC_CONTROL) & AR_PHY_AGC_CONTROL_NF) {
@@ -307,7 +309,7 @@ int16_t ath9k_hw_getnf(struct ath_hw *ah,
                          "NF did not complete in calibration window\n");
                nf = 0;
                caldata->rawNoiseFloor = nf;
-               return caldata->rawNoiseFloor;
+               return false;
        } else {
                ath9k_hw_do_getnf(ah, nfarray);
                ath9k_hw_nf_sanitize(ah, nfarray);
@@ -323,11 +325,10 @@ int16_t ath9k_hw_getnf(struct ath_hw *ah,
        }
 
        h = caldata->nfCalHist;
-
+       caldata->nfcal_pending = false;
        ath9k_hw_update_nfcal_hist_buffer(h, nfarray);
        caldata->rawNoiseFloor = h[0].privNF;
-
-       return ah->caldata->rawNoiseFloor;
+       return true;
 }
 
 void ath9k_init_nfcal_hist_buffer(struct ath_hw *ah,
index ca2e6b39c1d4386e8c18dd1795d037ed631d1108..0a304b3eeeb6dad1e414ee362888900a9d811e24 100644 (file)
@@ -110,8 +110,7 @@ struct ath9k_pacal_info{
 bool ath9k_hw_reset_calvalid(struct ath_hw *ah);
 void ath9k_hw_start_nfcal(struct ath_hw *ah, bool update);
 void ath9k_hw_loadnf(struct ath_hw *ah, struct ath9k_channel *chan);
-int16_t ath9k_hw_getnf(struct ath_hw *ah,
-                      struct ath9k_channel *chan);
+bool ath9k_hw_getnf(struct ath_hw *ah, struct ath9k_channel *chan);
 void ath9k_init_nfcal_hist_buffer(struct ath_hw *ah,
                                  struct ath9k_channel *chan);
 s16 ath9k_hw_getchan_noise(struct ath_hw *ah, struct ath9k_channel *chan);
index c1b701119d880cb4d4bf9923f465a9fd223189e1..399f7c1283cdf32ba4cdb5eef198c4cd2ed085bd 100644 (file)
@@ -354,6 +354,7 @@ struct ath9k_hw_cal_data {
        int8_t qCoff;
        int16_t rawNoiseFloor;
        bool paprd_done;
+       bool nfcal_pending;
        u16 small_signal_gain[AR9300_MAX_CHAINS];
        u32 pa_table[AR9300_MAX_CHAINS][PAPRD_TABLE_SZ];
        struct ath9k_nfcal_hist nfCalHist[NUM_NF_READINGS];