usb: ums: allows using every mmc device with ums.
authorPrzemyslaw Marczak <p.marczak@samsung.com>
Wed, 23 Oct 2013 12:30:43 +0000 (14:30 +0200)
committerChanho Park <chanho61.park@samsung.com>
Fri, 24 Jul 2015 07:29:56 +0000 (16:29 +0900)
Before this change ums command only allowed use of mmc 0.
Now this argument can be set.

Changes:
- remove mmc device number checking because it is always positive number
- remove printing "no such device" - it is done by find_mmc_device()

Change-Id: I767e45151ad515c7bef19e6c13087374f5e23c11
Signed-off-by: Przemyslaw Marczak <p.marczak@samsung.com>
Cc: Marek Vasut <marex@denx.de>
common/cmd_usb_mass_storage.c

index 62c444798435d7aeedd30f077c886c05e716bba2..f360c2d2c91ec7e5a77a64b8fb4ab38ab2b8a979 100644 (file)
@@ -33,10 +33,11 @@ int do_usb_mass_storage(cmd_tbl_t *cmdtp, int flag,
        const char *usb_controller = argv[1];
        const char *mmc_devstring  = argv[2];
 
-       unsigned int dev_num = (unsigned int)(simple_strtoul(mmc_devstring,
-                               NULL, 0));
-       if (dev_num)
-               return CMD_RET_USAGE;
+       unsigned int dev_num = simple_strtoul(mmc_devstring, NULL, 0);
+
+       struct ums *ums = ums_init(dev_num);
+       if (!ums)
+               return CMD_RET_FAILURE;
 
        unsigned int controller_index = (unsigned int)(simple_strtoul(
                                        usb_controller, NULL, 0));
@@ -45,12 +46,6 @@ int do_usb_mass_storage(cmd_tbl_t *cmdtp, int flag,
                return CMD_RET_FAILURE;
        }
 
-       struct ums *ums = ums_init(dev_num);
-       if (!ums) {
-               printf("MMC: %u no such device\n", dev_num);
-               return CMD_RET_FAILURE;
-       }
-
        int rc = fsg_init(ums);
        if (rc) {
                error("fsg_init failed");