On Mon, 14 Jul 2014, Johan Hovold wrote:
> From 8101c0dfd42a232f1d2872de4f412d8d61d5646f Mon Sep 17 00:00:00 2001
> From: Johan Hovold <johan@xxxxxxxxxx>
> Date: Mon, 14 Jul 2014 18:43:31 +0200
> Subject: [PATCH] HID: usbhid: add HID_QUIRK_IN
>
> Add quirk to submit the interrupt-in urb already at start() rather than
> at open().
>
> This is needed for devices that disconnects from the bus unless the
> interrupt endpoint has been polled at least once or when not responding
> to input events.
It's not really super-nice, but if no other way around it has been found
to be possible in USB core, I am willing to take this.
> ---
> drivers/hid/usbhid/hid-core.c | 26 +++++++++++++++++++++++---
> include/linux/hid.h | 1 +
> 2 files changed, 24 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/hid/usbhid/hid-core.c b/drivers/hid/usbhid/hid-core.c
> index 7b88f4c..4b5d986 100644
> --- a/drivers/hid/usbhid/hid-core.c
> +++ b/drivers/hid/usbhid/hid-core.c
> @@ -82,7 +82,7 @@ static int hid_start_in(struct hid_device *hid)
> struct usbhid_device *usbhid = hid->driver_data;
>
> spin_lock_irqsave(&usbhid->lock, flags);
> - if (hid->open > 0 &&
> + if ((hid->open > 0 || hid->quirks & HID_QUIRK_IN) &&
> !test_bit(HID_DISCONNECTED, &usbhid->iofl) &&
> !test_bit(HID_SUSPENDED, &usbhid->iofl) &&
> !test_and_set_bit(HID_IN_RUNNING, &usbhid->iofl)) {
> @@ -292,6 +292,8 @@ static void hid_irq_in(struct urb *urb)
> case 0: /* success */
> usbhid_mark_busy(usbhid);
> usbhid->retry_delay = 0;
> + if ((hid->quirks & HID_QUIRK_IN) && !hid->open)
> + break;
> hid_input_report(urb->context, HID_INPUT_REPORT,
> urb->transfer_buffer,
> urb->actual_length, 1);
> @@ -734,8 +736,10 @@ void usbhid_close(struct hid_device *hid)
> if (!--hid->open) {
> spin_unlock_irq(&usbhid->lock);
> hid_cancel_delayed_stuff(usbhid);
> - usb_kill_urb(usbhid->urbin);
> - usbhid->intf->needs_remote_wakeup = 0;
> + if (!(hid->quirks & HID_QUIRK_IN)) {
> + usb_kill_urb(usbhid->urbin);
> + usbhid->intf->needs_remote_wakeup = 0;
> + }
> } else {
> spin_unlock_irq(&usbhid->lock);
> }
> @@ -1133,6 +1137,20 @@ static int usbhid_start(struct hid_device *hid)
>
> set_bit(HID_STARTED, &usbhid->iofl);
>
> + hid->quirks |= HID_QUIRK_IN; /* FIXME */
This of course needs to be set on per-device basis :)
> + if (hid->quirks & HID_QUIRK_IN) {
> + ret = usb_autopm_get_interface(usbhid->intf);
> + if (ret)
> + goto fail;
> + usbhid->intf->needs_remote_wakeup = 1;
> + ret = hid_start_in(hid);
> + if (ret) {
> + dev_err(&hid->dev,
> + "failed to start in urb: %d\n", ret);
> + }
> + usb_autopm_put_interface(usbhid->intf);
> + }
> +
> /* Some keyboards don't work until their LEDs have been set.
> * Since BIOSes do set the LEDs, it must be safe for any device
> * that supports the keyboard boot protocol.
> @@ -1165,6 +1183,8 @@ static void usbhid_stop(struct hid_device *hid)
> if (WARN_ON(!usbhid))
> return;
>
> + usbhid->intf->needs_remote_wakeup = 0;
> +
> clear_bit(HID_STARTED, &usbhid->iofl);
> spin_lock_irq(&usbhid->lock); /* Sync with error and led handlers */
> set_bit(HID_DISCONNECTED, &usbhid->iofl);
> diff --git a/include/linux/hid.h b/include/linux/hid.h
> index 77632cf..13f81ae 100644
> --- a/include/linux/hid.h
> +++ b/include/linux/hid.h
> @@ -286,6 +286,7 @@ struct hid_item {
> #define HID_QUIRK_HIDINPUT_FORCE 0x00000080
> #define HID_QUIRK_NO_EMPTY_INPUT 0x00000100
> #define HID_QUIRK_NO_INIT_INPUT_REPORTS 0x00000200
> +#define HID_QUIRK_IN 0x00000400
0x00000400 has been removed before dynamic quirks started to be possible,
so there is no potential clash, that's fine.
I'd just propose some more descriptive name for the quirk ... how about
something like HID_QUIRK_EARLY_INTERRUPT?
Thanks,
--
Jiri Kosina
SUSE Labs
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
