]> git.kernelconcepts.de Git - karo-tx-uboot.git/commitdiff
dm: usb: Fix type problems in usb_stor_get_info()
authorSimon Glass <sjg@chromium.org>
Wed, 25 Mar 2015 18:22:12 +0000 (12:22 -0600)
committerLothar Waßmann <LW@KARO-electronics.de>
Tue, 8 Sep 2015 19:47:54 +0000 (21:47 +0200)
This function assumes that unsigned long is 32-bits wide, but it is not
on 64-bit machines. Use the correct type, and add a few debug() lines also.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Marek Vasut <marex@denx.de>
common/usb_storage.c

index 62527a98a0cac9bf4913614c0fb390bebc41bb8f..5108d234346f9a8e28964767cc0db464ce8aa2db 100644 (file)
@@ -1191,6 +1191,7 @@ int usb_storage_probe(struct usb_device *dev, unsigned int ifnum,
                        iface->desc.bInterfaceClass != USB_CLASS_MASS_STORAGE ||
                        iface->desc.bInterfaceSubClass < US_SC_MIN ||
                        iface->desc.bInterfaceSubClass > US_SC_MAX) {
+               debug("Not mass storage\n");
                /* if it's not a mass storage, we go no further */
                return 0;
        }
@@ -1317,8 +1318,10 @@ int usb_stor_get_info(struct usb_device *dev, struct us_data *ss,
        pccb->lun = dev_desc->lun;
        debug(" address %d\n", dev_desc->target);
 
-       if (usb_inquiry(pccb, ss))
+       if (usb_inquiry(pccb, ss)) {
+               debug("%s: usb_inquiry() failed\n", __func__);
                return -1;
+       }
 
        perq = usb_stor_buf[0];
        modi = usb_stor_buf[1];
@@ -1328,6 +1331,7 @@ int usb_stor_get_info(struct usb_device *dev, struct us_data *ss,
         * they would not respond to test_unit_ready .
         */
        if (((perq & 0x1f) == 0x1f) || ((perq & 0x1f) == 0x0d)) {
+               debug("%s: unknown/unsupported device\n", __func__);
                return 0;
        }
        if ((modi&0x80) == 0x80) {