diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2015-11-04 16:27:16 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2015-11-04 22:06:13 -0500 |
| commit | 8c169c28f40bd78b772c4d95306735c00f8db80d (patch) | |
| tree | b3243c1681d654bb3d6694530459b0992051c01b /drivers/net | |
| parent | 7362945aea72fccc9df4f4e7643c971e0a8c33dc (diff) | |
| download | linux-8c169c28f40bd78b772c4d95306735c00f8db80d.tar.bz2 | |
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>
Diffstat (limited to 'drivers/net')
| -rw-r--r-- | drivers/net/ethernet/qlogic/qed/qed_int.c | 5 |
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; |