From: Mike Frysinger <vapier@gentoo.org>
Date: Wed, 15 Jun 2011 19:29:19 +0000 (-0400)
Subject: ASoC: AD1836: drop unnecessary spi register check
X-Git-Tag: v3.3-rc1~14^2~943
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=15e870512956a1e573d033b3bb6ffbf3237e9723;p=platform%2Fkernel%2Flinux-3.10.git

ASoC: AD1836: drop unnecessary spi register check

The only thing the init func does is register a spi driver, so if that
fails, we return the value back up to the caller who will display an
error message for us.  So drop the redundant checking/message.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Acked-by: Liam Girdwood <lrg@ti.com>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
---

diff --git a/sound/soc/codecs/ad1836.c b/sound/soc/codecs/ad1836.c
index ff8e738..e8a986f 100644
--- a/sound/soc/codecs/ad1836.c
+++ b/sound/soc/codecs/ad1836.c
@@ -385,15 +385,7 @@ static struct spi_driver ad1836_spi_driver = {
 
 static int __init ad1836_init(void)
 {
-	int ret;
-
-	ret = spi_register_driver(&ad1836_spi_driver);
-	if (ret != 0) {
-		printk(KERN_ERR "Failed to register ad1836 SPI driver: %d\n",
-				ret);
-	}
-
-	return ret;
+	return spi_register_driver(&ad1836_spi_driver);
 }
 module_init(ad1836_init);