iwlagn: disable gen2b BT coexistence in IBSS
authorJohannes Berg <johannes.berg@intel.com>
Mon, 23 Aug 2010 14:57:00 +0000 (07:57 -0700)
committerJohn W. Linville <linville@tuxdriver.com>
Wed, 25 Aug 2010 18:33:22 +0000 (14:33 -0400)
IBSS doesn't allow for coexistence, so it
should be disabled.

Additionally, disable reacting to the BT
profile notification when in IBSS mode,
it likely won't be sent by the device to
start with though.

Also, in IBSS mode, BT coexistence isn't as fully-featured
and we must use a single antenna only. So instead of
peppering the code with new checks, simply pretend
we are in high BT traffic load, which has the needed
effect of disabling antenna B use.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/iwlwifi/iwl-6000.c
drivers/net/wireless/iwlwifi/iwl-core.c
drivers/net/wireless/iwlwifi/iwl-dev.h

index f049ebc4a6aa54d4239d7b6845f56dc7b5ccbc5b..51419444834f4d4972f167e7222f19ad9e987896 100644 (file)
@@ -217,7 +217,14 @@ static void iwl6000g2b_send_bt_config(struct iwl_priv *priv)
        BUILD_BUG_ON(sizeof(iwl6000g2b_def_3w_lookup) !=
                        sizeof(bt_cmd.bt3_lookup_table));
 
-       if (!bt_coex_active) {
+       /*
+        * Configure BT coex mode to "no coexistence" when the
+        * user disabled BT coexistence, we have no interface
+        * (might be in monitor mode), or the interface is in
+        * IBSS mode (no proper uCode support for coex then).
+        */
+       if (!bt_coex_active || !priv->vif ||
+           priv->iw_mode == NL80211_IFTYPE_ADHOC) {
                bt_cmd.flags = 0;
        } else {
                bt_cmd.flags = IWL6000G2B_BT_FLAG_CHANNEL_INHIBITION |
@@ -426,18 +433,23 @@ static void iwl6000g2b_bt_coex_profile_notif(struct iwl_priv *priv,
                        coex->uart_msg[3], coex->uart_msg[4], coex->uart_msg[5],
                        coex->uart_msg[6], coex->uart_msg[7]);
 
-       if (coex->bt_traffic_load != priv->bt_traffic_load) {
-               priv->bt_traffic_load = coex->bt_traffic_load;
+       priv->notif_bt_traffic_load = coex->bt_traffic_load;
 
-               queue_work(priv->workqueue, &priv->bt_traffic_change_work);
-       }
+       if (priv->iw_mode != NL80211_IFTYPE_ADHOC) {
+               if (coex->bt_traffic_load != priv->bt_traffic_load) {
+                       priv->bt_traffic_load = coex->bt_traffic_load;
 
-       /* FIXME: add defines for this check */
-       priv->bt_sco_active = coex->uart_msg[3] & 1;
-       if (priv->bt_sco_active)
-               sco_cmd.flags |= IWL6000G2B_BT_SCO_ACTIVE;
-       iwl_send_cmd_pdu_async(priv, REPLY_BT_COEX_SCO,
-                              sizeof(sco_cmd), &sco_cmd, NULL);
+                       queue_work(priv->workqueue,
+                                  &priv->bt_traffic_change_work);
+               }
+
+               /* FIXME: add defines for this check */
+               priv->bt_sco_active = coex->uart_msg[3] & 1;
+               if (priv->bt_sco_active)
+                       sco_cmd.flags |= IWL6000G2B_BT_SCO_ACTIVE;
+               iwl_send_cmd_pdu_async(priv, REPLY_BT_COEX_SCO,
+                                      sizeof(sco_cmd), &sco_cmd, NULL);
+       }
 }
 
 void iwl6000g2b_rx_handler_setup(struct iwl_priv *priv)
index 4beddade7423b8976d901d7500bf0243f1997da9..1921307a23b885589abd9f354d444b06efc11725 100644 (file)
@@ -1872,6 +1872,16 @@ int iwl_mac_add_interface(struct ieee80211_hw *hw, struct ieee80211_vif *vif)
        if (err)
                goto out_err;
 
+       if (priv->cfg->advanced_bt_coexist &&
+           vif->type == NL80211_IFTYPE_ADHOC) {
+               /*
+                * pretend to have high BT traffic as long as we
+                * are operating in IBSS mode, as this will cause
+                * the rate scaling etc. to behave as intended.
+                */
+               priv->bt_traffic_load = IWL_BT_COEX_TRAFFIC_LOAD_HIGH;
+       }
+
        goto out;
 
  out_err:
@@ -1909,6 +1919,17 @@ void iwl_mac_remove_interface(struct ieee80211_hw *hw,
                }
                memset(priv->bssid, 0, ETH_ALEN);
        }
+
+       /*
+        * When removing the IBSS interface, overwrite the
+        * BT traffic load with the stored one from the last
+        * notification, if any. If this is a device that
+        * doesn't implement this, this has no effect since
+        * both values are the same and zero.
+        */
+       if (vif->type == NL80211_IFTYPE_ADHOC)
+               priv->bt_traffic_load = priv->notif_bt_traffic_load;
+
        mutex_unlock(&priv->mutex);
 
        if (scan_completed)
index 5d327b44533b27750cfe5457b1f3ddeb4ab96d6a..815ba0af94cea9cfc0fc89ba8e1ef327352000f1 100644 (file)
@@ -1362,7 +1362,7 @@ struct iwl_priv {
 #endif
        };
 
-       u8 bt_traffic_load;
+       u8 bt_traffic_load, notif_bt_traffic_load;
        bool bt_sco_active;
        struct work_struct bt_traffic_change_work;