iwlwifi: first enable RF kill interrupt, then check register
authorEmmanuel Grumbach <emmanuel.grumbach@intel.com>
Wed, 28 Mar 2012 07:57:46 +0000 (09:57 +0200)
committerWey-Yi Guy <wey-yi.w.guy@intel.com>
Mon, 23 Apr 2012 21:19:06 +0000 (14:19 -0700)
This can solve a race (very unlikely to happen though).

Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@intel.com>
drivers/net/wireless/iwlwifi/iwl-trans-pcie.c

index 1d9724d..78aa320 100644 (file)
@@ -980,14 +980,13 @@ static int iwl_trans_pcie_start_fw(struct iwl_trans *trans,
                return -EIO;
        }
 
+       iwl_enable_rfkill_int(trans);
+
        /* If platform's RF_KILL switch is NOT set to KILL */
        hw_rfkill = iwl_is_rfkill_set(trans);
        iwl_op_mode_hw_rf_kill(trans->op_mode, hw_rfkill);
-
-       if (hw_rfkill) {
-               iwl_enable_rfkill_int(trans);
+       if (hw_rfkill)
                return -ERFKILL;
-       }
 
        iwl_write32(trans, CSR_INT, 0xFFFFFFFF);
 
@@ -1560,15 +1559,14 @@ static int iwl_trans_pcie_resume(struct iwl_trans *trans)
 {
        bool hw_rfkill;
 
+       iwl_enable_rfkill_int(trans);
+
        hw_rfkill = iwl_is_rfkill_set(trans);
+       iwl_op_mode_hw_rf_kill(trans->op_mode, hw_rfkill);
 
-       if (hw_rfkill)
-               iwl_enable_rfkill_int(trans);
-       else
+       if (!hw_rfkill)
                iwl_enable_interrupts(trans);
 
-       iwl_op_mode_hw_rf_kill(trans->op_mode, hw_rfkill);
-
        return 0;
 }
 #endif /* CONFIG_PM_SLEEP */