summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2015-11-04 16:27:16 +0300
committerDavid S. Miller <davem@davemloft.net>2015-11-04 22:06:13 -0500
commit8c169c28f40bd78b772c4d95306735c00f8db80d (patch)
treeb3243c1681d654bb3d6694530459b0992051c01b
parent7362945aea72fccc9df4f4e7643c971e0a8c33dc (diff)
downloadlinux-stable-8c169c28f40bd78b772c4d95306735c00f8db80d.tar.gz
linux-stable-8c169c28f40bd78b772c4d95306735c00f8db80d.tar.bz2
linux-stable-8c169c28f40bd78b772c4d95306735c00f8db80d.zip
qlogic/qed: remove bogus NULL check
We check if "p_hwfn" is NULL and then dereference it in the error handling code. I read the code and it isn't NULL so let's remove the check. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Acked-by: Yuval Mintz <Yuval.Mintz@qlogic.com> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--drivers/net/ethernet/qlogic/qed/qed_int.c5
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/net/ethernet/qlogic/qed/qed_int.c b/drivers/net/ethernet/qlogic/qed/qed_int.c
index 2e399b6137a2..de50e84902af 100644
--- a/drivers/net/ethernet/qlogic/qed/qed_int.c
+++ b/drivers/net/ethernet/qlogic/qed/qed_int.c
@@ -251,11 +251,6 @@ void qed_int_sp_dpc(unsigned long hwfn_cookie)
int arr_size;
u16 rc = 0;
- if (!p_hwfn) {
- DP_ERR(p_hwfn->cdev, "DPC called - no hwfn!\n");
- return;
- }
-
if (!p_hwfn->p_sp_sb) {
DP_ERR(p_hwfn->cdev, "DPC called - no p_sp_sb\n");
return;