On 08/05/2009 04:04 AM, Christian Lamparter wrote:
> no signed/cc here?
oops, forgot it. Seems like both patches miss some context lines at the end, too.
> > ...
> the beacon timer isn't exclusively used to notify the driver when its
> time for a new beacon... The STA mode uses the same _timer_
> in reverse to wait for the next beacon form the assoc. AP.
> that said: It does not look like the firmware implements anything
> in this direction... But this is a clearly MAC register and there
> could be something in the silicon which does something useful with
> this information.
I wasn't aware of this usage within STA mode.
> so, to be on the safe side: why not preserve the old behavior for the
> STA mode as well and simply tell the hardware about dtim & beacon interval?
ar9170_set_beacon_timers() uses the beacon interval only if for AP mode, dtim is left in STA mode.
> The only remaining question is where to disabled the timer for STA.
> (which is in some way relevant to: [PATCH 2/2] because previously,
> these timers were always disabled by remove_interface.)
>
> I think the best place is in ar9170_op_bss_info_changed:
> if (changed & BSS_CHANGED_ASSOC) {
>
> just when bss_conf->assoc gets "0".
Yes. IMHO also enable on bss_conf->assoc == 1.
I'll look into another version of both patches after running some tests in STA mode.
Regards,
Joerg.
--
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
