From: Dan Carpenter <error27@gmail.com>
Date: Tue, 12 May 2009 10:33:39 +0000 (+0100)
Subject: Remove unreached code in drivers/net/mlx4/en_rx.c
X-Git-Tag: v2.6.30-rc6~36
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=34fd5dad46fc494c6ed3b174b2c991ea9961ebe9;p=platform%2Fupstream%2Fkernel-adaptation-pc.git

Remove unreached code in drivers/net/mlx4/en_rx.c

Remove the return after the goto.  We want the goto because it frees
memory as well as returning err.

Found by smatch (http://repo.or.cz/w/smatch.git).

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
---

diff --git a/drivers/net/mlx4/en_rx.c b/drivers/net/mlx4/en_rx.c
index 7942c4d..9ee873e 100644
--- a/drivers/net/mlx4/en_rx.c
+++ b/drivers/net/mlx4/en_rx.c
@@ -951,7 +951,6 @@ static int mlx4_en_config_rss_qp(struct mlx4_en_priv *priv,
 	if (err) {
 		mlx4_err(mdev, "Failed to allocate qp #%d\n", qpn);
 		goto out;
-		return err;
 	}
 	qp->event = mlx4_en_sqp_event;