diff options
author | Stanislaw Gruszka <sgruszka@redhat.com> | 2012-03-07 12:52:24 -0500 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2012-03-08 13:59:50 -0500 |
commit | bfe4b80e9f7385f34986736cdc094be56782109a (patch) | |
tree | 29b1335d3045834bcbb5d23d53a4b212ef9355e0 /drivers/net/wireless/iwlwifi/iwl-agn.c | |
parent | aa5affbacb24cb5d8fd6f7c66e57d62164ed6d34 (diff) |
iwlwifi: always check if got h/w access before write
Before we write to the device registers always check if
iwl_grap_nic_access() was successful.
On the way change return type of grab_nic_access() to bool, and add
likely()/unlikely() statement.
Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/iwlwifi/iwl-agn.c')
-rw-r--r-- | drivers/net/wireless/iwlwifi/iwl-agn.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/iwlwifi/iwl-agn.c b/drivers/net/wireless/iwlwifi/iwl-agn.c index 5f390f8d3b8b..ce9ebeb89e4c 100644 --- a/drivers/net/wireless/iwlwifi/iwl-agn.c +++ b/drivers/net/wireless/iwlwifi/iwl-agn.c | |||
@@ -327,7 +327,7 @@ static void iwl_print_cont_event_trace(struct iwl_priv *priv, u32 base, | |||
327 | 327 | ||
328 | /* Make sure device is powered up for SRAM reads */ | 328 | /* Make sure device is powered up for SRAM reads */ |
329 | spin_lock_irqsave(&trans(priv)->reg_lock, reg_flags); | 329 | spin_lock_irqsave(&trans(priv)->reg_lock, reg_flags); |
330 | if (iwl_grab_nic_access(trans(priv))) { | 330 | if (unlikely(!iwl_grab_nic_access(trans(priv)))) { |
331 | spin_unlock_irqrestore(&trans(priv)->reg_lock, reg_flags); | 331 | spin_unlock_irqrestore(&trans(priv)->reg_lock, reg_flags); |
332 | return; | 332 | return; |
333 | } | 333 | } |