On Wed, Jun 15, 2011 at 10:14 PM, Tony Houghton <h...@realh.co.uk> wrote: > On Tue, 14 Jun 2011 18:50:24 +0530 > Mohammed Shafi <shafi.wirel...@gmail.com> wrote: > >> I had removed a check that looks redundant. this will be anyway called >> in radio_disable. >> if some one has time and has easy way of applying the patch like in >> compat wireless, can they see if it fixes by any chance(less >> probably). >> FYI I had gone through the pcipowersave module itself and did not find >> any obvious deviation apart from some junk work around that need to be >> done for AR9285E which does not seems to be related to this >> >> diff --git a/drivers/net/wireless/ath/ath9k/main.c >> b/drivers/net/wireless/ath/ath9k/main.c >> index 7f94533..e7f8ef2 100644 >> --- a/drivers/net/wireless/ath/ath9k/main.c >> +++ b/drivers/net/wireless/ath/ath9k/main.c >> @@ -1259,7 +1259,6 @@ static void ath9k_stop(struct ieee80211_hw *hw) >> >> /* disable HAL and put h/w to sleep */ >> ath9k_hw_disable(ah); >> - ath9k_hw_configpcipowersave(ah, 1, 1); >> >> spin_unlock_bh(&sc->sc_pcu_lock); > > This does seem to work for me (with compat-wireless 2011-06-14). Better > wait for the other affected people to confirm before committing the fix > though.
thats surprising because that just removes a redundant call, please check if you don't have Adrian's work around and also if it works without this patch itself :-) may be you can do some stress test to confirm if you have time > -- shafi _______________________________________________ ath9k-devel mailing list ath9k-devel@lists.ath9k.org https://lists.ath9k.org/mailman/listinfo/ath9k-devel