wil6210: fix return code of wmi_mgmt_tx and wmi_mgmt_tx_ext

The functions that send management TX frame have 3 possible
results: success and other side acknowledged receive (ACK=1),
success and other side did not acknowledge receive(ACK=0) and
failure to send the frame. The current implementation
incorrectly reports the ACK=0 case as failure.

Signed-off-by: Lior David <liord@codeaurora.org>
Signed-off-by: Maya Erez <merez@codeaurora.org>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>

diff --git a/drivers/net/wireless/ath/wil6210/cfg80211.c b/drivers/net/wireless/ath/wil6210/cfg80211.c
index e8d65ddb1b0a89c005656bb8847686c6c25e561a..218296e319b9a969932ff00edcae575afdfa3ed4 100644 (file)
--- a/drivers/net/wireless/ath/wil6210/cfg80211.c
+++ b/drivers/net/wireless/ath/wil6210/cfg80211.c
@@ -1274,7 +1274,12 @@ int wil_cfg80211_mgmt_tx(struct wiphy *wiphy, struct wireless_dev *wdev,
                             params->wait);
 
 out:
+       /* when the sent packet was not acked by receiver(ACK=0), rc will
+        * be -EAGAIN. In this case this function needs to return success,
+        * the ACK=0 will be reflected in tx_status.
+        */
        tx_status = (rc == 0);
+       rc = (rc == -EAGAIN) ? 0 : rc;
        cfg80211_mgmt_tx_status(wdev, cookie ? *cookie : 0, buf, len,
                                tx_status, GFP_KERNEL);
 

diff --git a/drivers/net/wireless/ath/wil6210/wmi.c b/drivers/net/wireless/ath/wil6210/wmi.c
index c5bcb8da07c1876033a06f531bf28f1c875cf28e..d89cd41e78aca635b57f6bf769cf3e48088aa0e5 100644 (file)
--- a/drivers/net/wireless/ath/wil6210/wmi.c
+++ b/drivers/net/wireless/ath/wil6210/wmi.c
@@ -3511,8 +3511,9 @@ int wmi_mgmt_tx(struct wil6210_vif *vif, const u8 *buf, size_t len)
        rc = wmi_call(wil, WMI_SW_TX_REQ_CMDID, vif->mid, cmd, total,
                      WMI_SW_TX_COMPLETE_EVENTID, &evt, sizeof(evt), 2000);
        if (!rc && evt.evt.status != WMI_FW_STATUS_SUCCESS) {
-               wil_err(wil, "mgmt_tx failed with status %d\n", evt.evt.status);
-               rc = -EINVAL;
+               wil_dbg_wmi(wil, "mgmt_tx failed with status %d\n",
+                           evt.evt.status);
+               rc = -EAGAIN;
        }
 
        kfree(cmd);
@@ -3564,9 +3565,9 @@ int wmi_mgmt_tx_ext(struct wil6210_vif *vif, const u8 *buf, size_t len,
        rc = wmi_call(wil, WMI_SW_TX_REQ_EXT_CMDID, vif->mid, cmd, total,
                      WMI_SW_TX_COMPLETE_EVENTID, &evt, sizeof(evt), 2000);
        if (!rc && evt.evt.status != WMI_FW_STATUS_SUCCESS) {
-               wil_err(wil, "mgmt_tx_ext failed with status %d\n",
-                       evt.evt.status);
-               rc = -EINVAL;
+               wil_dbg_wmi(wil, "mgmt_tx_ext failed with status %d\n",
+                           evt.evt.status);
+               rc = -EAGAIN;
        }
 
        kfree(cmd);