ath9k_hw: rename mrcCCKOff to fix smatch warning
authorRajkumar Manoharan <rmanohar@qca.qualcomm.com>
Thu, 21 Jun 2012 15:03:59 +0000 (20:33 +0530)
committerJohn W. Linville <linville@tuxdriver.com>
Fri, 22 Jun 2012 18:44:05 +0000 (14:44 -0400)
Rename mrcCCKOff for better code readability and also fixes
the smatch warning.

drivers/net/wireless/ath/ath9k/ar9003_phy.c:982
ar9003_hw_ani_control() Error invalid range 1 to 0.

Signed-off-by: Rajkumar Manoharan <rmanohar@qca.qualcomm.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ath/ath9k/ani.c
drivers/net/wireless/ath/ath9k/ani.h
drivers/net/wireless/ath/ath9k/ar5008_phy.c
drivers/net/wireless/ath/ath9k/ar9003_phy.c

index a90aa0b4c8f1a9f718c1c44eda1eff73f770d5c8..7ebc3465f22d636e300e33adfddc0703aa99528e 100644 (file)
@@ -238,7 +238,7 @@ static void ath9k_hw_set_cck_nil(struct ath_hw *ah, u_int8_t immunityLevel)
        if (!AR_SREV_9300_20_OR_LATER(ah) || AR_SREV_9485(ah))
                return;
 
-       if (aniState->mrcCCKOff == entry_cck->mrc_cck_on)
+       if (aniState->mrcCCK != entry_cck->mrc_cck_on)
                ath9k_hw_ani_control(ah,
                                     ATH9K_ANI_MRC_CCK,
                                     entry_cck->mrc_cck_on);
@@ -525,11 +525,7 @@ void ath9k_hw_ani_init(struct ath_hw *ah)
 
                ani->firstepLevel = ATH9K_ANI_FIRSTEP_LVL;
 
-               if (AR_SREV_9300_20_OR_LATER(ah))
-                       ani->mrcCCKOff =
-                               !ATH9K_ANI_ENABLE_MRC_CCK;
-               else
-                       ani->mrcCCKOff = true;
+               ani->mrcCCK = AR_SREV_9300_20_OR_LATER(ah) ? true : false;
 
                ani->ofdmsTurn = true;
 
index 79c85fc6c32ce0678804a77bf8f5c58758eee809..e9d841bbe86f807fbf4941e3042443359a4dc211 100644 (file)
@@ -62,8 +62,6 @@
 #define ATH9K_SIG_SPUR_IMM_SETTING_MIN    0
 #define ATH9K_SIG_SPUR_IMM_SETTING_MAX    22
 
-#define ATH9K_ANI_ENABLE_MRC_CCK          true
-
 /* values here are relative to the INI */
 
 enum ath9k_ani_cmd {
@@ -111,7 +109,7 @@ struct ar5416AniState {
        u8 ofdmNoiseImmunityLevel;
        u8 cckNoiseImmunityLevel;
        bool ofdmsTurn;
-       u8 mrcCCKOff;
+       u8 mrcCCK;
        u8 spurImmunityLevel;
        u8 firstepLevel;
        u8 ofdmWeakSigDetect;
index b6efcd9b31297ec37602bc1285b10574f8b256a0..874186bfda41d1fd93a3838725ab3e95d3210369 100644 (file)
@@ -1234,7 +1234,7 @@ static bool ar5008_hw_ani_control_new(struct ath_hw *ah,
                aniState->spurImmunityLevel,
                aniState->ofdmWeakSigDetect ? "on" : "off",
                aniState->firstepLevel,
-               !aniState->mrcCCKOff ? "on" : "off",
+               aniState->mrcCCK ? "on" : "off",
                aniState->listenTime,
                aniState->ofdmPhyErrCount,
                aniState->cckPhyErrCount);
@@ -1322,7 +1322,7 @@ static void ar5008_hw_ani_cache_ini_regs(struct ath_hw *ah)
        aniState->spurImmunityLevel = ATH9K_ANI_SPUR_IMMUNE_LVL;
        aniState->firstepLevel = ATH9K_ANI_FIRSTEP_LVL;
        aniState->ofdmWeakSigDetect = ATH9K_ANI_USE_OFDM_WEAK_SIG;
-       aniState->mrcCCKOff = true; /* not available on pre AR9003 */
+       aniState->mrcCCK = false; /* not available on pre AR9003 */
 }
 
 static void ar5008_hw_set_nf_limits(struct ath_hw *ah)
index ec8a8d5c6db5f61aa5b58252dafa4a29aa244810..f1975b9e5393a6db7bac0ec65d77fb9947d10830 100644 (file)
@@ -979,16 +979,16 @@ static bool ar9003_hw_ani_control(struct ath_hw *ah,
                              AR_PHY_MRC_CCK_ENABLE, is_on);
                REG_RMW_FIELD(ah, AR_PHY_MRC_CCK_CTRL,
                              AR_PHY_MRC_CCK_MUX_REG, is_on);
-               if (!is_on != aniState->mrcCCKOff) {
+               if (is_on != aniState->mrcCCK) {
                        ath_dbg(common, ANI, "** ch %d: MRC CCK: %s=>%s\n",
                                chan->channel,
-                               !aniState->mrcCCKOff ? "on" : "off",
+                               aniState->mrcCCK ? "on" : "off",
                                is_on ? "on" : "off");
                if (is_on)
                        ah->stats.ast_ani_ccklow++;
                else
                        ah->stats.ast_ani_cckhigh++;
-               aniState->mrcCCKOff = !is_on;
+               aniState->mrcCCK = is_on;
                }
        break;
        }
@@ -1004,7 +1004,7 @@ static bool ar9003_hw_ani_control(struct ath_hw *ah,
                aniState->spurImmunityLevel,
                aniState->ofdmWeakSigDetect ? "on" : "off",
                aniState->firstepLevel,
-               !aniState->mrcCCKOff ? "on" : "off",
+               aniState->mrcCCK ? "on" : "off",
                aniState->listenTime,
                aniState->ofdmPhyErrCount,
                aniState->cckPhyErrCount);
@@ -1114,7 +1114,7 @@ static void ar9003_hw_ani_cache_ini_regs(struct ath_hw *ah)
        aniState->spurImmunityLevel = ATH9K_ANI_SPUR_IMMUNE_LVL;
        aniState->firstepLevel = ATH9K_ANI_FIRSTEP_LVL;
        aniState->ofdmWeakSigDetect = ATH9K_ANI_USE_OFDM_WEAK_SIG;
-       aniState->mrcCCKOff = !ATH9K_ANI_ENABLE_MRC_CCK;
+       aniState->mrcCCK = true;
 }
 
 static void ar9003_hw_set_radar_params(struct ath_hw *ah,