On Tue, Dec 22, 2009 at 8:06 PM, Sujith <Sujith.Manoharan@xxxxxxxxxxx> wrote:
> Luis Rodriguez wrote:
>> > diff --git a/drivers/net/wireless/ath/ath9k/main.c b/drivers/net/wireless/ath/ath9k/main.c
>> > index 419c382..7fdcc6b 100644
>> > --- a/drivers/net/wireless/ath/ath9k/main.c
>> > +++ b/drivers/net/wireless/ath/ath9k/main.c
>> > @@ -2673,6 +2673,11 @@ static int ath9k_config(struct ieee80211_hw *hw, u32 changed)
>> >
>> > mutex_lock(&sc->mutex);
>> >
>> > + if (sc->sc_flags & SC_OP_INVALID) {
>> > + mutex_unlock(&sc->mutex);
>> > + return 0;
>> > + }
>> > +
>> > /*
>> > * Leave this as the first check because we need to turn on the
>> > * radio if it was disabled before prior to processing the rest
>>
>> That ones busts the resume.
>
> Weird, checking for the device status before allowing configuration
> changes is valid. Not sure why this is happening.
The proper and more clean solution is to see where in mac80211 this
gets called and avoid the call when local->suspended if
local->suspended is true after driver->stop() I forget.
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
