diff options
author | Johannes Berg <johannes.berg@intel.com> | 2012-06-29 08:24:12 -0400 |
---|---|---|
committer | Johannes Berg <johannes.berg@intel.com> | 2012-07-03 08:03:14 -0400 |
commit | bd408b33363035275403cfaadf2683d8c8e22dd7 (patch) | |
tree | 6ef8313d8d954231812be69fda1f3e5844e96363 | |
parent | 346c265a48169d73a69c88ce9d6b65e6bb3b1f20 (diff) |
iwlwifi: remove unneeded NULL check
There's no need to check trans for non-null
here as it has already been checked in the
caller. This fixes an smatch warning that we
check after having dereferenced it.
Reviewed-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
-rw-r--r-- | drivers/net/wireless/iwlwifi/pcie/rx.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/net/wireless/iwlwifi/pcie/rx.c b/drivers/net/wireless/iwlwifi/pcie/rx.c index be143eb4aa4f..39a6ca1f009c 100644 --- a/drivers/net/wireless/iwlwifi/pcie/rx.c +++ b/drivers/net/wireless/iwlwifi/pcie/rx.c | |||
@@ -879,9 +879,6 @@ static irqreturn_t iwl_isr(int irq, void *data) | |||
879 | 879 | ||
880 | lockdep_assert_held(&trans_pcie->irq_lock); | 880 | lockdep_assert_held(&trans_pcie->irq_lock); |
881 | 881 | ||
882 | if (!trans) | ||
883 | return IRQ_NONE; | ||
884 | |||
885 | trace_iwlwifi_dev_irq(trans->dev); | 882 | trace_iwlwifi_dev_irq(trans->dev); |
886 | 883 | ||
887 | /* Disable (but don't clear!) interrupts here to avoid | 884 | /* Disable (but don't clear!) interrupts here to avoid |