Date: Sun, 8 May 2011 15:55:52 +0000 (UTC) From: Adrian Chadd <adrian@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/sys/dev/ath/ath_hal ah_internal.h src/sys/dev/ath/ath_hal/ar9002 ar9280_attach.c Message-ID: <201105081556.p48Fu7OK024675@repoman.freebsd.org>
index | next in thread | raw e-mail
adrian 2011-05-08 15:55:52 UTC
FreeBSD src repository
Modified files:
sys/dev/ath/ath_hal ah_internal.h
sys/dev/ath/ath_hal/ar9002 ar9280_attach.c
Log:
SVN rev 221667 on 2011-05-08 15:55:52Z by adrian
Fix the 5ghz fast clock logic.
The macro which I incorrectly copied into ah_internal.h assumed
that it'd be called with an AR_SREV_MERLIN_20() check to ensure
it was only enabled for Merlin (AR9280) silicon revision 2.0 or
later.
Trouble is, the 5GHz fast clock EEPROM flag is only valid for
EEPROM revision 16 or greater; it's assumed to be enabled
by default for Merlin rev >= 2.0. This meant it'd be incorrectly
set for AR5416 and AR9160 in 5GHz mode.
This would have affected non-default clock timings such as SIFS,
ACK and slot time. The incorrect slot time was very likely wrong
for 5ghz mode.
Revision Changes Path
1.35 +15 -3 src/sys/dev/ath/ath_hal/ah_internal.h
1.19 +7 -1 src/sys/dev/ath/ath_hal/ar9002/ar9280_attach.c
help
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201105081556.p48Fu7OK024675>