rt2x00: Fix quality houskeeping for software diversity
[pandora-kernel.git] / drivers / net / wireless / rt2x00 / rt2x00link.c
index eb9b981..dd68f3a 100644 (file)
@@ -103,39 +103,21 @@ static int rt2x00link_antenna_get_link_rssi(struct rt2x00_dev *rt2x00dev)
        return DEFAULT_RSSI;
 }
 
-static int rt2x00link_antenna_get_rssi_history(struct rt2x00_dev *rt2x00dev,
-                                              enum antenna antenna)
+static int rt2x00link_antenna_get_rssi_history(struct rt2x00_dev *rt2x00dev)
 {
        struct link_ant *ant = &rt2x00dev->link.ant;
 
-       if (ant->rssi_history[antenna - ANTENNA_A])
-               return ant->rssi_history[antenna - ANTENNA_A];
+       if (ant->rssi_history)
+               return ant->rssi_history;
        return DEFAULT_RSSI;
 }
-/* Small wrapper for rt2x00link_antenna_get_rssi_history() */
-#define rt2x00link_antenna_get_rssi_rx_history(__dev) \
-       rt2x00link_antenna_get_rssi_history((__dev), \
-                                           (__dev)->link.ant.active.rx)
-#define rt2x00link_antenna_get_rssi_tx_history(__dev) \
-       rt2x00link_antenna_get_rssi_history((__dev), \
-                                           (__dev)->link.ant.active.tx)
 
 static void rt2x00link_antenna_update_rssi_history(struct rt2x00_dev *rt2x00dev,
-                                                  enum antenna antenna,
                                                   int rssi)
 {
        struct link_ant *ant = &rt2x00dev->link.ant;
-       ant->rssi_history[ant->active.rx - ANTENNA_A] = rssi;
+       ant->rssi_history = rssi;
 }
-/* Small wrapper for rt2x00link_antenna_get_rssi_history() */
-#define rt2x00link_antenna_update_rssi_rx_history(__dev, __rssi) \
-       rt2x00link_antenna_update_rssi_history((__dev), \
-                                              (__dev)->link.ant.active.rx, \
-                                              (__rssi))
-#define rt2x00link_antenna_update_rssi_tx_history(__dev, __rssi) \
-       rt2x00link_antenna_update_rssi_history((__dev), \
-                                              (__dev)->link.ant.active.tx, \
-                                              (__rssi))
 
 static void rt2x00link_antenna_reset(struct rt2x00_dev *rt2x00dev)
 {
@@ -146,8 +128,10 @@ static void rt2x00lib_antenna_diversity_sample(struct rt2x00_dev *rt2x00dev)
 {
        struct link_ant *ant = &rt2x00dev->link.ant;
        struct antenna_setup new_ant;
-       int sample_a = rt2x00link_antenna_get_rssi_history(rt2x00dev, ANTENNA_A);
-       int sample_b = rt2x00link_antenna_get_rssi_history(rt2x00dev, ANTENNA_B);
+       int other_antenna;
+
+       int sample_current = rt2x00link_antenna_get_link_rssi(rt2x00dev);
+       int sample_other = rt2x00link_antenna_get_rssi_history(rt2x00dev);
 
        memcpy(&new_ant, &ant->active, sizeof(new_ant));
 
@@ -158,22 +142,27 @@ static void rt2x00lib_antenna_diversity_sample(struct rt2x00_dev *rt2x00dev)
 
        /*
         * During the last period we have sampled the RSSI
-        * from both antenna's. It now is time to determine
+        * from both antennas. It now is time to determine
         * which antenna demonstrated the best performance.
         * When we are already on the antenna with the best
-        * performance, then there really is nothing for us
-        * left to do.
+        * performance, just create a good starting point
+        * for the history and we are done.
         */
-       if (sample_a == sample_b)
+       if (sample_current >= sample_other) {
+               rt2x00link_antenna_update_rssi_history(rt2x00dev,
+                       sample_current);
                return;
+       }
+
+       other_antenna = (ant->active.rx == ANTENNA_A) ? ANTENNA_B : ANTENNA_A;
 
        if (ant->flags & ANTENNA_RX_DIVERSITY)
-               new_ant.rx = (sample_a > sample_b) ? ANTENNA_A : ANTENNA_B;
+               new_ant.rx = other_antenna;
 
        if (ant->flags & ANTENNA_TX_DIVERSITY)
-               new_ant.tx = (sample_a > sample_b) ? ANTENNA_A : ANTENNA_B;
+               new_ant.tx = other_antenna;
 
-       rt2x00lib_config_antenna(rt2x00dev, &new_ant);
+       rt2x00lib_config_antenna(rt2x00dev, new_ant);
 }
 
 static void rt2x00lib_antenna_diversity_eval(struct rt2x00_dev *rt2x00dev)
@@ -190,8 +179,8 @@ static void rt2x00lib_antenna_diversity_eval(struct rt2x00_dev *rt2x00dev)
         * after that update the history with the current value.
         */
        rssi_curr = rt2x00link_antenna_get_link_rssi(rt2x00dev);
-       rssi_old = rt2x00link_antenna_get_rssi_rx_history(rt2x00dev);
-       rt2x00link_antenna_update_rssi_rx_history(rt2x00dev, rssi_curr);
+       rssi_old = rt2x00link_antenna_get_rssi_history(rt2x00dev);
+       rt2x00link_antenna_update_rssi_history(rt2x00dev, rssi_curr);
 
        /*
         * Legacy driver indicates that we should swap antenna's
@@ -213,12 +202,13 @@ static void rt2x00lib_antenna_diversity_eval(struct rt2x00_dev *rt2x00dev)
        if (ant->flags & ANTENNA_TX_DIVERSITY)
                new_ant.tx = (new_ant.tx == ANTENNA_A) ? ANTENNA_B : ANTENNA_A;
 
-       rt2x00lib_config_antenna(rt2x00dev, &new_ant);
+       rt2x00lib_config_antenna(rt2x00dev, new_ant);
 }
 
-static void rt2x00lib_antenna_diversity(struct rt2x00_dev *rt2x00dev)
+static bool rt2x00lib_antenna_diversity(struct rt2x00_dev *rt2x00dev)
 {
        struct link_ant *ant = &rt2x00dev->link.ant;
+       unsigned int flags = ant->flags;
 
        /*
         * Determine if software diversity is enabled for
@@ -226,30 +216,38 @@ static void rt2x00lib_antenna_diversity(struct rt2x00_dev *rt2x00dev)
         * Always perform this check since within the link
         * tuner interval the configuration might have changed.
         */
-       ant->flags &= ~ANTENNA_RX_DIVERSITY;
-       ant->flags &= ~ANTENNA_TX_DIVERSITY;
+       flags &= ~ANTENNA_RX_DIVERSITY;
+       flags &= ~ANTENNA_TX_DIVERSITY;
 
        if (rt2x00dev->default_ant.rx == ANTENNA_SW_DIVERSITY)
-               ant->flags |= ANTENNA_RX_DIVERSITY;
+               flags |= ANTENNA_RX_DIVERSITY;
        if (rt2x00dev->default_ant.tx == ANTENNA_SW_DIVERSITY)
-               ant->flags |= ANTENNA_TX_DIVERSITY;
+               flags |= ANTENNA_TX_DIVERSITY;
 
        if (!(ant->flags & ANTENNA_RX_DIVERSITY) &&
            !(ant->flags & ANTENNA_TX_DIVERSITY)) {
                ant->flags = 0;
-               return;
+               return true;
        }
 
+       /* Update flags */
+       ant->flags = flags;
+
        /*
         * If we have only sampled the data over the last period
         * we should now harvest the data. Otherwise just evaluate
         * the data. The latter should only be performed once
         * every 2 seconds.
         */
-       if (ant->flags & ANTENNA_MODE_SAMPLE)
+       if (ant->flags & ANTENNA_MODE_SAMPLE) {
                rt2x00lib_antenna_diversity_sample(rt2x00dev);
-       else if (rt2x00dev->link.count & 1)
+               return true;
+       } else if (rt2x00dev->link.count & 1) {
                rt2x00lib_antenna_diversity_eval(rt2x00dev);
+               return true;
+       }
+
+       return false;
 }
 
 void rt2x00link_update_stats(struct rt2x00_dev *rt2x00dev,
@@ -351,8 +349,8 @@ void rt2x00link_start_tuner(struct rt2x00_dev *rt2x00dev)
 
        rt2x00link_reset_tuner(rt2x00dev, false);
 
-       queue_delayed_work(rt2x00dev->hw->workqueue,
-                          &link->work, LINK_TUNE_INTERVAL);
+       ieee80211_queue_delayed_work(rt2x00dev->hw,
+                                    &link->work, LINK_TUNE_INTERVAL);
 }
 
 void rt2x00link_stop_tuner(struct rt2x00_dev *rt2x00dev)
@@ -447,22 +445,19 @@ static void rt2x00link_tuner(struct work_struct *work)
        rt2x00leds_led_quality(rt2x00dev, link->avg_rssi);
 
        /*
-        * Evaluate antenna setup, make this the last step since this could
-        * possibly reset some statistics.
-        */
-       rt2x00lib_antenna_diversity(rt2x00dev);
-
-       /*
-        * Reset the quality counters which recounted during each period.
+        * Evaluate antenna setup, make this the last step when
+        * rt2x00lib_antenna_diversity made changes the quality
+        * statistics will be reset.
         */
-       rt2x00link_reset_qual(rt2x00dev);
+       if (rt2x00lib_antenna_diversity(rt2x00dev))
+               rt2x00link_reset_qual(rt2x00dev);
 
        /*
         * Increase tuner counter, and reschedule the next link tuner run.
         */
        link->count++;
-       queue_delayed_work(rt2x00dev->hw->workqueue,
-                          &link->work, LINK_TUNE_INTERVAL);
+       ieee80211_queue_delayed_work(rt2x00dev->hw,
+                                    &link->work, LINK_TUNE_INTERVAL);
 }
 
 void rt2x00link_register(struct rt2x00_dev *rt2x00dev)