wl12xx: BA receiver support
authorLevi, Shahar <shahar_levi@ti.com>
Sun, 23 Jan 2011 06:27:23 +0000 (07:27 +0100)
committerLuciano Coelho <coelho@ti.com>
Mon, 24 Jan 2011 20:58:47 +0000 (22:58 +0200)
Add new ampdu_action ops to support receiver BA.
The BA initiator session management in FW independently.

Signed-off-by: Shahar Levi <shahar_levi@ti.com>
Reviewed-by: Luciano Coelho <coelho@ti.com>
Signed-off-by: Luciano Coelho <coelho@ti.com>
drivers/net/wireless/wl12xx/acx.c
drivers/net/wireless/wl12xx/acx.h
drivers/net/wireless/wl12xx/init.c
drivers/net/wireless/wl12xx/main.c
drivers/net/wireless/wl12xx/wl12xx.h

index 0b9cacc749035d200fd67b6f9ec24fc65e9e0e8f..afdc601aa7ea7ecb79263231a56425d6e15834e4 100644 (file)
@@ -1393,6 +1393,40 @@ out:
        return ret;
 }
 
+/* setup BA session receiver setting in the FW. */
+int wl1271_acx_set_ba_receiver_session(struct wl1271 *wl, u8 tid_index, u16 ssn,
+                                       bool enable)
+{
+       struct wl1271_acx_ba_receiver_setup *acx;
+       int ret;
+
+       wl1271_debug(DEBUG_ACX, "acx ba receiver session setting");
+
+       acx = kzalloc(sizeof(*acx), GFP_KERNEL);
+       if (!acx) {
+               ret = -ENOMEM;
+               goto out;
+       }
+
+       /* Single link for now */
+       acx->link_id = 1;
+       acx->tid = tid_index;
+       acx->enable = enable;
+       acx->win_size = 0;
+       acx->ssn = ssn;
+
+       ret = wl1271_cmd_configure(wl, ACX_BA_SESSION_RX_SETUP, acx,
+                                  sizeof(*acx));
+       if (ret < 0) {
+               wl1271_warning("acx ba receiver session failed: %d", ret);
+               goto out;
+       }
+
+out:
+       kfree(acx);
+       return ret;
+}
+
 int wl1271_acx_tsf_info(struct wl1271 *wl, u64 *mactime)
 {
        struct wl1271_acx_fw_tsf_information *tsf_info;
index f643e60a566b39e07639d0ccd76cad86e2bdff12..4bbaf04f434ee1564ed101b9656c1506591e1017 100644 (file)
@@ -1095,6 +1095,25 @@ struct wl1271_acx_ba_session_policy {
        u8 padding[3];
 } __packed;
 
+struct wl1271_acx_ba_receiver_setup {
+       struct acx_header header;
+
+       /* Specifies Link Id, Range 0-31, 0xFF means ANY  Link Id */
+       u8 link_id;
+
+       u8 tid;
+
+       u8 enable;
+
+       u8 padding[1];
+
+       /* Windows size in number of packets */
+       u16 win_size;
+
+       /* BA session starting sequence number.  RANGE 0-FFF */
+       u16 ssn;
+} __packed;
+
 struct wl1271_acx_fw_tsf_information {
        struct acx_header header;
 
@@ -1244,8 +1263,10 @@ int wl1271_acx_set_ht_capabilities(struct wl1271 *wl,
 int wl1271_acx_set_ht_information(struct wl1271 *wl,
                                   u16 ht_operation_mode);
 int wl1271_acx_set_ba_session(struct wl1271 *wl,
-                              enum ieee80211_back_parties direction,
-                              u8 tid_index, u8 policy);
+                             enum ieee80211_back_parties direction,
+                             u8 tid_index, u8 policy);
+int wl1271_acx_set_ba_receiver_session(struct wl1271 *wl, u8 tid_index, u16 ssn,
+                                      bool enable);
 int wl1271_acx_tsf_info(struct wl1271 *wl, u64 *mactime);
 int wl1271_acx_max_tx_retry(struct wl1271 *wl);
 
index 2348eadc0de1f3eb6d1cd030789b91eea86fda17..70b3dc88a2195655e3a7d35dd90c078a28ff6438 100644 (file)
@@ -473,7 +473,6 @@ static int wl1271_set_ba_policies(struct wl1271 *wl)
        u8 ret = 0;
 
        /* Reset the BA RX indicators */
-       wl->ba_allowed = true;
        wl->ba_rx_bitmap = 0;
 
        /* validate that FW support BA */
index 01ca666b6c2d6012fd49026a4769b3e1ed62dd44..eda2f3d8edf19f7ed6eff6dda77ee0fe1d4791b4 100644 (file)
@@ -2724,6 +2724,65 @@ out:
        return ret;
 }
 
+int wl1271_op_ampdu_action(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
+                           enum ieee80211_ampdu_mlme_action action,
+                           struct ieee80211_sta *sta, u16 tid, u16 *ssn)
+{
+       struct wl1271 *wl = hw->priv;
+       int ret;
+
+       mutex_lock(&wl->mutex);
+
+       if (unlikely(wl->state == WL1271_STATE_OFF)) {
+               ret = -EAGAIN;
+               goto out;
+       }
+
+       ret = wl1271_ps_elp_wakeup(wl, false);
+       if (ret < 0)
+               goto out;
+
+       switch (action) {
+       case IEEE80211_AMPDU_RX_START:
+               if (wl->ba_support) {
+                       ret = wl1271_acx_set_ba_receiver_session(wl, tid, *ssn,
+                                                                true);
+                       if (!ret)
+                               wl->ba_rx_bitmap |= BIT(tid);
+               } else {
+                       ret = -ENOTSUPP;
+               }
+               break;
+
+       case IEEE80211_AMPDU_RX_STOP:
+               ret = wl1271_acx_set_ba_receiver_session(wl, tid, 0, false);
+               if (!ret)
+                       wl->ba_rx_bitmap &= ~BIT(tid);
+               break;
+
+       /*
+        * The BA initiator session management in FW independently.
+        * Falling break here on purpose for all TX APDU commands.
+        */
+       case IEEE80211_AMPDU_TX_START:
+       case IEEE80211_AMPDU_TX_STOP:
+       case IEEE80211_AMPDU_TX_OPERATIONAL:
+               ret = -EINVAL;
+               break;
+
+       default:
+               wl1271_error("Incorrect ampdu action id=%x\n", action);
+               ret = -EINVAL;
+       }
+
+       wl1271_ps_elp_sleep(wl);
+
+out:
+       mutex_unlock(&wl->mutex);
+
+       return ret;
+}
+
 /* can't be const, mac80211 writes to this */
 static struct ieee80211_rate wl1271_rates[] = {
        { .bitrate = 10,
@@ -2973,6 +3032,7 @@ static const struct ieee80211_ops wl1271_ops = {
        .get_survey = wl1271_op_get_survey,
        .sta_add = wl1271_op_sta_add,
        .sta_remove = wl1271_op_sta_remove,
+       .ampdu_action = wl1271_op_ampdu_action,
        CFG80211_TESTMODE_CMD(wl1271_tm_cmd)
 };
 
index e0bac79cd51691316111c69764160112eadead94..d1de13fe7d9a0dafb9afd23b4b0ad5e4be01ddec 100644 (file)
@@ -473,7 +473,6 @@ struct wl1271 {
 
        /* RX BA constraint value */
        bool ba_support;
-       u8 ba_allowed;
        u8 ba_rx_bitmap;
 };