ath9k: Fix MCI scheme initialization
authorSujith Manoharan <c_manoha@qca.qualcomm.com>
Sun, 1 Mar 2015 06:23:46 +0000 (11:53 +0530)
committerKalle Valo <kvalo@codeaurora.org>
Tue, 3 Mar 2015 13:46:54 +0000 (15:46 +0200)
Commit "ath9k_hw: remove ATH_BTCOEX_CFG_MCI" removed
MCI as a separate coex scheme, but we need it to
avoid fiddling with GPIO registers during
ath9k_init_btcoex() that are meant only for 3-wire
cards.

Cc: Rajkumar Manoharan <rmanohar@qti.qualcomm.com>
Signed-off-by: Sujith Manoharan <c_manoha@qca.qualcomm.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/ath/ath9k/btcoex.c
drivers/net/wireless/ath/ath9k/btcoex.h
drivers/net/wireless/ath/ath9k/gpio.c

index 3dfc2c7..78e892e 100644 (file)
@@ -103,7 +103,9 @@ void ath9k_hw_btcoex_init_scheme(struct ath_hw *ah)
                return;
        }
 
-       if (AR_SREV_9300_20_OR_LATER(ah)) {
+       if (ah->caps.hw_caps & ATH9K_HW_CAP_MCI) {
+               btcoex_hw->scheme = ATH_BTCOEX_CFG_MCI;
+       } else if (AR_SREV_9300_20_OR_LATER(ah)) {
                btcoex_hw->scheme = ATH_BTCOEX_CFG_3WIRE;
                btcoex_hw->btactive_gpio = ATH_BTACTIVE_GPIO_9300;
                btcoex_hw->wlanactive_gpio = ATH_WLANACTIVE_GPIO_9300;
@@ -307,6 +309,18 @@ static void ath9k_hw_btcoex_enable_mci(struct ath_hw *ah)
        btcoex->enabled = true;
 }
 
+static void ath9k_hw_btcoex_disable_mci(struct ath_hw *ah)
+{
+       struct ath_btcoex_hw *btcoex_hw = &ah->btcoex_hw;
+       int i;
+
+       ath9k_hw_btcoex_bt_stomp(ah, ATH_BTCOEX_STOMP_NONE);
+
+       for (i = 0; i < AR9300_NUM_BT_WEIGHTS; i++)
+               REG_WRITE(ah, AR_MCI_COEX_WL_WEIGHTS(i),
+                         btcoex_hw->wlan_weight[i]);
+}
+
 void ath9k_hw_btcoex_enable(struct ath_hw *ah)
 {
        struct ath_btcoex_hw *btcoex_hw = &ah->btcoex_hw;
@@ -318,17 +332,18 @@ void ath9k_hw_btcoex_enable(struct ath_hw *ah)
                ath9k_hw_btcoex_enable_2wire(ah);
                break;
        case ATH_BTCOEX_CFG_3WIRE:
-               if (AR_SREV_9462(ah) || AR_SREV_9565(ah)) {
-                       ath9k_hw_btcoex_enable_mci(ah);
-                       return;
-               }
                ath9k_hw_btcoex_enable_3wire(ah);
                break;
+       case ATH_BTCOEX_CFG_MCI:
+               ath9k_hw_btcoex_enable_mci(ah);
+               break;
        }
 
-       REG_RMW(ah, AR_GPIO_PDPU,
-               (0x2 << (btcoex_hw->btactive_gpio * 2)),
-               (0x3 << (btcoex_hw->btactive_gpio * 2)));
+       if (ath9k_hw_get_btcoex_scheme(ah) != ATH_BTCOEX_CFG_MCI) {
+               REG_RMW(ah, AR_GPIO_PDPU,
+                       (0x2 << (btcoex_hw->btactive_gpio * 2)),
+                       (0x3 << (btcoex_hw->btactive_gpio * 2)));
+       }
 
        ah->btcoex_hw.enabled = true;
 }
@@ -340,13 +355,12 @@ void ath9k_hw_btcoex_disable(struct ath_hw *ah)
        int i;
 
        btcoex_hw->enabled = false;
-       if (AR_SREV_9462(ah) || AR_SREV_9565(ah)) {
-               ath9k_hw_btcoex_bt_stomp(ah, ATH_BTCOEX_STOMP_NONE);
-               for (i = 0; i < AR9300_NUM_BT_WEIGHTS; i++)
-                       REG_WRITE(ah, AR_MCI_COEX_WL_WEIGHTS(i),
-                                 btcoex_hw->wlan_weight[i]);
+
+       if (ath9k_hw_get_btcoex_scheme(ah) == ATH_BTCOEX_CFG_MCI) {
+               ath9k_hw_btcoex_disable_mci(ah);
                return;
        }
+
        ath9k_hw_set_gpio(ah, btcoex_hw->wlanactive_gpio, 0);
 
        ath9k_hw_cfg_output(ah, btcoex_hw->wlanactive_gpio,
index 6de26ea..5fe62ff 100644 (file)
@@ -58,6 +58,7 @@ enum ath_btcoex_scheme {
        ATH_BTCOEX_CFG_NONE,
        ATH_BTCOEX_CFG_2WIRE,
        ATH_BTCOEX_CFG_3WIRE,
+       ATH_BTCOEX_CFG_MCI,
 };
 
 struct ath9k_hw_mci {
index 257ffce..b4a0612 100644 (file)
@@ -280,6 +280,7 @@ static void ath_init_btcoex_timer(struct ath_softc *sc)
                btcoex->btcoex_period / 100;
        btcoex->btscan_no_stomp = (100 - ATH_BTCOEX_BTSCAN_DUTY_CYCLE) *
                                   btcoex->btcoex_period / 100;
+       btcoex->bt_stomp_type = ATH_BTCOEX_STOMP_LOW;
 
        setup_timer(&btcoex->period_timer, ath_btcoex_period_timer,
                        (unsigned long) sc);
@@ -408,19 +409,19 @@ int ath9k_init_btcoex(struct ath_softc *sc)
        case ATH_BTCOEX_CFG_3WIRE:
                ath9k_hw_btcoex_init_3wire(sc->sc_ah);
                ath_init_btcoex_timer(sc);
-
                txq = sc->tx.txq_map[IEEE80211_AC_BE];
                ath9k_hw_init_btcoex_hw(sc->sc_ah, txq->axq_qnum);
-               sc->btcoex.bt_stomp_type = ATH_BTCOEX_STOMP_LOW;
-               if (ath9k_hw_mci_is_enabled(ah)) {
-                       sc->btcoex.duty_cycle = ATH_BTCOEX_DEF_DUTY_CYCLE;
-                       INIT_LIST_HEAD(&sc->btcoex.mci.info);
-                       ath9k_hw_btcoex_init_mci(ah);
-
-                       r = ath_mci_setup(sc);
-                       if (r)
-                               return r;
-               }
+               break;
+       case ATH_BTCOEX_CFG_MCI:
+               ath_init_btcoex_timer(sc);
+
+               sc->btcoex.duty_cycle = ATH_BTCOEX_DEF_DUTY_CYCLE;
+               INIT_LIST_HEAD(&sc->btcoex.mci.info);
+               ath9k_hw_btcoex_init_mci(ah);
+
+               r = ath_mci_setup(sc);
+               if (r)
+                       return r;
 
                break;
        default: