On 2014-02-28 19:43, Sujith Manoharan wrote:
> Felix Fietkau wrote:
>> - if (ah->opmode != NL80211_IFTYPE_AP &&
>> - aniState->ofdmWeakSigDetect != weak_sig) {
>> - ath9k_hw_ani_control(ah,
>> - ATH9K_ANI_OFDM_WEAK_SIGNAL_DETECTION,
>> - entry_ofdm->ofdm_weak_signal_on);
>> - }
>> + else if (AR_SREV_9300_20_OR_LATER(ah) &&
>> + ah->opmode != NL80211_IFTYPE_STATION)
>> + weak_sig = true;
>> +
>> + if (aniState->ofdmWeakSigDetect != weak_sig)
>> + ath9k_hw_ani_control(ah, ATH9K_ANI_OFDM_WEAK_SIGNAL_DETECTION,
>> + weak_sig);
>
> Any reason why this has been changed to disregard the WS-DET parameter from
> ofdm_level_table[] ?
It doesn't disregard it - weak_sig is initialized to the value of that
parameter. Comparing aniState->ofdmWeakSigDetect against weak_sig, but
then passing entry_ofdm->ofdm_weak_signal_on is buggy.
- Felix
--
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
