Date: Wed, 24 Aug 2011 22:04:35 +0800 From: Kang Yin Su <cantona@cantona.net> To: Adrian Chadd <adrian@freebsd.org> Cc: freebsd-wireless@freebsd.org Subject: Re: [Patch] Set timers mode for AR5416 on TDMA support Message-ID: <CAHjFwoBrnUVkGoP4L1Uuq%2B=NXbyUkEjxmR41zUwSyHUkpzeiFA@mail.gmail.com> In-Reply-To: <CAJ-VmoksXS%2Bq5kcL09mF-zVJ9msUENZdb_j1%2BLj_Y5c7AdSTsg@mail.gmail.com> References: <CAHjFwoAyYYSD1VpW7f_=OSQ04paWUhzPJA%2BuT67s_OS124hbjQ@mail.gmail.com> <CAJ-Vmo=heE_3LTgANyhnUjKEUAS2GEKFr1w%2BcxD39Ss5LAp3=Q@mail.gmail.com> <CAHjFwoAgPg7YHJLdP=mX7kq-J4R8CnG_-dcJPSuPPZhTT2BBFQ@mail.gmail.com> <CAJ-Vmokt3%2BL4brGM5AcaNV1BhNBrv4C6FedxPaowO68GwDb9fg@mail.gmail.com> <CAJ-VmoksXS%2Bq5kcL09mF-zVJ9msUENZdb_j1%2BLj_Y5c7AdSTsg@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
That works too I tried, I just think explicit the mode is better. thanks, Yin On Wed, Aug 24, 2011 at 9:27 PM, Adrian Chadd <adrian@freebsd.org> wrote: > Hi, > > Please try this patch. I bet bt_flags isn't 0 sometimes and the "right > bits" aren't set. :) > > > Adrian > > Index: if_ath.c > =================================================================== > --- if_ath.c (revision 224813) > +++ if_ath.c (working copy) > @@ -5485,6 +5485,8 @@ > bt.bt_nextdba = (nexttbtt<<3) - sc->sc_tdmadbaprep; > bt.bt_nextswba = (nexttbtt<<3) - sc->sc_tdmaswbaprep; > bt.bt_nextatim = nexttbtt+1; > + /* Enables TBTT, DBA, SWBA timers by default */ > + bt.bt_flags = 0; > ath_hal_beaconsettimers(ah, &bt); > } >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAHjFwoBrnUVkGoP4L1Uuq%2B=NXbyUkEjxmR41zUwSyHUkpzeiFA>