On Wed, Dec 22, 2010 at 22:44, Luciano Coelho <luciano.coelho@xxxxxxxxx> wrote:
>
> On Wed, 2010-12-22 at 16:27 +0200, ext Arik Nemtsov wrote:
> > Add AP max retries and rate policy configuration.
> > Rename STA rate policy configuration function.
> >
> > Signed-off-by: Arik Nemtsov <arik@xxxxxxxxxx>
> > ---
>
> [...]
>
> > diff --git a/drivers/net/wireless/wl12xx/acx.h b/drivers/net/wireless/wl12xx/acx.h
> > index 9cbc3f4..d2a68bf 100644
> > --- a/drivers/net/wireless/wl12xx/acx.h
> > +++ b/drivers/net/wireless/wl12xx/acx.h
>
> [...]
>
> > + __le32 rate_policy_idx; /* The index of the rate policy */
>
> This comment is quite useless. ;)
Removed :)
>
> > @@ -1062,6 +1072,19 @@ struct wl1271_acx_fw_tsf_information {
> > u8 padding[3];
> > } __packed;
> >
> > +#define WL1271_AP_MAX_TX_RETRY 100
>
> Should this be part of the conf.h settings? Should it be made more
> easily configurable?
Actually this configures the FW for sending up an event once the
number of TX retries was exceeded. For now this event is masked out
and we just set to to a sane minimum (to prevent the FW from working
too hard sending these events).
Recently Johannes sent patches that allow this to be hooked up to
hostapd and actually disassociate the station -
http://www.spinics.net/lists/linux-wireless/msg59831.html
When this is implemented we should perhaps make this easily
configurable. For now conf.h sounds like a good place.
Regards,
Arik
--
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
