infiniband: remove self-assignment from nes_query_qp()

PATCH Dave Jones 1 participant 1 message Sep 17, 2013 InfiniBand

lore

Share & tools

lore

Dave Jones

author

2013-09-17 21:45 UTC #0 lore raw

Assigning a value to itself is pointless.

Spotted with coverity, no hardware to test.

Signed-off-by Dave Jones <davej-rxtnV0ftBwyoClj4AeEUq9i2O/[email protected]>

drivers/infiniband/hw/nes/nes_verbs.c +1 -1

index 5b53ca5..8308e36 100644
@@ -2834,7 +2834,7 @@ static int nes_query_qp(struct ib_qp *ibqp, struct ib_qp_attr *attr,
 	init_attr->qp_context = nesqp->ibqp.qp_context;
 	init_attr->send_cq = nesqp->ibqp.send_cq;
 	init_attr->recv_cq = nesqp->ibqp.recv_cq;
-	init_attr->srq = nesqp->ibqp.srq = nesqp->ibqp.srq;
+	init_attr->srq = nesqp->ibqp.srq;
 	init_attr->cap = attr->cap;
 
 	return 0;

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html