On Tue, Sep 13, 2011 at 8:18 AM, Rajkumar Manoharan
<rmanohar@xxxxxxxxxxxxxxxx> wrote:
> AR9003 seems to have issues sometimes with fast channel change
> in 5GHz and this case is handled specifically for AR9002 by doing
> a full reset. So let's do a full reset for 5GHz channles for all
> chipsets.
>
> Signed-off-by: Rajkumar Manoharan <rmanohar@xxxxxxxxxxxxxxxx>
> ---
> drivers/net/wireless/ath/ath9k/hw.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/net/wireless/ath/ath9k/hw.c b/drivers/net/wireless/ath/ath9k/hw.c
> index 308847d..622bfc9 100644
> --- a/drivers/net/wireless/ath/ath9k/hw.c
> +++ b/drivers/net/wireless/ath/ath9k/hw.c
> @@ -1504,7 +1504,7 @@ int ath9k_hw_reset(struct ath_hw *ah, struct ath9k_channel *chan,
> (chan->channel != ah->curchan->channel) &&
> ((chan->channelFlags & CHANNEL_ALL) ==
> (ah->curchan->channelFlags & CHANNEL_ALL)) &&
> - (!AR_SREV_9280(ah) || AR_DEVID_7010(ah))) {
> + !IS_CHAN_5GHZ(chan)) {
I'm not sure I understand. Are you asserting that AR9002 does not
need a full reset for channel change in 2GHz? The code before your
change would fall through for AR_SREV_9280(ah) && !AR_DEVID_7010(ah)
whether in 2HGz or 5GHz, but that case gets removed with your change.
Is that intentional?
>
> if (ath9k_hw_channel_change(ah, chan)) {
> ath9k_hw_loadnf(ah, ah->curchan);
> --
> 1.7.6.1
>
> --
> 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
>
--
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
