On Tue, Sep 13, 2011 at 10:38:16AM -0700, Paul Stewart wrote: > On Tue, Sep 13, 2011 at 10:13 AM, Rajkumar Manoharan > <rmanohar@xxxxxxxxxxxxxxxx> wrote: > > On Tue, Sep 13, 2011 at 09:01:29AM -0700, Paul Stewart wrote: > >> 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. > >> > >> 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? > >> > > Yes. Fast channel change is completely supported on 2GHz band. And also > > if you look at the log history, the fast channel change was initially > > disabled in 5GHz alone for AR9280 chips. AR7010 chip is the usb version of > > AR9280. Instead of adding chip specific checks, the fastcc is disabled for > > 5GHz. > > I might not be finding the right part of the history. I do see things > like 0a475cc6a0ac8e38ac3587cff87026551800b7fe: Vasanthakumar > Thiagarajan <vasanth@xxxxxxxxxxx> "ath9k: Do a full reset for AR9280; > AR9280 requires a full reset during channel change and HW reset. > Currently, a fast channel change is done. This patch fixes this bug." > This patch disabled fast-channel-change for the 9280 AND 2GHz channels > (what I assume 5MHZ_SPACED to be filtering). I still see no point in > history where 9280 was enabled for fast channel change in 2Ghz. > True. Let me check on this. Meanwhile I will send v2 patch to disable fastcc on 11A for AR9380 alone. -- Rajkumar -- 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
