wl12xx: wakeup chip from ELP during scan
authorArik Nemtsov <arik@wizery.com>
Tue, 1 Mar 2011 10:27:26 +0000 (12:27 +0200)
committerLuciano Coelho <coelho@ti.com>
Thu, 3 Mar 2011 14:20:25 +0000 (16:20 +0200)
Commands are sometimes sent to FW on scan completion. Make sure the chip
is awake to receive them. Sending commands while the chip is in ELP
can cause SDIO read errors and/or crash the FW.

Signed-off-by: Arik Nemtsov <arik@wizery.com>
Signed-off-by: Ido Yariv <ido@wizery.com>
Signed-off-by: Luciano Coelho <coelho@ti.com>
drivers/net/wireless/wl12xx/cmd.c
drivers/net/wireless/wl12xx/scan.c

index 97ffd7aa57a81e54509d5fcffab7a589ccbe2462..f0aa7ab97bf7ddcd52e9451befa96c8f1025981c 100644 (file)
@@ -63,6 +63,7 @@ int wl1271_cmd_send(struct wl1271 *wl, u16 id, void *buf, size_t len,
        cmd->status = 0;
 
        WARN_ON(len % 4 != 0);
+       WARN_ON(test_bit(WL1271_FLAG_IN_ELP, &wl->flags));
 
        wl1271_write(wl, wl->cmd_box_addr, buf, len, false);
 
index 6f897b9d90caf21784342d700bba64a98b40dc09..420653a2859ce68c40ba8be0b93b80d624f48598 100644 (file)
@@ -27,6 +27,7 @@
 #include "cmd.h"
 #include "scan.h"
 #include "acx.h"
+#include "ps.h"
 
 void wl1271_scan_complete_work(struct work_struct *work)
 {
@@ -40,10 +41,11 @@ void wl1271_scan_complete_work(struct work_struct *work)
 
        mutex_lock(&wl->mutex);
 
-       if (wl->scan.state == WL1271_SCAN_STATE_IDLE) {
-               mutex_unlock(&wl->mutex);
-               return;
-       }
+       if (wl->state == WL1271_STATE_OFF)
+               goto out;
+
+       if (wl->scan.state == WL1271_SCAN_STATE_IDLE)
+               goto out;
 
        wl->scan.state = WL1271_SCAN_STATE_IDLE;
        kfree(wl->scan.scanned_ch);
@@ -52,13 +54,19 @@ void wl1271_scan_complete_work(struct work_struct *work)
        ieee80211_scan_completed(wl->hw, false);
 
        /* restore hardware connection monitoring template */
-       if (test_bit(WL1271_FLAG_STA_ASSOCIATED, &wl->flags))
-               wl1271_cmd_build_ap_probe_req(wl, wl->probereq);
+       if (test_bit(WL1271_FLAG_STA_ASSOCIATED, &wl->flags)) {
+               if (wl1271_ps_elp_wakeup(wl) == 0) {
+                       wl1271_cmd_build_ap_probe_req(wl, wl->probereq);
+                       wl1271_ps_elp_sleep(wl);
+               }
+       }
 
        if (wl->scan.failed) {
                wl1271_info("Scan completed due to error.");
                ieee80211_queue_work(wl->hw, &wl->recovery_work);
        }
+
+out:
        mutex_unlock(&wl->mutex);
 
 }