[PATCH 48/99] cw1200: Refactor workaround for WFD 6.1.10

[Dmitry Tarnyagin]

From: "Ajitpal.Singh" <ajitpal.singh at stericsson.com>

Refactoring code for WFD 6.1.10 workaround.
Now if an ACTION frame is received from a mapped MAC address then
reset the link and re-map it.
If ACTION frame is received from an unmapped MAC address then map
the MAC addreess and reset it

ST-Ericsson ID: 372706
ST-Ericsson FOSS-OUT ID: NA
Signed-off-by: Ajitpal.Singh <ajitpal.singh at stericsson.com>
Change-Id: I8ea4fd6368a6bf5a23273ff25ddca3d0c589ec9a
Reviewed-on: http://gerrit.lud.stericsson.com/gerrit/39432
Reviewed-by: Bartosz MARKOWSKI <bartosz.markowski at tieto.com>
Tested-by: Bartosz MARKOWSKI <bartosz.markowski at tieto.com>
---
 drivers/staging/cw1200/cw1200.h |    1 +
 drivers/staging/cw1200/txrx.c   |   73 +++++++++++++++++++++++++++------------
 2 files changed, 52 insertions(+), 22 deletions(-)

diff --git a/drivers/staging/cw1200/cw1200.h b/drivers/staging/cw1200/cw1200.h
index 0bdd3a6..8ee88b6 100644
--- a/drivers/staging/cw1200/cw1200.h
+++ b/drivers/staging/cw1200/cw1200.h
@@ -251,6 +251,7 @@ struct cw1200_common {
 	/* Workaround for WFD testcase 6.1.10*/
 	struct work_struct	linkid_reset_work;
 	u8			action_frame_sa[ETH_ALEN];
+	u8			action_linkid;
 #endif
 };
 
diff --git a/drivers/staging/cw1200/txrx.c b/drivers/staging/cw1200/txrx.c
index 8b92180..1931e3e 100644
--- a/drivers/staging/cw1200/txrx.c
+++ b/drivers/staging/cw1200/txrx.c
@@ -730,15 +730,9 @@ void cw1200_tx(struct ieee80211_hw *dev, struct sk_buff *skb)
 	if (WARN_ON(t.queue >= 4))
 		goto drop;
 
-	/* Temporary change*/
-	if (unlikely(ieee80211_is_probe_resp(t.hdr->frame_control))) {
-		t.txpriv.raw_link_id =
-				t.txpriv.link_id = 0;
-	} else {
-		ret = cw1200_tx_h_calc_link_ids(priv, &t);
-		if (ret)
-			goto drop;
-	}
+	ret = cw1200_tx_h_calc_link_ids(priv, &t);
+	if (ret)
+		goto drop;
 
 	txrx_printk(KERN_DEBUG "[TX] TX %d bytes "
 			"(queue: %d, link_id: %d (%d)).\n",
@@ -1035,7 +1029,26 @@ void cw1200_rx_cb(struct cw1200_common *priv,
 			&& ieee80211_is_action(frame->frame_control)
 			&& (mgmt->u.action.category == WLAN_CATEGORY_PUBLIC)) {
 		txrx_printk(KERN_DEBUG "[RX] Going to MAP&RESET link ID\n");
-		memcpy(&priv->action_frame_sa[0], ieee80211_get_SA(frame), ETH_ALEN);
+
+		if (work_pending(&priv->linkid_reset_work))
+			WARN_ON(1);
+
+		memcpy(&priv->action_frame_sa[0],
+				ieee80211_get_SA(frame), ETH_ALEN);
+		priv->action_linkid = 0;
+		schedule_work(&priv->linkid_reset_work);
+	}
+
+	if (arg->link_id && (priv->vif->p2p == WSM_START_MODE_P2P_GO)
+			&& ieee80211_is_action(frame->frame_control)
+			&& (mgmt->u.action.category == WLAN_CATEGORY_PUBLIC)) {
+		/* Link ID already exists for the ACTION frame.
+		 * Reset and Remap */
+		if (work_pending(&priv->linkid_reset_work))
+			WARN_ON(1);
+		memcpy(&priv->action_frame_sa[0],
+				ieee80211_get_SA(frame), ETH_ALEN);
+		priv->action_linkid = arg->link_id;
 		schedule_work(&priv->linkid_reset_work);
 	}
 #endif
@@ -1225,21 +1238,37 @@ void cw1200_link_id_reset(struct work_struct *work)
 	struct cw1200_common *priv =
 		container_of(work, struct cw1200_common, linkid_reset_work);
 	int temp_linkid;
-
-	/* In GO mode we can receive ACTION frames without a linkID */
-	temp_linkid = cw1200_alloc_link_id(priv, &priv->action_frame_sa[0]);
-	WARN_ON(!temp_linkid);
-	if (temp_linkid) {
-		/* Make sure we execute the WQ */
-		flush_workqueue(priv->workqueue);
-		/* Release the link ID */
+	struct cw1200_link_entry *entry;
+
+	if (!priv->action_linkid) {
+		/* In GO mode we can receive ACTION frames without a linkID */
+		temp_linkid = cw1200_alloc_link_id(priv,
+				&priv->action_frame_sa[0]);
+		WARN_ON(!temp_linkid);
+		if (temp_linkid) {
+			/* Make sure we execute the WQ */
+			flush_workqueue(priv->workqueue);
+			/* Release the link ID */
+			spin_lock(&priv->ps_state_lock);
+			priv->link_id_db[temp_linkid - 1].status =
+				CW1200_LINK_RESERVE;
+			spin_unlock(&priv->ps_state_lock);
+			wsm_lock_tx_async(priv);
+			if (queue_work(priv->workqueue,
+					&priv->link_id_work) <= 0)
+				wsm_unlock_tx(priv);
+		}
+	} else {
 		spin_lock(&priv->ps_state_lock);
-		priv->link_id_db[temp_linkid - 1].status =
-			CW1200_LINK_RESERVE;
-		spin_unlock(&priv->ps_state_lock);
+		entry = &priv->link_id_db[priv->action_linkid - 1];
 		wsm_lock_tx_async(priv);
 		if (queue_work(priv->workqueue, &priv->link_id_work) <= 0)
-			wsm_unlock_tx(priv);
+				wsm_unlock_tx(priv);
+		spin_unlock(&priv->ps_state_lock);
+		flush_workqueue(priv->workqueue);
+		temp_linkid = cw1200_alloc_link_id(priv,
+				&priv->action_frame_sa[0]);
+		WARN_ON(!temp_linkid);
 	}
 }
 #endif
-- 
1.7.8.3