diff options
author | Roland Dreier <rolandd@cisco.com> | 2010-07-19 16:13:09 -0400 |
---|---|---|
committer | Roland Dreier <rolandd@cisco.com> | 2010-07-19 16:13:09 -0400 |
commit | 85963e4cbcf11c00b1d27ea0e0fcab8cb3d7a69b (patch) | |
tree | 27674e8ad976d61c6a7301186ef33600d44df03e /drivers/infiniband/hw | |
parent | c1d7356c8572f3fe0445336d8e75914bdcadad59 (diff) |
RDMA/cxgb4: Remove unneeded NULL check
The rest of the code seems to assume that ep->com.cm_id can't be NULL,
so remove an unneeded test.
Reported-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
Diffstat (limited to 'drivers/infiniband/hw')
-rw-r--r-- | drivers/infiniband/hw/cxgb4/cm.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/infiniband/hw/cxgb4/cm.c b/drivers/infiniband/hw/cxgb4/cm.c index b5e676c61096..4185c3b0635b 100644 --- a/drivers/infiniband/hw/cxgb4/cm.c +++ b/drivers/infiniband/hw/cxgb4/cm.c | |||
@@ -780,11 +780,11 @@ static void connect_reply_upcall(struct c4iw_ep *ep, int status) | |||
780 | event.private_data_len = ep->plen; | 780 | event.private_data_len = ep->plen; |
781 | event.private_data = ep->mpa_pkt + sizeof(struct mpa_message); | 781 | event.private_data = ep->mpa_pkt + sizeof(struct mpa_message); |
782 | } | 782 | } |
783 | if (ep->com.cm_id) { | 783 | |
784 | PDBG("%s ep %p tid %u status %d\n", __func__, ep, | 784 | PDBG("%s ep %p tid %u status %d\n", __func__, ep, |
785 | ep->hwtid, status); | 785 | ep->hwtid, status); |
786 | ep->com.cm_id->event_handler(ep->com.cm_id, &event); | 786 | ep->com.cm_id->event_handler(ep->com.cm_id, &event); |
787 | } | 787 | |
788 | if (status < 0) { | 788 | if (status < 0) { |
789 | ep->com.cm_id->rem_ref(ep->com.cm_id); | 789 | ep->com.cm_id->rem_ref(ep->com.cm_id); |
790 | ep->com.cm_id = NULL; | 790 | ep->com.cm_id = NULL; |