EXT4: Fix number base handling of "ext4write" command
authorWolfgang Denk <wd@denx.de>
Fri, 31 Jan 2014 08:28:25 +0000 (09:28 +0100)
committerTom Rini <trini@ti.com>
Wed, 19 Feb 2014 15:47:36 +0000 (10:47 -0500)
Unlike other commands (for example, "fatwrite"), ext4write would
interpret the "sizebytes" as decimal number.  This is not only
inconsistend and unexpected to most users, it also breaks usage
like this:

tftp ${addr} ${name}
ext4write mmc 0:2 ${addr} ${filename} ${filesize}

Change this to use the standard notation of base 16 input format.
See also commit b770e88

WARNING: this is a change to the user interface!!

Signed-off-by: Wolfgang Denk <wd@denx.de>
Cc: Uma Shankar <uma.shankar@samsung.com>
Cc: Stephen Warren <swarren@nvidia.com>
common/cmd_ext4.c

index 8289d25b0623468017282393e9127841918fbba5..68b047ba6aed1b07509b7ffd8b26fc715c139ce6 100644 (file)
@@ -79,8 +79,8 @@ int do_ext4_write(cmd_tbl_t *cmdtp, int flag, int argc,
        /* get the address in hexadecimal format (string to int) */
        ram_address = simple_strtoul(argv[3], NULL, 16);
 
-       /* get the filesize in base 10 format */
-       file_size = simple_strtoul(argv[5], NULL, 10);
+       /* get the filesize in hexadecimal format */
+       file_size = simple_strtoul(argv[5], NULL, 16);
 
        /* set the device as block device */
        ext4fs_set_blk_dev(dev_desc, &info);