]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/commitdiff
RDMA/nes: Remove unnecessary if-else statement
authorTatyana Nikolova <Tatyana.E.Nikolova@intel.com>
Tue, 2 Oct 2012 16:40:33 +0000 (16:40 +0000)
committerRoland Dreier <roland@purestorage.com>
Wed, 3 Oct 2012 21:27:58 +0000 (14:27 -0700)
Remove unnecessary if-else statement -- we do the same thing either way.

Signed-off-by: Tatyana Nikolova <Tatyana.E.Nikolova@intel.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
drivers/infiniband/hw/nes/nes_verbs.c

index 9eb54b7794fbb4a5dfd78dd95e947114f0f30d5e..cd0ecb215cca01c5be942da3bf4c07bcc3895c5f 100644 (file)
@@ -3069,18 +3069,9 @@ int nes_modify_qp(struct ib_qp *ibqp, struct ib_qp_attr *attr,
                }
 
                nesqp->ibqp_state = attr->qp_state;
-               if (((nesqp->iwarp_state & NES_CQP_QP_IWARP_STATE_MASK) ==
-                               (u32)NES_CQP_QP_IWARP_STATE_RTS) &&
-                               ((next_iwarp_state & NES_CQP_QP_IWARP_STATE_MASK) >
-                               (u32)NES_CQP_QP_IWARP_STATE_RTS)) {
-                       nesqp->iwarp_state = next_iwarp_state & NES_CQP_QP_IWARP_STATE_MASK;
-                       nes_debug(NES_DBG_MOD_QP, "Change nesqp->iwarp_state=%08x\n",
-                                       nesqp->iwarp_state);
-               } else {
-                       nesqp->iwarp_state = next_iwarp_state & NES_CQP_QP_IWARP_STATE_MASK;
-                       nes_debug(NES_DBG_MOD_QP, "Change nesqp->iwarp_state=%08x\n",
-                                       nesqp->iwarp_state);
-               }
+               nesqp->iwarp_state = next_iwarp_state & NES_CQP_QP_IWARP_STATE_MASK;
+               nes_debug(NES_DBG_MOD_QP, "Change nesqp->iwarp_state=%08x\n",
+                               nesqp->iwarp_state);
        }
 
        if (attr_mask & IB_QP_ACCESS_FLAGS) {