Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 29 May 2010 16:14:02 +0000 (UTC)
From:      Rui Paulo <rpaulo@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r208644 - head/sys/dev/ath/ath_hal/ar5211
Message-ID:  <201005291614.o4TGE2cM064435@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: rpaulo
Date: Sat May 29 16:14:02 2010
New Revision: 208644
URL: http://svn.freebsd.org/changeset/base/208644

Log:
  Due to the way HALDEBUG() is defined, we need to add curly brackets
  when using it as a sole if clause instruction.
  While there, fix 'const static' typo.
  
  Submitted by:	Arnaud Lacombe <alc@FreeBSD.org>
  MFC after:	1 week

Modified:
  head/sys/dev/ath/ath_hal/ar5211/ar5211_reset.c

Modified: head/sys/dev/ath/ath_hal/ar5211/ar5211_reset.c
==============================================================================
--- head/sys/dev/ath/ath_hal/ar5211/ar5211_reset.c	Sat May 29 16:11:51 2010	(r208643)
+++ head/sys/dev/ath/ath_hal/ar5211/ar5211_reset.c	Sat May 29 16:14:02 2010	(r208644)
@@ -46,7 +46,7 @@ typedef struct {
 } CHAN_INFO_2GHZ;
 
 #define CI_2GHZ_INDEX_CORRECTION 19
-const static CHAN_INFO_2GHZ chan2GHzData[] = {
+static const CHAN_INFO_2GHZ chan2GHzData[] = {
 	{ 1, 0x46, 96  },	/* 2312 -19 */
 	{ 1, 0x46, 97  },	/* 2317 -18 */
 	{ 1, 0x46, 98  },	/* 2322 -17 */
@@ -926,9 +926,10 @@ ar5211IsNfGood(struct ath_hal *ah, struc
 
 	if (!getNoiseFloorThresh(ah, chan, &nfThresh))
 		return AH_FALSE;
-	if (OS_REG_READ(ah, AR_PHY_AGC_CONTROL) & AR_PHY_AGC_CONTROL_NF)
+	if (OS_REG_READ(ah, AR_PHY_AGC_CONTROL) & AR_PHY_AGC_CONTROL_NF) {
 		HALDEBUG(ah, HAL_DEBUG_ANY,
 		    "%s: NF did not complete in calibration window\n", __func__);
+	}
 	nf = ar5211GetNoiseFloor(ah);
 	if (nf > nfThresh) {
 		HALDEBUG(ah, HAL_DEBUG_ANY,



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201005291614.o4TGE2cM064435>