Commit 73576110 authored by Christian Lamparter's avatar Christian Lamparter Committed by John W. Linville

carl9170: reinit phy after HT settings have changed

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: default avatarChristian Lamparter <chunkeey@googlemail.com>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 78ec789b
...@@ -325,6 +325,7 @@ struct ar9170 { ...@@ -325,6 +325,7 @@ struct ar9170 {
unsigned int chan_fail; unsigned int chan_fail;
unsigned int total_chan_fail; unsigned int total_chan_fail;
u8 heavy_clip; u8 heavy_clip;
u8 ht_settings;
/* power calibration data */ /* power calibration data */
u8 power_5G_leg[4]; u8 power_5G_leg[4];
......
...@@ -1610,7 +1610,7 @@ int carl9170_set_channel(struct ar9170 *ar, struct ieee80211_channel *channel, ...@@ -1610,7 +1610,7 @@ int carl9170_set_channel(struct ar9170 *ar, struct ieee80211_channel *channel,
const struct carl9170_phy_freq_params *freqpar; const struct carl9170_phy_freq_params *freqpar;
struct carl9170_rf_init_result rf_res; struct carl9170_rf_init_result rf_res;
struct carl9170_rf_init rf; struct carl9170_rf_init rf;
u32 cmd, tmp, offs = 0; u32 cmd, tmp, offs = 0, new_ht = 0;
int err; int err;
enum carl9170_bw bw; enum carl9170_bw bw;
bool warm_reset; bool warm_reset;
...@@ -1618,12 +1618,19 @@ int carl9170_set_channel(struct ar9170 *ar, struct ieee80211_channel *channel, ...@@ -1618,12 +1618,19 @@ int carl9170_set_channel(struct ar9170 *ar, struct ieee80211_channel *channel,
bw = nl80211_to_carl(_bw); 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 */ /* may be NULL at first setup */
if (ar->channel) { if (ar->channel) {
old_channel = ar->channel; old_channel = ar->channel;
warm_reset = (old_channel->band != channel->band) || warm_reset = (old_channel->band != channel->band) ||
(old_channel->center_freq == (old_channel->center_freq ==
channel->center_freq); channel->center_freq) ||
(ar->ht_settings != new_ht);
ar->channel = NULL; ar->channel = NULL;
} else { } else {
...@@ -1724,16 +1731,9 @@ int carl9170_set_channel(struct ar9170 *ar, struct ieee80211_channel *channel, ...@@ -1724,16 +1731,9 @@ int carl9170_set_channel(struct ar9170 *ar, struct ieee80211_channel *channel,
freqpar = carl9170_get_hw_dyn_params(channel, bw); freqpar = carl9170_get_hw_dyn_params(channel, bw);
rf.ht_settings = 0; rf.ht_settings = new_ht;
if (conf_is_ht(&ar->hw->conf)) { if (conf_is_ht40(&ar->hw->conf))
rf.ht_settings |= CARL9170FW_PHY_HT_ENABLE; SET_VAL(CARL9170FW_PHY_HT_EXT_CHAN_OFF, rf.ht_settings, offs);
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.freq = cpu_to_le32(channel->center_freq * 1000); rf.freq = cpu_to_le32(channel->center_freq * 1000);
rf.delta_slope_coeff_exp = cpu_to_le32(freqpar->coeff_exp); 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, ...@@ -1805,5 +1805,6 @@ int carl9170_set_channel(struct ar9170 *ar, struct ieee80211_channel *channel,
ar->ps.off_override &= ~PS_OFF_5GHZ; ar->ps.off_override &= ~PS_OFF_5GHZ;
ar->channel = channel; ar->channel = channel;
ar->ht_settings = new_ht;
return 0; return 0;
} }
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