ath5k: Stop queues only for NF calibration
authorBruno Randolf <br1@einfach.org>
Wed, 19 May 2010 01:31:05 +0000 (10:31 +0900)
committerJohn W. Linville <linville@tuxdriver.com>
Wed, 2 Jun 2010 20:13:23 +0000 (16:13 -0400)
As far as we know, only NF calibration interferes with RX/TX so we can
leave the queues enabled for the other calibrations.

BTW: Stopping the queues is not enough for avoiding transmissions, since there
might be packets in the queue + beacons are also sent regularly! But i leave it
like this until we have a better solution (stopping TX DMA?).

Signed-off-by: Bruno Randolf <br1@einfach.org>
Acked-by: Nick Kossifidis <mickflemm@gmail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ath/ath5k/base.c

index 3d854f0853acbd304c0a6f0304be2c2720eed114..78a0c420c494175ff550263ec125916964939bd0 100644 (file)
@@ -2786,10 +2786,6 @@ ath5k_tasklet_calibrate(unsigned long data)
        /* Only full calibration for now */
        ah->ah_cal_mask |= AR5K_CALIBRATION_FULL;
 
-       /* Stop queues so that calibration
-        * doesn't interfere with tx */
-       ieee80211_stop_queues(sc->hw);
-
        ATH5K_DBG(sc, ATH5K_DEBUG_CALIBRATE, "channel %u/%x\n",
                ieee80211_frequency_to_channel(sc->curchan->center_freq),
                sc->curchan->hw_value);
@@ -2807,8 +2803,13 @@ ath5k_tasklet_calibrate(unsigned long data)
                        ieee80211_frequency_to_channel(
                                sc->curchan->center_freq));
 
+       /* TODO: We don't need to run noise floor calibration as often
+        * as I/Q calibration.*/
+
+       /* Noise floor calibration interrupts rx/tx path while I/Q calibration
+        * doesn't. Stop queues so that calibration doesn't interfere with tx */
+       ieee80211_stop_queues(sc->hw);
        ath5k_hw_update_noise_floor(ah);
-       /* Wake queues */
        ieee80211_wake_queues(sc->hw);
 
        ah->ah_cal_mask &= ~AR5K_CALIBRATION_FULL;