diff options
author | Harald Seiler <hws@denx.de> | 2020-05-28 18:54:45 +0300 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2020-06-02 20:06:14 +0300 |
commit | 50bb682c5cc52769e7cbd835a00dbcbc75a4f0e7 (patch) | |
tree | 944af99afe1442b42cf68a845900994231d3bdd6 | |
parent | 95712afc17bb4141e7f902983fb27e5bbc5aa62f (diff) | |
download | u-boot-50bb682c5cc52769e7cbd835a00dbcbc75a4f0e7.tar.xz |
tools: fw_env: Fix warning when reading too little
When using CONFIG_ENV_IS_IN_FAT and the config-file specifies a size
larger than what U-Boot wrote into the env-file, a confusing error
message is shown:
$ fw_printenv
Read error on /boot/uboot.env: Success
Fix this by showing a different error message when read returns too
little data.
Signed-off-by: Harald Seiler <hws@denx.de>
-rw-r--r-- | tools/env/fw_env.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/tools/env/fw_env.c b/tools/env/fw_env.c index 8734663cd4..c6378ecf34 100644 --- a/tools/env/fw_env.c +++ b/tools/env/fw_env.c @@ -946,11 +946,17 @@ static int flash_read_buf(int dev, int fd, void *buf, size_t count, lseek(fd, blockstart + block_seek, SEEK_SET); rc = read(fd, buf + processed, readlen); - if (rc != readlen) { + if (rc == -1) { fprintf(stderr, "Read error on %s: %s\n", DEVNAME(dev), strerror(errno)); return -1; } + if (rc != readlen) { + fprintf(stderr, "Read error on %s: " + "Attempted to read %d bytes but got %d\n", + DEVNAME(dev), readlen, rc); + return -1; + } #ifdef DEBUG fprintf(stderr, "Read 0x%x bytes at 0x%llx on %s\n", rc, (unsigned long long)blockstart + block_seek, |