diff options
| author | Simon Glass <sjg@chromium.org> | 2020-07-19 10:15:32 -0600 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2020-08-03 22:19:54 -0400 |
| commit | e567ec849a9aba4cd683ea23bd57c878b59714c4 (patch) | |
| tree | b60237cd3ceb1dc7bc9af89d2e3329e45d364bf7 /drivers/mtd/spi/sf_probe.c | |
| parent | f38a29997d1cdb71a85fc9dcdbee86d952b45482 (diff) | |
| download | u-boot-e567ec849a9aba4cd683ea23bd57c878b59714c4.tar.gz u-boot-e567ec849a9aba4cd683ea23bd57c878b59714c4.tar.xz u-boot-e567ec849a9aba4cd683ea23bd57c878b59714c4.zip | |
mtd: spi-nor: Tidy up error handling / debug code
The -ENODEV error value in spi_nor_read_id() is incorrect since there
clearly is a device - it just cannot be supported. Use -ENOMEDIUM instead
which has the virtue of being less common.
Fix the return value in spi_nor_scan().
Also there are a few printf() statements which should be debug() since
they bloat the code with unused strings at present. Fix those while here.
Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'drivers/mtd/spi/sf_probe.c')
| -rw-r--r-- | drivers/mtd/spi/sf_probe.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mtd/spi/sf_probe.c b/drivers/mtd/spi/sf_probe.c index 475f6c31db..b959e3453a 100644 --- a/drivers/mtd/spi/sf_probe.c +++ b/drivers/mtd/spi/sf_probe.c @@ -119,7 +119,7 @@ static int spi_flash_std_erase(struct udevice *dev, u32 offset, size_t len) struct erase_info instr; if (offset % mtd->erasesize || len % mtd->erasesize) { - printf("SF: Erase offset/length not multiple of erase size\n"); + debug("SF: Erase offset/length not multiple of erase size\n"); return -EINVAL; } |