Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=8de8c5162b157884aa4855564cbfd9ec9119c819
Commit:     8de8c5162b157884aa4855564cbfd9ec9119c819
Parent:     39e75857d08fe35ffad4dd9004580acf0d725b75
Author:     Ivo van Doorn <[EMAIL PROTECTED]>
AuthorDate: Sat Oct 13 16:26:32 2007 +0200
Committer:  David S. Miller <[EMAIL PROTECTED]>
CommitDate: Mon Jan 28 15:02:52 2008 -0800

    [PATCH] rt2x00: Remove rt2x00_clear_link
    
    rt2x00_clear_link() was becoming too large for statically inline,
    also it was used on a single location and shouldn't really be
    used anywhere else. So move the entire code into the function
    rt2x00lib_start_link_tuner()
    
    Signed-off-by: Ivo van Doorn <[EMAIL PROTECTED]>
    Signed-off-by: John W. Linville <[EMAIL PROTECTED]>
    Signed-off-by: David S. Miller <[EMAIL PROTECTED]>
---
 drivers/net/wireless/rt2x00/rt2x00.h    |   18 +++---------------
 drivers/net/wireless/rt2x00/rt2x00dev.c |   16 +++++++++++++++-
 2 files changed, 18 insertions(+), 16 deletions(-)

diff --git a/drivers/net/wireless/rt2x00/rt2x00.h 
b/drivers/net/wireless/rt2x00/rt2x00.h
index c47a1e6..d6f0a72 100644
--- a/drivers/net/wireless/rt2x00/rt2x00.h
+++ b/drivers/net/wireless/rt2x00/rt2x00.h
@@ -277,25 +277,13 @@ struct link {
 };
 
 /*
- * Clear all counters inside the link structure.
- */
-static inline void rt2x00_clear_link(struct link *link)
-{
-       link->count = 0;
-       memset(&link->qual, 0, sizeof(link->qual));
-       link->qual.rx_percentage = 50;
-       link->qual.tx_percentage = 50;
-}
-
-/*
  * Update the rssi using the walking average approach.
  */
 static inline void rt2x00_update_link_rssi(struct link *link, int rssi)
 {
-       if (!link->qual.avg_rssi)
-               link->qual.avg_rssi = rssi;
-       else
-               link->qual.avg_rssi = ((link->qual.avg_rssi * 7) + rssi) / 8;
+       if (link->qual.avg_rssi)
+               rssi = ((link->qual.avg_rssi * 7) + rssi) / 8;
+       link->qual.avg_rssi = rssi;
 }
 
 /*
diff --git a/drivers/net/wireless/rt2x00/rt2x00dev.c 
b/drivers/net/wireless/rt2x00/rt2x00dev.c
index b2016f4..0f824b2 100644
--- a/drivers/net/wireless/rt2x00/rt2x00dev.c
+++ b/drivers/net/wireless/rt2x00/rt2x00dev.c
@@ -67,7 +67,21 @@ EXPORT_SYMBOL_GPL(rt2x00lib_get_ring);
  */
 static void rt2x00lib_start_link_tuner(struct rt2x00_dev *rt2x00dev)
 {
-       rt2x00_clear_link(&rt2x00dev->link);
+       rt2x00dev->link.count = 0;
+       rt2x00dev->link.vgc_level = 0;
+
+       memset(&rt2x00dev->link.qual, 0, sizeof(rt2x00dev->link.qual));
+
+       /*
+        * The RX and TX percentage should start at 50%
+        * this will assure we will get at least get some
+        * decent value when the link tuner starts.
+        * The value will be dropped and overwritten with
+        * the correct (measured )value anyway during the
+        * first run of the link tuner.
+        */
+       rt2x00dev->link.qual.rx_percentage = 50;
+       rt2x00dev->link.qual.tx_percentage = 50;
 
        /*
         * Reset the link tuner.
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to