After enabling/disabling interrupts flushing is required Signed-off-by: Joonwoo Park <joonwpark81@xxxxxxxxx> --- drivers/net/wireless/iwlwifi/iwl-io.h | 2 ++ drivers/net/wireless/iwlwifi/iwl3945-base.c | 6 ++++++ 2 files changed, 8 insertions(+), 0 deletions(-) diff --git a/drivers/net/wireless/iwlwifi/iwl-io.h b/drivers/net/wireless/iwlwifi/iwl-io.h index 8a8b96f..f2bc30e 100644 --- a/drivers/net/wireless/iwlwifi/iwl-io.h +++ b/drivers/net/wireless/iwlwifi/iwl-io.h @@ -87,6 +87,8 @@ static inline u32 __iwl_read32(char *f, u32 l, struct iwl_priv *iwl, u32 ofs) #define iwl_read32(p, o) _iwl_read32(p, o) #endif +#define iwl_flush32(iwl, ofs) iwl_read32(iwl, ofs) + static inline int _iwl_poll_bit(struct iwl_priv *priv, u32 addr, u32 bits, u32 mask, int timeout) { diff --git a/drivers/net/wireless/iwlwifi/iwl3945-base.c b/drivers/net/wireless/iwlwifi/iwl3945-base.c index 8ed898d..85f1112 100644 --- a/drivers/net/wireless/iwlwifi/iwl3945-base.c +++ b/drivers/net/wireless/iwlwifi/iwl3945-base.c @@ -4410,6 +4410,7 @@ static void iwl_enable_interrupts(struct iwl_priv *priv) IWL_DEBUG_ISR("Enabling interrupts\n"); set_bit(STATUS_INT_ENABLED, &priv->status); iwl_write32(priv, CSR_INT_MASK, CSR_INI_SET_MASK); + iwl_flush32(priv, CSR_INT_MASK); } static inline void iwl_disable_interrupts(struct iwl_priv *priv) @@ -4418,11 +4419,15 @@ static inline void iwl_disable_interrupts(struct iwl_priv *priv) /* disable interrupts from uCode/NIC to host */ iwl_write32(priv, CSR_INT_MASK, 0x00000000); + iwl_flush32(priv, CSR_INT_MASK); /* acknowledge/clear/reset any interrupts still pending * from uCode or flow handler (Rx/Tx DMA) */ iwl_write32(priv, CSR_INT, 0xffffffff); + iwl_flush32(priv, CSR_INT); iwl_write32(priv, CSR_FH_INT_STATUS, 0xffffffff); + iwl_flush32(priv, CSR_FH_INT_STATUS); + IWL_DEBUG_ISR("Disabled interrupts\n"); } @@ -4840,6 +4845,7 @@ static irqreturn_t iwl_isr(int irq, void *data) * If we *don't* have something, we'll re-enable before leaving here. */ inta_mask = iwl_read32(priv, CSR_INT_MASK); /* just for debug */ iwl_write32(priv, CSR_INT_MASK, 0x00000000); + iwl_flush32(priv, CSR_INT_MASK); /* Discover which interrupts are active/pending */ inta = iwl_read32(priv, CSR_INT); -- 1.5.3.rc5 - To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html