From owner-cvs-src-old@FreeBSD.ORG Sun May 8 15:56:07 2011 Return-Path: Delivered-To: cvs-src-old@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id D6C01106568B for ; Sun, 8 May 2011 15:56:07 +0000 (UTC) (envelope-from adrian@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id C76628FC0A for ; Sun, 8 May 2011 15:56:07 +0000 (UTC) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.4/8.14.4) with ESMTP id p48Fu7Bb024676 for ; Sun, 8 May 2011 15:56:07 GMT (envelope-from adrian@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.4/8.14.4/Submit) id p48Fu7OK024675 for cvs-src-old@freebsd.org; Sun, 8 May 2011 15:56:07 GMT (envelope-from adrian@repoman.freebsd.org) Message-Id: <201105081556.p48Fu7OK024675@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to adrian@repoman.freebsd.org using -f From: Adrian Chadd Date: Sun, 8 May 2011 15:55:52 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: HEAD Subject: cvs commit: src/sys/dev/ath/ath_hal ah_internal.h src/sys/dev/ath/ath_hal/ar9002 ar9280_attach.c X-BeenThere: cvs-src-old@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: **OBSOLETE** CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 08 May 2011 15:56:07 -0000 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