On Tue, 1 Mar 2016, Daniel Fraga wrote:
> On Tue, 1 Mar 2016 10:25:30 -0500 (EST)
> Alan Stern <st...@rowland.harvard.edu> wrote:
>
> > Now we're making progress! That shows a problem right there; we ought
> > to have more stuff about 3-1.6 between those two lines.
> >
> > The next patch adds some more debugging output. For this test you
> > don't even have to suspend the system; all I need to see is the output
> > for 3-1.6 during boot-up and shortly thereafter.
>
> No new messages during boot-up (I tried to reboot 3 times). I just got
> the following:
>
> Mar 1 16:33:55 tux kernel: [ 3.558535] usb 3-1.6: new full-speed USB
> device number 6 using ehci-pci
No messages about "usbhid_start urb" or "no input endpoint!" or
"usbhid_start fail urb"? That means usbhid_start() isn't getting
called. Which means the device in question probably isn't being used
at all.
Do you know what this device is? What does
/sys/kernel/debug/usb/devices show?
Let's make sure this is really what's happening. Please try this
patch.
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
@@ -1159,6 +1159,7 @@ static int usbhid_start(struct hid_devic
usbhid_set_leds(hid);
device_set_wakeup_enable(&dev->dev, 1);
}
+ dev_info(&usbhid->intf->dev, "usbhid_start -> 0, %p\n", usbhid->urbin);
return 0;
fail:
@@ -1169,6 +1170,7 @@ fail:
usbhid->urbout = NULL;
usbhid->urbctrl = NULL;
hid_free_buffers(dev, hid);
+ dev_info(&usbhid->intf->dev, "usbhid_start -> %d\n", ret);
return ret;
}
@@ -1178,6 +1180,7 @@ static void usbhid_stop(struct hid_devic
if (WARN_ON(!usbhid))
return;
+ dev_info(&usbhid->intf->dev, "usbhid_stop\n");
if (hid->quirks & HID_QUIRK_ALWAYS_POLL)
usbhid->intf->needs_remote_wakeup = 0;
@@ -1366,11 +1369,13 @@ static int usbhid_probe(struct usb_inter
goto err_free;
}
+ dev_info(&intf->dev, "usbhid_probe -> %d\n", 0);
return 0;
err_free:
kfree(usbhid);
err:
hid_destroy_device(hid);
+ dev_info(&intf->dev, "usbhid_probe -> %d\n", ret);
return ret;
}
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
