Re: Re: [ath9k-devel] ath9k module-Unable to set channel-since ath comes

[Davide Pesavento <davidepesa@gmail.com>]

Hi,

On Tue, May 12, 2009 at 09:14, Luis R. Rodriguez <mcgrof@gmail.com> wrote:
> On Tue, May 12, 2009 at 12:11 AM, Johannes Berg
> <johannes@sipsolutions.net> wrote:
>> On Mon, 2009-05-11 at 23:55 -0700, Luis R. Rodriguez wrote:
>>
>>> I've tested this and its not the "ath" module which introduces an
>>> issue its a patch which enables PS by default. You can disable that by
>>> recompiling with:
>>>
>>> CONFIG_MAC80211_DEFAULT_PS_VALUE=0
>>>
>>> If using compat-wireless just stuff it into config.mk.
>>
>> I object to that assertion, or at least this way of putting it.
>>
>> If powersave by default causes issues then it's not my patch that's
>> causing the problem, it's the fact that ath9k can't properly deal with
>> powersave in this case. Had the user enabled powersave manually they
>> would have run into the same issue. As such, I think ath9k should
>> probably simply stop claiming it can support PS.
>
> Heh yes, what I meant was ath9k's PS support has some issues which
> still need to be resolved (at least as I've tested it).
>

I've already reported ath9k's issues with PS multiple times [1] [2]
[3], but the driver hasn't been fixed yet. You can find a verbose
debugging output at [4], I hope that helps in fixing the problem (I
can provide even more info if needed and I'm willing to test patches).

Anyway, if the issue isn't going to be resolved quickly (e.g. before
Johannes' "enable PS by default" patch lands in mainline), I think
it's better to disable PS in ath9k as previously suggested.

Regards,
Davide

[1] http://marc.info/?l=linux-wireless&m=123816406913599&w=2
[2] http://marc.info/?l=linux-wireless&m=123999864507104&w=2
[3] http://marc.info/?l=linux-wireless&m=124025926613089&w=2
[4] http://marc.info/?l=linux-wireless&m=124026772102023&w=2