Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit b2a232d2 authored by Leon Romanovsky's avatar Leon Romanovsky Committed by Doug Ledford
Browse files

IB/mlx5: Simplify code by removing return variable 



Return variable was set in a line before the
actual return was called in begin_wqe function.

This patch removes such variable and simplifies the code.

Signed-off-by: default avatarLeon Romanovsky <leonro@mellanox.com>
Signed-off-by: default avatarDoug Ledford <dledford@redhat.com>
parent 24be409b

drivers/infiniband/hw/mlx5/qp.c

+3 −7
Original line number Diff line number Diff line
@@ -3658,12 +3658,8 @@ static int begin_wqe(struct mlx5_ib_qp *qp, void **seg, 
		     struct ib_send_wr *wr, unsigned *idx,

		     int *size, int nreq)

{

	int err = 0;



	if (unlikely(mlx5_wq_overflow(&qp->sq, nreq, qp->ibqp.send_cq))) {

		err = -ENOMEM;

		return err;

	}

	if (unlikely(mlx5_wq_overflow(&qp->sq, nreq, qp->ibqp.send_cq)))

		return -ENOMEM;



	*idx = qp->sq.cur_post & (qp->sq.wqe_cnt - 1);

	*seg = mlx5_get_send_wqe(qp, *idx);
@@ -3679,7 +3675,7 @@ static int begin_wqe(struct mlx5_ib_qp *qp, void **seg, 
	*seg += sizeof(**ctrl);

	*size = sizeof(**ctrl) / 16;



	return err;

	return 0;

}



static void finish_wqe(struct mlx5_ib_qp *qp,