On Mon, Mar 15, 2010 at 2:43 PM, Luis R. Rodriguez <lrodriguez@xxxxxxxxxxx> wrote: > On Fri, Mar 12, 2010 at 09:46:44PM -0800, Luis R. Rodriguez wrote: >> * I spotted a check added to bail out for tweaking the ACK/CTS >> time out values -- this check should no longer be required >> since Felix sent a proper fix for this recenlty. I removed that >> hunk: >> >> @@ -1210,6 +1195,9 @@ void ath9k_hw_init_global_settings(struct ath_hw *ah) >> REG_WRITE(ah, AR_PCU_MISC, >> REG_READ(ah, AR_PCU_MISC) | ah->misc_mode); >> >> + if (AR_SREV_9271(ah)) >> + return; >> + >> if (conf->channel && conf->channel->band == IEEE80211_BAND_5GHZ) >> sifstime = 16; >> else >> -- > > Turns out this hunk is required... but the rf changes are not (patch 4/12) > so I'll just respin and send a new series out. > > We should look for a better way to address this issue instead of the above hunk > otherwise we will be skipping all changes to the slot time from mac80211. > Right now ath9k_htc does not support this (BSS_CHANGED_ERP_SLOT) or a > set_coverage_class() callback. We'll need to support at least BSS_CHANGED_ERP_SLOT in case the AP switches to compatibility mode. Luis -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html