Bug 801935 - Don't unload wireless drivers, r=gal
authorMichael Wu <mwu@mozilla.com>
Mon, 15 Oct 2012 19:51:20 -0400
changeset 110350 74816ae5dd43249ee708b7f216b8c79b0a2f8c8d
parent 110349 952dd10342381d8de0b5cf35f6dc0a0a0741be22
child 110351 a8a4f26cbb7855ed2b4707452a36131af5d4a890
push id16497
push usermwu@mozilla.com
push dateMon, 15 Oct 2012 23:52:14 +0000
treeherdermozilla-inbound@74816ae5dd43 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersgal
bugs801935
milestone19.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 801935 - Don't unload wireless drivers, r=gal
dom/wifi/WifiWorker.js
--- a/dom/wifi/WifiWorker.js
+++ b/dom/wifi/WifiWorker.js
@@ -123,21 +123,22 @@ var WifiManager = (function() {
 
     voidControlMessage("load_driver", function(status) {
       driverLoaded = (status >= 0);
       callback(status)
     });
   }
 
   function unloadDriver(callback) {
-    // Otoro ICS can't unload and then load the driver, so never unload it.
-    if (device === "otoro") {
-      callback(0);
-      return;
-    }
+    // Unloading drivers is generally unnecessary and
+    // can trigger bugs in some drivers.
+    // On properly written drivers, bringing the interface
+    // down powers down the interface.
+    callback(0);
+    return;
 
     voidControlMessage("unload_driver", function(status) {
       driverLoaded = (status < 0);
       callback(status);
     });
   }
 
   function startSupplicant(callback) {