On Mon, Mar 2, 2009 at 5:29 PM, Alina Friedrichsen <x-alina@xxxxxxx> wrote:
> The old code doesn't work correctly e.g. on newer chipsets like AR5418+AR2122 and AR5416+AR2133.
>
> Signed-off-by: Alina Friedrichsen <x-alina@xxxxxxx>
> ---
> diff -urN wireless-testing.orig/drivers/net/wireless/ath5k/pcu.c wireless-testing/drivers/net/wireless/ath5k/pcu.c
> --- wireless-testing.orig/drivers/net/wireless/ath5k/pcu.c 2009-01-23 22:54:07.000000000 +0100
> +++ wireless-testing/drivers/net/wireless/ath5k/pcu.c 2009-01-24 08:47:16.000000000 +0100
> @@ -657,9 +657,8 @@
> {
> ATH5K_TRACE(ah->ah_sc);
>
> - ath5k_hw_reg_write(ah, 0x00000000, AR5K_TSF_L32);
> - ath5k_hw_reg_write(ah, (tsf64 >> 32) & 0xffffffff, AR5K_TSF_U32);
> ath5k_hw_reg_write(ah, tsf64 & 0xffffffff, AR5K_TSF_L32);
> + ath5k_hw_reg_write(ah, (tsf64 >> 32) & 0xffffffff, AR5K_TSF_U32);
> }
It's in the original so I guess it's ok, but it'd be nice to have a version
without the unnecessary "& 0xffffffff" part. Also can you use the -p option
to diff in the future?
Otherwise,
Acked-by: Bob Copeland <me@xxxxxxxxxxxxxxx>
--
Bob Copeland %% www.bobcopeland.com
--
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
