diff options
| author | Simon Glass <sjg@chromium.org> | 2015-01-25 08:27:09 -0700 |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2015-01-29 17:09:56 -0700 |
| commit | 1603bf3cc189da65362b83b85831e094a2fe8516 (patch) | |
| tree | 3fb6880c240011808fb02f34a36965e1a24de3e6 | |
| parent | 081f2fcbd9a95ba10677065359791f8fea3f8c58 (diff) | |
| download | u-boot-1603bf3cc189da65362b83b85831e094a2fe8516.tar.gz u-boot-1603bf3cc189da65362b83b85831e094a2fe8516.tar.xz u-boot-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>
| -rw-r--r-- | drivers/mtd/spi/sandbox.c | 7 |
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; } |