summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorSimon Glass <sjg@chromium.org>2015-01-25 08:27:09 -0700
committerSimon Glass <sjg@chromium.org>2015-01-29 17:09:56 -0700
commit1603bf3cc189da65362b83b85831e094a2fe8516 (patch)
tree3fb6880c240011808fb02f34a36965e1a24de3e6 /drivers
parent081f2fcbd9a95ba10677065359791f8fea3f8c58 (diff)
downloadblackbird-obmc-uboot-1603bf3cc189da65362b83b85831e094a2fe8516.tar.gz
blackbird-obmc-uboot-1603bf3cc189da65362b83b85831e094a2fe8516.zip
dm: sandbox: sf: Tidy up the error handling in sandbox_sf_probe()
Use a single exit point when we have an error and add debugging there. Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/mtd/spi/sandbox.c7
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/mtd/spi/sandbox.c b/drivers/mtd/spi/sandbox.c
index 3024b988fe..106dda9ccc 100644
--- a/drivers/mtd/spi/sandbox.c
+++ b/drivers/mtd/spi/sandbox.c
@@ -141,8 +141,10 @@ static int sandbox_sf_probe(struct udevice *dev)
assert(bus->seq != -1);
if (bus->seq < CONFIG_SANDBOX_SPI_MAX_BUS)
spec = state->spi[bus->seq][cs].spec;
- if (!spec)
- return -ENOENT;
+ if (!spec) {
+ ret = -ENOENT;
+ goto error;
+ }
file = strchr(spec, ':');
if (!file) {
@@ -196,6 +198,7 @@ static int sandbox_sf_probe(struct udevice *dev)
return 0;
error:
+ debug("%s: Got error %d\n", __func__, ret);
return ret;
}
OpenPOWER on IntegriCloud