Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1352943 > unrolled thread

[patch] libertas: fix an error code in probe

Started byDan Carpenter <dan.carpenter@oracle.com>
First post2016-03-08 13:10 +0100
Last post2016-03-10 14:00 +0100
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [patch] libertas: fix an error code in probe Dan Carpenter <dan.carpenter@oracle.com> - 2016-03-08 13:10 +0100
    Re: libertas: fix an error code in probe Kalle Valo <kvalo@codeaurora.org> - 2016-03-10 14:00 +0100

#1352943 — [patch] libertas: fix an error code in probe

FromDan Carpenter <dan.carpenter@oracle.com>
Date2016-03-08 13:10 +0100
Subject[patch] libertas: fix an error code in probe
Message-ID<raoCC-DB-15@gated-at.bofh.it>
We accidentally return success instead of a negative error code.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>

diff --git a/drivers/net/wireless/marvell/libertas/main.c b/drivers/net/wireless/marvell/libertas/main.c
index b35b8bc..8541cbe 100644
--- a/drivers/net/wireless/marvell/libertas/main.c
+++ b/drivers/net/wireless/marvell/libertas/main.c
@@ -1118,7 +1118,8 @@ int lbs_start_card(struct lbs_private *priv)
 	else
 		pr_info("%s: mesh disabled\n", dev->name);
 
-	if (lbs_cfg_register(priv)) {
+	ret = lbs_cfg_register(priv);
+	if (ret) {
 		pr_err("cannot register device\n");
 		goto done;
 	}

[toc] | [next] | [standalone]


#1355094 — Re: libertas: fix an error code in probe

FromKalle Valo <kvalo@codeaurora.org>
Date2016-03-10 14:00 +0100
SubjectRe: libertas: fix an error code in probe
Message-ID<rb8m6-78l-3@gated-at.bofh.it>
In reply to#1352943
> We accidentally return success instead of a negative error code.
> 
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>

Thanks, applied to wireless-drivers-next.git.

Kalle Valo

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web