Use this new function since it implements the required functionality and
reduces duplicated code.

Signed-off-by: Simon Glass <s...@chromium.org>
---

 cmd/cat.c | 47 +++++++++++++++++------------------------------
 1 file changed, 17 insertions(+), 30 deletions(-)

diff --git a/cmd/cat.c b/cmd/cat.c
index 1273a26b1457..b059080193d4 100644
--- a/cmd/cat.c
+++ b/cmd/cat.c
@@ -17,8 +17,8 @@ static int do_cat(struct cmd_tbl *cmdtp, int flag, int argc,
        char *dev;
        char *file;
        char *buffer;
-       phys_addr_t addr;
-       loff_t file_size;
+       ulong file_size;
+       int ret;
 
        if (argc < 4)
                return CMD_RET_USAGE;
@@ -27,40 +27,27 @@ static int do_cat(struct cmd_tbl *cmdtp, int flag, int argc,
        dev = argv[2];
        file = argv[3];
 
+       ret = fs_load_alloc(ifname, dev, file, 0, 0, (void **)&buffer,
+                           &file_size);
+
        // check file exists
-       if (fs_set_blk_dev(ifname, dev, FS_TYPE_ANY))
+       switch (ret) {
+       case 0:
+               break;
+       case -ENOMEDIUM:
                return CMD_RET_FAILURE;
-
-       if (!fs_exists(file)) {
+       case -ENOENT:
                log_err("File does not exist: ifname=%s dev=%s file=%s\n", 
ifname, dev, file);
                return CMD_RET_FAILURE;
-       }
-
-       // get file size
-       if (fs_set_blk_dev(ifname, dev, FS_TYPE_ANY))
+       case -E2BIG:
+               log_err("File is too large: ifname=%s dev=%s file=%s\n", 
ifname, dev, file);
                return CMD_RET_FAILURE;
-
-       if (fs_size(file, &file_size)) {
-               log_err("Cannot read file size: ifname=%s dev=%s file=%s\n", 
ifname, dev, file);
+       case -ENOMEM:
+               log_err("Not enough memory: ifname=%s dev=%s file=%s\n", 
ifname, dev, file);
                return CMD_RET_FAILURE;
-       }
-
-       // allocate memory for file content
-       buffer = calloc(sizeof(char), file_size + 1);
-       if (!buffer) {
-               log_err("Out of memory\n");
-               return CMD_RET_FAILURE;
-       }
-
-       // map pointer to system memory
-       addr = map_to_sysmem(buffer);
-
-       // read file to memory
-       if (fs_set_blk_dev(ifname, dev, FS_TYPE_ANY))
-               return CMD_RET_FAILURE;
-
-       if (fs_read(file, addr, 0, 0, &file_size)) {
-               log_err("Cannot read file: ifname=%s dev=%s file=%s\n", ifname, 
dev, file);
+       default:
+       case -EIO:
+               log_err("File-read failed: ifname=%s dev=%s file=%s\n", ifname, 
dev, file);
                return CMD_RET_FAILURE;
        }
 
-- 
2.41.0.rc0.172.g3f132b7071-goog

Reply via email to