ath5k: don't enable probe request rx for STAs
authorBob Copeland <me@bobcopeland.com>
Sun, 15 Aug 2010 17:03:15 +0000 (13:03 -0400)
committerJohn W. Linville <linville@tuxdriver.com>
Tue, 24 Aug 2010 20:28:04 +0000 (16:28 -0400)
AR5K_RX_FILTER_PROBEREQ enables reception of probe requests,
but the filter flag FIF_BCN_PRBRESP_PROMISC is actually about
receiving beacons and probe _responses_, so we shouldn't
turn on the filter when scanning.

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

index 99d56c6f06f1098092d72c5677242ee7394c28a9..f2fceb10c9fcdd11f370f5b907e958420fbc708e 100644 (file)
@@ -3227,9 +3227,9 @@ static void ath5k_configure_filter(struct ieee80211_hw *hw,
                rfilt |= AR5K_RX_FILTER_PHYERR;
 
        /* FIF_BCN_PRBRESP_PROMISC really means to enable beacons
-       * and probes for any BSSID, this needs testing */
+       * and probes for any BSSID */
        if (*new_flags & FIF_BCN_PRBRESP_PROMISC)
-               rfilt |= AR5K_RX_FILTER_BEACON | AR5K_RX_FILTER_PROBEREQ;
+               rfilt |= AR5K_RX_FILTER_BEACON;
 
        /* FIF_CONTROL doc says that if FIF_PROMISC_IN_BSS is not
         * set we should only pass on control frames for this