diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-02-06 07:48:23 -0500 |
---|---|---|
committer | Roland Dreier <roland@purestorage.com> | 2014-03-17 19:29:37 -0400 |
commit | 349850f0a9188cb87b4ef6c01c057cb42cca478e (patch) | |
tree | ab7bacf61b277801c276057ef20edf7ae1afa5d2 /drivers/infiniband | |
parent | 43adff3979c8f887b364c887e447c443e2f835e8 (diff) |
RDMA/nes: Clean up a condition
We don't need to test "ret" twice and also the white space is messed up.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Tatyana Nikolova <tatyana.e.nikolova@intel.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
Diffstat (limited to 'drivers/infiniband')
-rw-r--r-- | drivers/infiniband/hw/nes/nes_verbs.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/infiniband/hw/nes/nes_verbs.c b/drivers/infiniband/hw/nes/nes_verbs.c index 797d8876d948..9722eeeaadaa 100644 --- a/drivers/infiniband/hw/nes/nes_verbs.c +++ b/drivers/infiniband/hw/nes/nes_verbs.c | |||
@@ -3136,9 +3136,7 @@ int nes_modify_qp(struct ib_qp *ibqp, struct ib_qp_attr *attr, | |||
3136 | " original_last_aeq = 0x%04X. last_aeq = 0x%04X.\n", | 3136 | " original_last_aeq = 0x%04X. last_aeq = 0x%04X.\n", |
3137 | nesqp->hwqp.qp_id, atomic_read(&nesqp->refcount), | 3137 | nesqp->hwqp.qp_id, atomic_read(&nesqp->refcount), |
3138 | original_last_aeq, nesqp->last_aeq); | 3138 | original_last_aeq, nesqp->last_aeq); |
3139 | if ((!ret) || | 3139 | if (!ret || original_last_aeq != NES_AEQE_AEID_RDMAP_ROE_BAD_LLP_CLOSE) { |
3140 | ((original_last_aeq != NES_AEQE_AEID_RDMAP_ROE_BAD_LLP_CLOSE) && | ||
3141 | (ret))) { | ||
3142 | if (dont_wait) { | 3140 | if (dont_wait) { |
3143 | if (nesqp->cm_id && nesqp->hw_tcp_state != 0) { | 3141 | if (nesqp->cm_id && nesqp->hw_tcp_state != 0) { |
3144 | nes_debug(NES_DBG_MOD_QP, "QP%u Queuing fake disconnect for QP refcount (%d)," | 3142 | nes_debug(NES_DBG_MOD_QP, "QP%u Queuing fake disconnect for QP refcount (%d)," |