diff options
| author | Dave Jones <davej@redhat.com> | 2013-09-17 23:45:54 +0200 |
|---|---|---|
| committer | Roland Dreier <roland@purestorage.com> | 2013-11-09 11:34:27 +0100 |
| commit | 4127c365c9dbd4412e70b1c01f6ae1e3624cafac (patch) | |
| tree | c46a2364d63b5f55b7042f8a71d07e8bced999d0 /drivers/infiniband/hw | |
| parent | Linux 3.12-rc7 (diff) | |
| download | linux-4127c365c9dbd4412e70b1c01f6ae1e3624cafac.tar.xz linux-4127c365c9dbd4412e70b1c01f6ae1e3624cafac.zip | |
RDMA/nes: Remove self-assignment from nes_query_qp()
Assigning a value to itself is pointless.
Spotted with coverity, no hardware to test.
Signed-off-by: Dave Jones <davej@fedoraproject.org>
Signed-off-by: Roland Dreier <roland@purestorage.com>
Diffstat (limited to 'drivers/infiniband/hw')
| -rw-r--r-- | drivers/infiniband/hw/nes/nes_verbs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/infiniband/hw/nes/nes_verbs.c b/drivers/infiniband/hw/nes/nes_verbs.c index 5b53ca5a2284..8308e3634767 100644 --- a/drivers/infiniband/hw/nes/nes_verbs.c +++ b/drivers/infiniband/hw/nes/nes_verbs.c @@ -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; |