IB/ocrdma: Skip using unneeded intermediate variable
authorMarkus Elfring <elfring@users.sourceforge.net>
Sat, 26 Dec 2015 17:28:35 +0000 (18:28 +0100)
committerDoug Ledford <dledford@redhat.com>
Thu, 3 Mar 2016 18:33:10 +0000 (13:33 -0500)
Return zero at the end without using the local variable "status".

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Doug Ledford <dledford@redhat.com>
drivers/infiniband/hw/ocrdma/ocrdma_hw.c

index 9b8ff26..16740dc 100644 (file)
@@ -2141,7 +2141,6 @@ int ocrdma_qp_state_change(struct ocrdma_qp *qp, enum ib_qp_state new_ib_state,
                           enum ib_qp_state *old_ib_state)
 {
        unsigned long flags;
-       int status = 0;
        enum ocrdma_qp_state new_state;
        new_state = get_ocrdma_qp_state(new_ib_state);
 
@@ -2166,7 +2165,7 @@ int ocrdma_qp_state_change(struct ocrdma_qp *qp, enum ib_qp_state new_ib_state,
        qp->state = new_state;
 
        spin_unlock_irqrestore(&qp->q_lock, flags);
-       return status;
+       return 0;
 }
 
 static u32 ocrdma_set_create_qp_mbx_access_flags(struct ocrdma_qp *qp)