Commit e30eb4ab authored by Joerg Albert's avatar Joerg Albert Committed by John W. Linville

ath5k: fix missing output in monitor mode after ifconfig up

Let ath5k_chan_set() always call ath5k_reset().
This fixes the bug that we don't
get any packets in monitor mode after:

ifconfig wlan0 down
iwconfig wlan0 mode monitor channel 1
ifconfig wlan0 up

but they arrive after

iwconfig wlan0 channel 2
Signed-off-by: default avatarJoerg Albert <jal2@gmx.de>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent c458bc50
...@@ -1071,10 +1071,9 @@ ath5k_setup_bands(struct ieee80211_hw *hw) ...@@ -1071,10 +1071,9 @@ ath5k_setup_bands(struct ieee80211_hw *hw)
} }
/* /*
* Set/change channels. If the channel is really being changed, * Set/change channels. We always reset the chip.
* it's done by reseting the chip. To accomplish this we must * To accomplish this we must first cleanup any pending DMA,
* first cleanup any pending DMA, then restart stuff after a la * then restart stuff after a la ath5k_init.
* ath5k_init.
* *
* Called with sc->lock. * Called with sc->lock.
*/ */
...@@ -1084,9 +1083,6 @@ ath5k_chan_set(struct ath5k_softc *sc, struct ieee80211_channel *chan) ...@@ -1084,9 +1083,6 @@ ath5k_chan_set(struct ath5k_softc *sc, struct ieee80211_channel *chan)
ATH5K_DBG(sc, ATH5K_DEBUG_RESET, "(%u MHz) -> (%u MHz)\n", ATH5K_DBG(sc, ATH5K_DEBUG_RESET, "(%u MHz) -> (%u MHz)\n",
sc->curchan->center_freq, chan->center_freq); sc->curchan->center_freq, chan->center_freq);
if (chan->center_freq != sc->curchan->center_freq ||
chan->hw_value != sc->curchan->hw_value) {
/* /*
* To switch channels clear any pending DMA operations; * To switch channels clear any pending DMA operations;
* wait long enough for the RX fifo to drain, reset the * wait long enough for the RX fifo to drain, reset the
...@@ -1094,9 +1090,6 @@ ath5k_chan_set(struct ath5k_softc *sc, struct ieee80211_channel *chan) ...@@ -1094,9 +1090,6 @@ ath5k_chan_set(struct ath5k_softc *sc, struct ieee80211_channel *chan)
* the relevant bits of the h/w. * the relevant bits of the h/w.
*/ */
return ath5k_reset(sc, chan); return ath5k_reset(sc, chan);
}
return 0;
} }
static void static void
...@@ -2811,9 +2804,11 @@ ath5k_config(struct ieee80211_hw *hw, u32 changed) ...@@ -2811,9 +2804,11 @@ ath5k_config(struct ieee80211_hw *hw, u32 changed)
mutex_lock(&sc->lock); mutex_lock(&sc->lock);
if (changed & IEEE80211_CONF_CHANGE_CHANNEL) {
ret = ath5k_chan_set(sc, conf->channel); ret = ath5k_chan_set(sc, conf->channel);
if (ret < 0) if (ret < 0)
goto unlock; goto unlock;
}
if ((changed & IEEE80211_CONF_CHANGE_POWER) && if ((changed & IEEE80211_CONF_CHANGE_POWER) &&
(sc->power_level != conf->power_level)) { (sc->power_level != conf->power_level)) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment