diff options
| author | Johannes Berg | 2015-08-28 10:52:54 +0200 |
|---|---|---|
| committer | Johannes Berg | 2015-09-22 15:21:26 +0200 |
| commit | 338c17ae311e6b5a439573a4043fd2d9237cd1d5 (patch) | |
| tree | 3383987c5faa4975b424e1eb5fa6e1b8e29678c8 /net/mac80211/util.c | |
| parent | mac80211: fix driver RSSI event calculations (diff) | |
| download | kernel-qcow2-linux-338c17ae311e6b5a439573a4043fd2d9237cd1d5.tar.gz kernel-qcow2-linux-338c17ae311e6b5a439573a4043fd2d9237cd1d5.tar.xz kernel-qcow2-linux-338c17ae311e6b5a439573a4043fd2d9237cd1d5.zip | |
mac80211: use DECLARE_EWMA for ave_beacon_signal
It doesn't seem problematic to change the weight for the average
beacon signal from 3 to 4, so use DECLARE_EWMA. This also makes
the code easier to maintain since bugs like the one fixed in the
previous patch can't happen as easily.
With a fix from Avraham Stern to invert the sign since EMWA uses
unsigned values only.
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/mac80211/util.c')
| -rw-r--r-- | net/mac80211/util.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/mac80211/util.c b/net/mac80211/util.c index f16705675339..9cabf076d718 100644 --- a/net/mac80211/util.c +++ b/net/mac80211/util.c @@ -2543,7 +2543,7 @@ int ieee80211_ave_rssi(struct ieee80211_vif *vif) /* non-managed type inferfaces */ return 0; } - return ifmgd->ave_beacon_signal / 16; + return -ewma_beacon_signal_read(&ifmgd->ave_beacon_signal); } EXPORT_SYMBOL_GPL(ieee80211_ave_rssi); |