Messages in this thread | | | Date | Thu, 13 Nov 2008 23:05:09 +0100 | From | Jiri Slaby <> | Subject | Re: [PATCH] HID: don't grab devices with no input |
| |
On 11/13/2008 10:37 PM, Alan Stern wrote: >> + for (n = 0; n < interface->desc.bNumEndpoints; n++) >> + if (usb_endpoint_dir_in(&interface->endpoint[n].desc)) >> + has_in++; >> + if (!has_in) { >> + dev_err(&intf->dev, "couldn't find an input interrupt " >> + "endpoint"); >> + return -ENODEV; >> + } >> + > > Do you want to use usb_endpoint_is_int_in() instead? It matches the > error message more closely.
Yes, good catch, thanks.
| |