ath5k: fix off-by-one in pilot magnitude mask
authorBob Copeland <me@bobcopeland.com>
Wed, 25 Aug 2010 01:37:14 +0000 (21:37 -0400)
committerJohn W. Linville <linville@tuxdriver.com>
Fri, 27 Aug 2010 17:27:05 +0000 (13:27 -0400)
If the symbol offset is 46, it will be counted in both
the third and fourth bytes of the mask, and in this
case the shift will be negative which can pollute
high order bits in the mask.  This may negatively impact
OFDM symbol detection.

Signed-off-by: Bob Copeland <me@bobcopeland.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ath/ath5k/phy.c

index 8c27a3644d2f54190172168d4afaca7f8921eb61..984ba92c7df3781fef83719fb8f07431e36ea9be 100644 (file)
@@ -1582,7 +1582,7 @@ ath5k_hw_set_spur_mitigation_filter(struct ath5k_hw *ah,
                        else if (curr_sym_off >= 31 && curr_sym_off <= 46)
                                mag_mask[2] |=
                                        plt_mag_map << (curr_sym_off - 31) * 2;
-                       else if (curr_sym_off >= 46 && curr_sym_off <= 53)
+                       else if (curr_sym_off >= 47 && curr_sym_off <= 53)
                                mag_mask[3] |=
                                        plt_mag_map << (curr_sym_off - 47) * 2;