On Jan 18, 2008 7:51 AM, Bruno Randolf <bruno@xxxxxxxxxxxxx> wrote: > reviewed beacon timer initialization with register traces from madwifi: what we > are doing is correct :). one minor fix: use 3 instead of 0x00000003 - it's more > readable. > > drivers/net/wireless/ath5k/hw.c: Changes-licensed-under: ISC > > Signed-off-by: Bruno Randolf <bruno@xxxxxxxxxxxxx> > --- > > drivers/net/wireless/ath5k/hw.c | 6 ++---- > 1 files changed, 2 insertions(+), 4 deletions(-) > > > diff --git a/drivers/net/wireless/ath5k/hw.c b/drivers/net/wireless/ath5k/hw.c > index eb00818..3a4bf40 100644 > --- a/drivers/net/wireless/ath5k/hw.c > +++ b/drivers/net/wireless/ath5k/hw.c > @@ -2605,10 +2605,8 @@ void ath5k_hw_init_beacon(struct ath5k_hw *ah, u32 next_beacon, u32 interval) > break; > > default: > - timer1 = (next_beacon - AR5K_TUNE_DMA_BEACON_RESP) << > - 0x00000003; > - timer2 = (next_beacon - AR5K_TUNE_SW_BEACON_RESP) << > - 0x00000003; > + timer1 = (next_beacon - AR5K_TUNE_DMA_BEACON_RESP) << 3; > + timer2 = (next_beacon - AR5K_TUNE_SW_BEACON_RESP) << 3; > } > > timer3 = next_beacon + (ah->ah_atim_window ? ah->ah_atim_window : 1); > > Sure why not, Acked-by: Luis R. Rodriguez <mcgrof@xxxxxxxxxxxxxxxxxx> 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
