]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
usb: return 0 from usb_stor_get_info even if removable media
authorTroy Kisky <troy.kisky@boundarydevices.com>
Tue, 11 Apr 2017 01:23:11 +0000 (18:23 -0700)
committerMarek Vasut <marex@denx.de>
Fri, 14 Apr 2017 14:44:17 +0000 (16:44 +0200)
This fixes a regression caused by

commit 07b2b78ce4bc8ae25e066c65245eaf58c0d9a67c
    dm: usb: Convert USB storage to use driver-model for block devs

which caused part_init to be called when it was not previously.
Without this patch, the following happens when a USB sd card reader is used.

=> usb start
starting USB...
USB0:   Port not available.
USB1:   USB EHCI 1.00
scanning bus 1 for devices... 3 USB Device(s) found
       scanning usb for storage devices... Device NOT ready
   Request Sense returned 02 3A 00
 ### ERROR ### Please RESET the board ###

This happens because dev_desc->blksz is 0.

Signed-off-by: Troy Kisky <troy.kisky@boundarydevices.com>
common/usb_storage.c

index b524a15e2bf93714c9ea758046d6c651decc4723..83279c449df5b7d8e844f0942dad69b05fad9101 100644 (file)
@@ -1440,10 +1440,8 @@ int usb_stor_get_info(struct usb_device *dev, struct us_data *ss,
                       "   Request Sense returned %02X %02X %02X\n",
                       pccb->sense_buf[2], pccb->sense_buf[12],
                       pccb->sense_buf[13]);
-               if (dev_desc->removable == 1) {
+               if (dev_desc->removable == 1)
                        dev_desc->type = perq;
-                       return 1;
-               }
                return 0;
        }
        pccb->pdata = (unsigned char *)cap;