usb: usbtest: two super speed fixes for usbtest

commit 6a23ccd216b6a8ba2c67a9f9d8969b4431ad2920 upstream.

bMaxPacketSize0 field for super speed is a power of 2, not a count.
The size itself is always 512.

Max packet size for a super speed bulk endpoint is 1024, so
allocate the urb size in halt_simple() accordingly.

Signed-off-by: Paul Zimmerman <paulz@synopsys.com>
Acked-by: Felipe Balbi <balbi@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

1 files changed, 12 insertions, 5 deletions

diff --git a/drivers/usb/misc/usbtest.c b/drivers/usb/misc/usbtest.c
index bb10846affc..5707f56d804 100644
--- a/drivers/usb/misc/usbtest.c
+++ b/drivers/usb/misc/usbtest.c
@@ -1023,7 +1023,10 @@ test_ctrl_queue(struct usbtest_dev *dev, struct usbtest_param *param)
                 case 13:        /* short read, resembling case 10 */
                         req.wValue = cpu_to_le16((USB_DT_CONFIG << 8) | 0);
                         /* last data packet "should" be DATA1, not DATA0 */
-                        len = 1024 - udev->descriptor.bMaxPacketSize0;
+                        if (udev->speed == USB_SPEED_SUPER)
+                                len = 1024 - 512;
+                        else
+                                len = 1024 - udev->descriptor.bMaxPacketSize0;
                         expected = -EREMOTEIO;
                         break;
                 case 14:        /* short read; try to fill the last packet */
@@ -1382,11 +1385,15 @@ static int test_halt(struct usbtest_dev *tdev, int ep, struct urb *urb)
 
 static int halt_simple(struct usbtest_dev *dev)
 {
-        int             ep;
-        int             retval = 0;
-        struct urb      *urb;
+        int                     ep;
+        int                     retval = 0;
+        struct urb              *urb;
+        struct usb_device       *udev = testdev_to_usbdev(dev);
 
-        urb = simple_alloc_urb(testdev_to_usbdev(dev), 0, 512);
+        if (udev->speed == USB_SPEED_SUPER)
+                urb = simple_alloc_urb(udev, 0, 1024);
+        else
+                urb = simple_alloc_urb(udev, 0, 512);
         if (urb == NULL)
                 return -ENOMEM;