carl9170: fix noise dBm conversion
authorChristian Lamparter <chunkeey@googlemail.com>
Fri, 17 Sep 2010 20:22:50 +0000 (22:22 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Tue, 21 Sep 2010 15:05:20 +0000 (11:05 -0400)
Ever since carl9170 gained support to read the noisefloor,
the reported noisefloor level was pretty poor.

Initially I assumed that something was wrong in the PHY
setup and it would be impossible to fix without any
guidances. But this was not the case. In fact the nf
readings were correct and the thing that was broken
was the "simple" sign extension code!

Signed-off-by: Christian Lamparter <chunkeey@googlemail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ath/carl9170/phy.c

index fe265e3..7df8f71 100644 (file)
@@ -1558,9 +1558,9 @@ static int carl9170_set_power_cal(struct ar9170 *ar, u32 freq,
 static int carl9170_calc_noise_dbm(u32 raw_noise)
 {
        if (raw_noise & 0x100)
-               return ~((raw_noise & 0x0ff) >> 1);
+               return ~0x1ff | raw_noise;
        else
-               return (raw_noise & 0xff) >> 1;
+               return raw_noise;
 }
 
 int carl9170_get_noisefloor(struct ar9170 *ar)