carl9170: reinit phy after HT settings have changed
authorChristian Lamparter <chunkeey@googlemail.com>
Fri, 17 Sep 2010 21:09:19 +0000 (23:09 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Tue, 21 Sep 2010 15:05:25 +0000 (11:05 -0400)
The driver has a set of different initvals for 20 MHz
vs dynamic HT2040 operation. Because we can't change
some of the registers "in-flight", the driver needs to
perform a warm reset.

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

index d43675aca0bb557d9de8f6f600f20571b2092ef8..d7c5482d74ce85a42c0d9008a124e15ba5ef0da4 100644 (file)
@@ -325,6 +325,7 @@ struct ar9170 {
        unsigned int chan_fail;
        unsigned int total_chan_fail;
        u8 heavy_clip;
+       u8 ht_settings;
 
        /* power calibration data */
        u8 power_5G_leg[4];
index fa349b86ea62cc0d7f6ddf5e9a117247bfd9d5cb..89deca37a9888963d4e9cca97b5173c84a528946 100644 (file)
@@ -1610,7 +1610,7 @@ int carl9170_set_channel(struct ar9170 *ar, struct ieee80211_channel *channel,
        const struct carl9170_phy_freq_params *freqpar;
        struct carl9170_rf_init_result rf_res;
        struct carl9170_rf_init rf;
-       u32 cmd, tmp, offs = 0;
+       u32 cmd, tmp, offs = 0, new_ht = 0;
        int err;
        enum carl9170_bw bw;
        bool warm_reset;
@@ -1618,12 +1618,19 @@ int carl9170_set_channel(struct ar9170 *ar, struct ieee80211_channel *channel,
 
        bw = nl80211_to_carl(_bw);
 
+       if (conf_is_ht(&ar->hw->conf))
+               new_ht |= CARL9170FW_PHY_HT_ENABLE;
+
+       if (conf_is_ht40(&ar->hw->conf))
+               new_ht |= CARL9170FW_PHY_HT_DYN2040;
+
        /* may be NULL at first setup */
        if (ar->channel) {
                old_channel = ar->channel;
                warm_reset = (old_channel->band != channel->band) ||
                             (old_channel->center_freq ==
-                             channel->center_freq);
+                             channel->center_freq) ||
+                            (ar->ht_settings != new_ht);
 
                ar->channel = NULL;
        } else {
@@ -1724,16 +1731,9 @@ int carl9170_set_channel(struct ar9170 *ar, struct ieee80211_channel *channel,
 
        freqpar = carl9170_get_hw_dyn_params(channel, bw);
 
-       rf.ht_settings = 0;
-       if (conf_is_ht(&ar->hw->conf)) {
-               rf.ht_settings |= CARL9170FW_PHY_HT_ENABLE;
-
-               if (conf_is_ht40(&ar->hw->conf)) {
-                       rf.ht_settings |= CARL9170FW_PHY_HT_DYN2040;
-                       SET_VAL(CARL9170FW_PHY_HT_EXT_CHAN_OFF,
-                               rf.ht_settings, offs);
-               }
-       }
+       rf.ht_settings = new_ht;
+       if (conf_is_ht40(&ar->hw->conf))
+               SET_VAL(CARL9170FW_PHY_HT_EXT_CHAN_OFF, rf.ht_settings, offs);
 
        rf.freq = cpu_to_le32(channel->center_freq * 1000);
        rf.delta_slope_coeff_exp = cpu_to_le32(freqpar->coeff_exp);
@@ -1805,5 +1805,6 @@ int carl9170_set_channel(struct ar9170 *ar, struct ieee80211_channel *channel,
                ar->ps.off_override &= ~PS_OFF_5GHZ;
 
        ar->channel = channel;
+       ar->ht_settings = new_ht;
        return 0;
 }