summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorIvo van Doorn <ivdoorn@gmail.com>2007-09-25 18:53:20 (GMT)
committerDavid S. Miller <davem@sunset.davemloft.net>2007-10-10 23:53:01 (GMT)
commit725d99d4660fcd9abe37d7b733c9412a58465d13 (patch)
tree9b5a436993fc31499d89e8a127aa657d6094d9c4
parent79010420cc3f78eab911598bfdd29c4b06a83e1f (diff)
downloadlinux-fsl-qoriq-725d99d4660fcd9abe37d7b733c9412a58465d13.tar.xz
[PATCH] rt2x00: Correct error in calculating rssi for link tuner
The call to rt2x00lib_precalculate_link_signal resets link.rx_success which is needed when calculating the average rssi for the link tuner. Change the call order so the link tuner runs first as it doesn't need the result of the precalculate. Signed-off-by: Adam Baker <linux@baker-net.org.uk> Signed-off-by: Ivo van Doorn <IvDoorn@gmail.com> Signed-off-by: John W. Linville <linville@tuxdriver.com>
-rw-r--r--drivers/net/wireless/rt2x00/rt2x00dev.c8
1 files changed, 6 insertions, 2 deletions
diff --git a/drivers/net/wireless/rt2x00/rt2x00dev.c b/drivers/net/wireless/rt2x00/rt2x00dev.c
index bbccb89..4f66adc 100644
--- a/drivers/net/wireless/rt2x00/rt2x00dev.c
+++ b/drivers/net/wireless/rt2x00/rt2x00dev.c
@@ -242,8 +242,6 @@ static void rt2x00lib_link_tuner(struct work_struct *work)
rt2x00dev->low_level_stats.dot11FCSErrorCount +=
rt2x00dev->link.rx_failed;
- rt2x00lib_precalculate_link_signal(&rt2x00dev->link);
-
/*
* Only perform the link tuning when Link tuning
* has been enabled (This could have been disabled from the EEPROM).
@@ -252,6 +250,12 @@ static void rt2x00lib_link_tuner(struct work_struct *work)
rt2x00dev->ops->lib->link_tuner(rt2x00dev);
/*
+ * Precalculate a portion of the link signal which is
+ * in based on the tx/rx success/failure counters.
+ */
+ rt2x00lib_precalculate_link_signal(&rt2x00dev->link);
+
+ /*
* Increase tuner counter, and reschedule the next link tuner run.
*/
rt2x00dev->link.count++;