On Thu, 18 Feb 2016, Daniel Fraga wrote:

> On Thu, 18 Feb 2016 14:30:15 -0500 (EST)
> Alan Stern <[email protected]> wrote:
> 
> > It looks like there's some problem in the usbhid driver.  Apparently
> > hid_start_in() gets some sort of error when it submits the input URB, 
> > because that URB doesn't show up in the usbmon output.
> > 
> > Can you add a debugging line near the end of hid_post_reset() in
> > drivers/hid/usbhid/hid-core.c to find out what the return value from
> > hid_start_in() is?
> 
>       Yes, can you provide the patch (or the specific line you need)
> so I can recompile it?

Something like the patch below (untested).

Alan Stern


Index: usb-4.4/drivers/hid/usbhid/hid-core.c
===================================================================
--- usb-4.4.orig/drivers/hid/usbhid/hid-core.c
+++ usb-4.4/drivers/hid/usbhid/hid-core.c
@@ -1458,6 +1458,7 @@ static int hid_post_reset(struct usb_int
        spin_unlock_irq(&usbhid->lock);
        hid_set_idle(dev, intf->cur_altsetting->desc.bInterfaceNumber, 0, 0);
        status = hid_start_in(hid);
+       dev_info(&intf->dev, "post reset hid_start_in -> %d\n", status);
        if (status < 0)
                hid_io_error(hid);
        usbhid_restart_queues(usbhid);

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to