iwlwifi: pcie: Re-configure IVAR table after stop device

When getting RF_KILL and disabling radio, the device gets stopped
and reset. This erases the IVAR table that matches the interrupt
to its cause, and is essential for MSIX proper functionality.
Till now, the table wasn't re-configured after the reset, and
therefore the interrupt that enabled radio didn't fire on the
right irq, and the driver didn't handle it correctly.

To fix this, configure the IVAR table again after resetting the
device.

Signed-off-by: Golan Ben-Ami <golan.ben.ami@intel.com>
Signed-off-by: Luca Coelho <luciano.coelho@intel.com>

diff --git a/drivers/net/wireless/intel/iwlwifi/pcie/trans.c b/drivers/net/wireless/intel/iwlwifi/pcie/trans.c
index bb2a9a151957af6e97ff215c02a5fcbe29dc46fd..7f05fc56587add6336fc4b453b2282e6ff8c4730 100644 (file)
--- a/drivers/net/wireless/intel/iwlwifi/pcie/trans.c
+++ b/drivers/net/wireless/intel/iwlwifi/pcie/trans.c
@@ -1245,6 +1245,15 @@ static void _iwl_trans_pcie_stop_device(struct iwl_trans *trans, bool low_power)
        iwl_write32(trans, CSR_RESET, CSR_RESET_REG_FLAG_SW_RESET);
        usleep_range(1000, 2000);
 
+       /*
+        * Upon stop, the IVAR table gets erased, so msi-x won't
+        * work. This causes a bug in RF-KILL flows, since the interrupt
+        * that enables radio won't fire on the correct irq, and the
+        * driver won't be able to handle the interrupt.
+        * Configure the IVAR table again after reset.
+        */
+       iwl_pcie_conf_msix_hw(trans_pcie);
+
        /*
         * Upon stop, the APM issues an interrupt if HW RF kill is set.
         * This is a bug in certain verions of the hardware.