[PATCH] Check the return value

michael michael at panicking.kicks-ass.org
Wed Mar 18 12:01:41 CET 2009


Signed-off-by: Michael Trimarchi <michael at panicking.kicks-ass.org>
---
 drivers/ar6000/hif/hif2.c |    9 ++++++++-
 1 files changed, 8 insertions(+), 1 deletions(-)

diff --git a/drivers/ar6000/hif/hif2.c b/drivers/ar6000/hif/hif2.c
index 571833f..ccfd8e1 100644
--- a/drivers/ar6000/hif/hif2.c
+++ b/drivers/ar6000/hif/hif2.c
@@ -508,7 +508,8 @@ static int ar6000_do_activate(struct hif_device *hif)
 	sdio_release_host(func);
 
 	hif->io_task = kthread_run(io, hif, "ar6000_io");
-	if (IS_ERR(hif->io_task)) {
+	ret = IS_ERR(hif->io_task);
+	if (ret) {
 		dev_err(dev, "kthread_run(ar6000_io): %d\n", ret);
 		goto out_func_ready;
 	}
@@ -580,8 +581,14 @@ static int ar6000_activate(struct hif_device *hif)
 	mutex_lock(&hif->activate_lock);
 	if (!hif->active) {
 		ret = ar6000_do_activate(hif);
+		if (ret) {
+			printk(KERN_ERR "%s: Failed to activate %d\n",
+				__FUNCTION__, ret);
+			goto out;
+		}
 		hif->active = 1;
 	}
+out:
 	mutex_unlock(&hif->activate_lock);
 	return ret;
 }
-- 
1.5.6.5




More information about the openmoko-kernel mailing list