ath9k: Use proper TX channel width for setting channels

The TX channel width of the BSS can be obtained only after association.
In all cases, default to HT20 if HT is enabled, and set
chan width to HT40 only if the BSS supports it.

Signed-off-by: Sujith <Sujith.Manoharan@atheros.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>

diff --git a/drivers/net/wireless/ath9k/core.h b/drivers/net/wireless/ath9k/core.h
index 21ee0c24052877175b3c5420f51e25b74a132bc7..f0c54377dfe6748dde7004f3a1b574bf62aefa70 100644 (file)
--- a/drivers/net/wireless/ath9k/core.h
+++ b/drivers/net/wireless/ath9k/core.h
@@ -633,11 +633,6 @@ enum PROT_MODE {
        PROT_M_CTSONLY
 };
 
-struct ath_ht_info {
-       enum ath9k_ht_macmode tx_chan_width;
-       u8 ext_chan_offset;
-};
-
 #define SC_OP_INVALID          BIT(0)
 #define SC_OP_BEACONS          BIT(1)
 #define SC_OP_RXAGGR           BIT(2)
@@ -691,8 +686,8 @@ struct ath_softc {
        u8 sc_rxotherant;               /* rx's on non-default antenna */
 
        struct ath9k_node_stats sc_halstats; /* station-mode rssi stats */
-       struct ath_ht_info sc_ht_info;
        enum ath9k_ht_extprotspacing sc_ht_extprotspacing;
+       enum ath9k_ht_macmode tx_chan_width;
 
 #ifdef CONFIG_SLOW_ANT_DIV
        struct ath_antdiv sc_antdiv;

diff --git a/drivers/net/wireless/ath9k/hw.c b/drivers/net/wireless/ath9k/hw.c
index 10f6c08f6eb2530891fee7671fb64a72c327d082..6eef10477896cdce5b52833dc87b5510ac2f0572 100644 (file)
--- a/drivers/net/wireless/ath9k/hw.c
+++ b/drivers/net/wireless/ath9k/hw.c
@@ -2232,8 +2232,7 @@ bool ath9k_hw_reset(struct ath_hal *ah, struct ath9k_channel *chan,
            ((chan->channelFlags & CHANNEL_ALL) ==
             (ah->ah_curchan->channelFlags & CHANNEL_ALL)) &&
            (!AR_SREV_9280(ah) || (!IS_CHAN_A_5MHZ_SPACED(chan) &&
-                                  !IS_CHAN_A_5MHZ_SPACED(ah->
-                                                         ah_curchan)))) {
+                                  !IS_CHAN_A_5MHZ_SPACED(ah->ah_curchan)))) {
 
                if (ath9k_hw_channel_change(ah, chan, macmode)) {
                        ath9k_hw_loadnf(ah, ah->ah_curchan);

diff --git a/drivers/net/wireless/ath9k/main.c b/drivers/net/wireless/ath9k/main.c
index f226a4daef758c01b0d19f08adc9cf1d1464b317..437e38a3c6d75064d2b0d8f3f7f4538ac7381a68 100644 (file)
--- a/drivers/net/wireless/ath9k/main.c
+++ b/drivers/net/wireless/ath9k/main.c
@@ -260,15 +260,6 @@ static int ath_set_channel(struct ath_softc *sc, struct ath9k_channel *hchan)
        if (sc->sc_flags & SC_OP_INVALID)
                return -EIO;
 
-       DPRINTF(sc, ATH_DBG_CONFIG,
-               "%s: %u (%u MHz) -> %u (%u MHz), cflags:%x\n",
-               __func__,
-               ath9k_hw_mhz2ieee(ah, sc->sc_ah->ah_curchan->channel,
-                                 sc->sc_ah->ah_curchan->channelFlags),
-               sc->sc_ah->ah_curchan->channel,
-               ath9k_hw_mhz2ieee(ah, hchan->channel, hchan->channelFlags),
-               hchan->channel, hchan->channelFlags);
-
        if (hchan->channel != sc->sc_ah->ah_curchan->channel ||
            hchan->channelFlags != sc->sc_ah->ah_curchan->channelFlags ||
            (sc->sc_flags & SC_OP_CHAINMASK_UPDATE) ||
@@ -294,8 +285,14 @@ static int ath_set_channel(struct ath_softc *sc, struct ath9k_channel *hchan)
                if (!stopped || (sc->sc_flags & SC_OP_FULL_RESET))
                        fastcc = false;
 
+               DPRINTF(sc, ATH_DBG_CONFIG,
+                       "%s: (%u MHz) -> (%u MHz), cflags:%x, chanwidth: %d\n",
+                       __func__,
+                       sc->sc_ah->ah_curchan->channel,
+                       hchan->channel, hchan->channelFlags, sc->tx_chan_width);
+
                spin_lock_bh(&sc->sc_resetlock);
-               if (!ath9k_hw_reset(ah, hchan, sc->sc_ht_info.tx_chan_width,
+               if (!ath9k_hw_reset(ah, hchan, sc->tx_chan_width,
                                    sc->sc_tx_chainmask, sc->sc_rx_chainmask,
                                    sc->sc_ht_extprotspacing, fastcc, &status)) {
                        DPRINTF(sc, ATH_DBG_FATAL,
@@ -626,11 +623,13 @@ static int ath_get_channel(struct ath_softc *sc,
 }
 
 static u32 ath_get_extchanmode(struct ath_softc *sc,
-                                    struct ieee80211_channel *chan)
+                              struct ieee80211_channel *chan,
+                              struct ieee80211_bss_conf *bss_conf)
 {
        u32 chanmode = 0;
-       u8 ext_chan_offset = sc->sc_ht_info.ext_chan_offset;
-       enum ath9k_ht_macmode tx_chan_width = sc->sc_ht_info.tx_chan_width;
+       u8 ext_chan_offset = bss_conf->ht.secondary_channel_offset;
+       enum ath9k_ht_macmode tx_chan_width = (bss_conf->ht.width_40_ok) ?
+               ATH9K_HT_MACMODE_2040 : ATH9K_HT_MACMODE_20;
 
        switch (chan->band) {
        case IEEE80211_BAND_2GHZ:
@@ -828,17 +827,17 @@ static void setup_ht_cap(struct ieee80211_sta_ht_cap *ht_info)
 static void ath9k_ht_conf(struct ath_softc *sc,
                          struct ieee80211_bss_conf *bss_conf)
 {
-       struct ath_ht_info *ht_info = &sc->sc_ht_info;
-
        if (sc->hw->conf.ht.enabled) {
-               ht_info->ext_chan_offset = bss_conf->ht.secondary_channel_offset;
-
                if (bss_conf->ht.width_40_ok)
-                       ht_info->tx_chan_width = ATH9K_HT_MACMODE_2040;
+                       sc->tx_chan_width = ATH9K_HT_MACMODE_2040;
                else
-                       ht_info->tx_chan_width = ATH9K_HT_MACMODE_20;
+                       sc->tx_chan_width = ATH9K_HT_MACMODE_20;
 
-               ath9k_hw_set11nmac2040(sc->sc_ah, ht_info->tx_chan_width);
+               ath9k_hw_set11nmac2040(sc->sc_ah, sc->tx_chan_width);
+
+               DPRINTF(sc, ATH_DBG_CONFIG,
+                       "%s: BSS Changed HT, chanwidth: %d\n",
+                       __func__, sc->tx_chan_width);
        }
 }
 
@@ -892,26 +891,30 @@ static void ath9k_bss_assoc_info(struct ath_softc *sc,
                        return;
                }
 
-               if (hw->conf.ht.enabled)
+               if (hw->conf.ht.enabled) {
                        sc->sc_ah->ah_channels[pos].chanmode =
-                               ath_get_extchanmode(sc, curchan);
-               else
+                               ath_get_extchanmode(sc, curchan, bss_conf);
+
+                       if (bss_conf->ht.width_40_ok)
+                               sc->tx_chan_width = ATH9K_HT_MACMODE_2040;
+                       else
+                               sc->tx_chan_width = ATH9K_HT_MACMODE_20;
+               } else {
                        sc->sc_ah->ah_channels[pos].chanmode =
                                (curchan->band == IEEE80211_BAND_2GHZ) ?
                                CHANNEL_G : CHANNEL_A;
+               }
 
                /* set h/w channel */
                if (ath_set_channel(sc, &sc->sc_ah->ah_channels[pos]) < 0)
                        DPRINTF(sc, ATH_DBG_FATAL,
-                               "%s: Unable to set channel\n",
-                               __func__);
+                               "%s: Unable to set channel\n", __func__);
                /* Start ANI */
                mod_timer(&sc->sc_ani.timer,
                        jiffies + msecs_to_jiffies(ATH_ANI_POLLINTERVAL));
 
        } else {
-               DPRINTF(sc, ATH_DBG_CONFIG,
-               "%s: Bss Info DISSOC\n", __func__);
+               DPRINTF(sc, ATH_DBG_CONFIG, "%s: Bss Info DISSOC\n", __func__);
                sc->sc_curaid = 0;
        }
 }
@@ -1044,7 +1047,7 @@ static void ath_radio_enable(struct ath_softc *sc)
 
        spin_lock_bh(&sc->sc_resetlock);
        if (!ath9k_hw_reset(ah, ah->ah_curchan,
-                           sc->sc_ht_info.tx_chan_width,
+                           sc->tx_chan_width,
                            sc->sc_tx_chainmask,
                            sc->sc_rx_chainmask,
                            sc->sc_ht_extprotspacing,
@@ -1102,7 +1105,7 @@ static void ath_radio_disable(struct ath_softc *sc)
 
        spin_lock_bh(&sc->sc_resetlock);
        if (!ath9k_hw_reset(ah, ah->ah_curchan,
-                           sc->sc_ht_info.tx_chan_width,
+                           sc->tx_chan_width,
                            sc->sc_tx_chainmask,
                            sc->sc_rx_chainmask,
                            sc->sc_ht_extprotspacing,
@@ -1636,7 +1639,7 @@ int ath_reset(struct ath_softc *sc, bool retry_tx)
 
        spin_lock_bh(&sc->sc_resetlock);
        if (!ath9k_hw_reset(ah, sc->sc_ah->ah_curchan,
-                           sc->sc_ht_info.tx_chan_width,
+                           sc->tx_chan_width,
                            sc->sc_tx_chainmask, sc->sc_rx_chainmask,
                            sc->sc_ht_extprotspacing, false, &status)) {
                DPRINTF(sc, ATH_DBG_FATAL,
@@ -1864,8 +1867,6 @@ static int ath9k_start(struct ieee80211_hw *hw)
        DPRINTF(sc, ATH_DBG_CONFIG, "%s: Starting driver with "
                "initial channel: %d MHz\n", __func__, curchan->center_freq);
 
-       memset(&sc->sc_ht_info, 0, sizeof(struct ath_ht_info));
-
        /* setup initial channel */
 
        pos = ath_get_channel(sc, curchan);
@@ -1875,6 +1876,7 @@ static int ath9k_start(struct ieee80211_hw *hw)
                goto error;
        }
 
+       sc->tx_chan_width = ATH9K_HT_MACMODE_20;
        sc->sc_ah->ah_channels[pos].chanmode =
                (curchan->band == IEEE80211_BAND_2GHZ) ? CHANNEL_G : CHANNEL_A;
        init_channel = &sc->sc_ah->ah_channels[pos];
@@ -1891,7 +1893,7 @@ static int ath9k_start(struct ieee80211_hw *hw)
         */
        spin_lock_bh(&sc->sc_resetlock);
        if (!ath9k_hw_reset(sc->sc_ah, init_channel,
-                           sc->sc_ht_info.tx_chan_width,
+                           sc->tx_chan_width,
                            sc->sc_tx_chainmask, sc->sc_rx_chainmask,
                            sc->sc_ht_extprotspacing, false, &status)) {
                DPRINTF(sc, ATH_DBG_FATAL,
@@ -2149,39 +2151,37 @@ static void ath9k_remove_interface(struct ieee80211_hw *hw,
 static int ath9k_config(struct ieee80211_hw *hw, u32 changed)
 {
        struct ath_softc *sc = hw->priv;
-       struct ieee80211_channel *curchan = hw->conf.channel;
        struct ieee80211_conf *conf = &hw->conf;
-       int pos;
-
-       DPRINTF(sc, ATH_DBG_CONFIG, "%s: Set channel: %d MHz\n",
-               __func__,
-               curchan->center_freq);
 
-       /* Update chainmask */
-       ath_update_chainmask(sc, conf->ht.enabled);
+       if (changed & IEEE80211_CONF_CHANGE_CHANNEL) {
+               struct ieee80211_channel *curchan = hw->conf.channel;
+               int pos;
 
-       pos = ath_get_channel(sc, curchan);
-       if (pos == -1) {
-               DPRINTF(sc, ATH_DBG_FATAL, "%s: Invalid channel\n", __func__);
-               return -EINVAL;
-       }
+               DPRINTF(sc, ATH_DBG_CONFIG, "%s: Set channel: %d MHz\n",
+                       __func__, curchan->center_freq);
 
-       sc->sc_ah->ah_channels[pos].chanmode =
-               (curchan->band == IEEE80211_BAND_2GHZ) ?
-               CHANNEL_G : CHANNEL_A;
+               pos = ath_get_channel(sc, curchan);
+               if (pos == -1) {
+                       DPRINTF(sc, ATH_DBG_FATAL, "%s: Invalid channel\n", __func__);
+                       return -EINVAL;
+               }
 
-       if (sc->sc_curaid && hw->conf.ht.enabled)
+               sc->tx_chan_width = ATH9K_HT_MACMODE_20;
                sc->sc_ah->ah_channels[pos].chanmode =
-                       ath_get_extchanmode(sc, curchan);
+                       (curchan->band == IEEE80211_BAND_2GHZ) ?
+                       CHANNEL_G : CHANNEL_A;
+
+               if (ath_set_channel(sc, &sc->sc_ah->ah_channels[pos]) < 0)
+                       DPRINTF(sc, ATH_DBG_FATAL,
+                               "%s: Unable to set channel\n", __func__);
+       }
+
+       if (changed & IEEE80211_CONF_CHANGE_HT)
+               ath_update_chainmask(sc, conf->ht.enabled);
 
        if (changed & IEEE80211_CONF_CHANGE_POWER)
                sc->sc_config.txpowlimit = 2 * conf->power_level;
 
-       /* set h/w channel */
-       if (ath_set_channel(sc, &sc->sc_ah->ah_channels[pos]) < 0)
-               DPRINTF(sc, ATH_DBG_FATAL, "%s: Unable to set channel\n",
-                       __func__);
-
        return 0;
 }
 
@@ -2421,11 +2421,8 @@ static void ath9k_bss_info_changed(struct ieee80211_hw *hw,
                        sc->sc_flags &= ~SC_OP_PROTECT_ENABLE;
        }
 
-       if (changed & BSS_CHANGED_HT) {
-               DPRINTF(sc, ATH_DBG_CONFIG, "%s: BSS Changed HT\n",
-                       __func__);
+       if (changed & BSS_CHANGED_HT)
                ath9k_ht_conf(sc, bss_conf);
-       }
 
        if (changed & BSS_CHANGED_ASSOC) {
                DPRINTF(sc, ATH_DBG_CONFIG, "%s: BSS Changed ASSOC %d\n",

diff --git a/drivers/net/wireless/ath9k/xmit.c b/drivers/net/wireless/ath9k/xmit.c
index 3de60c51e5f21d3175ab4a1e9747a8ab1aead4c8..413fbdd38ab62959c7abeeb4a81ec62211704d8e 100644 (file)
--- a/drivers/net/wireless/ath9k/xmit.c
+++ b/drivers/net/wireless/ath9k/xmit.c
@@ -1148,7 +1148,7 @@ static void ath_drain_txdataq(struct ath_softc *sc, bool retry_tx)
                spin_lock_bh(&sc->sc_resetlock);
                if (!ath9k_hw_reset(ah,
                                    sc->sc_ah->ah_curchan,
-                                   sc->sc_ht_info.tx_chan_width,
+                                   sc->tx_chan_width,
                                    sc->sc_tx_chainmask, sc->sc_rx_chainmask,
                                    sc->sc_ht_extprotspacing, true, &status)) {