diff options
| author |   Benjamin Tissoires <benjamin.tissoires@redhat.com> | 2017-01-20 16:20:13 +0100 |
|---|---|---|
| committer |   Jiri Kosina <jkosina@suse.cz> | 2017-01-23 11:00:13 +0100 |
| commit | a544c619a54bd5e552b1c8c308e77f37f1e3ad7c (patch) | |
| tree | e6cad1fd495f494235f1a64548c06b9b5add4f0a /drivers/hid/wacom.h | |
| parent | HID: wacom: remove warning while disconnecting devices (diff) | |
| download | linux-dev-a544c619a54bd5e552b1c8c308e77f37f1e3ad7c.tar.xz linux-dev-a544c619a54bd5e552b1c8c308e77f37f1e3ad7c.zip | |
HID: wacom: do not attempt to switch mode while in probe
The Intuos Pro seems to not like when we set the features right after
being powered up. Instead of waiting during probe, we can schedule the
switch mode and LED control in a deferred worker so that we don't have the
5 secs of delay from USB when the device is not accessible.
The USB timeout delays were really a pain because if you happen to unplug
the tablet while it is still waiting, you are just adding 5 second timeouts
to the USB stack. Which means that a new plug of the same tablet will also
gets delayed, and will also attempt to access the hardware while in
.probe(). So the tablet doesn't appear in the dmesg, the user unplug/replug
it to make it appearing... and so on so forth.
Really, this is for the best :)
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
Acked-by: Jason Gerecke <jason.gerecke@wacom.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Diffstat (limited to 'drivers/hid/wacom.h')
| -rw-r--r-- | drivers/hid/wacom.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/hid/wacom.h b/drivers/hid/wacom.h index d303e413306d..8d8fe309c872 100644 --- a/drivers/hid/wacom.h +++ b/drivers/hid/wacom.h @@ -166,6 +166,7 @@ struct wacom { struct work_struct wireless_work; struct work_struct battery_work; struct work_struct remote_work; + struct delayed_work init_work; struct wacom_remote *remote; struct wacom_leds { struct wacom_group_leds *groups; |