Commit 51d00293 authored by Glen Lee's avatar Glen Lee Committed by Greg Kroah-Hartman

staging: wilc1000: linux_sdio_probe: use return value

Return ret from wilc_netdev_init instead of -1 for proper error handling.
Signed-off-by: default avatarGlen Lee <glen.lee@atmel.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 0f34e924
...@@ -91,7 +91,7 @@ static int linux_sdio_probe(struct sdio_func *func, ...@@ -91,7 +91,7 @@ static int linux_sdio_probe(struct sdio_func *func,
const struct sdio_device_id *id) const struct sdio_device_id *id)
{ {
struct wilc *wilc; struct wilc *wilc;
int gpio; int gpio, ret;
gpio = -1; gpio = -1;
if (IS_ENABLED(CONFIG_WILC1000_HW_OOB_INTR)) { if (IS_ENABLED(CONFIG_WILC1000_HW_OOB_INTR)) {
...@@ -101,10 +101,11 @@ static int linux_sdio_probe(struct sdio_func *func, ...@@ -101,10 +101,11 @@ static int linux_sdio_probe(struct sdio_func *func,
} }
dev_dbg(&func->dev, "Initializing netdev\n"); dev_dbg(&func->dev, "Initializing netdev\n");
if (wilc_netdev_init(&wilc, &func->dev, HIF_SDIO, gpio, ret = wilc_netdev_init(&wilc, &func->dev, HIF_SDIO, gpio,
&wilc_hif_sdio)) { &wilc_hif_sdio);
if (ret) {
dev_err(&func->dev, "Couldn't initialize netdev\n"); dev_err(&func->dev, "Couldn't initialize netdev\n");
return -1; return ret;
} }
sdio_set_drvdata(func, wilc); sdio_set_drvdata(func, wilc);
wilc->dev = &func->dev; wilc->dev = &func->dev;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment