Skip to content

Commit 2232973

Browse files
committed
HID: wacom: Ensure bootloader PID is usable in hidraw mode
Some Wacom devices have a special "bootloader" mode that is used for firmware flashing. When operating in this mode, the device cannot be used for input, and the HID descriptor is not able to be processed by the driver. The driver generates an "Unknown device_type" warning and then returns an error code from wacom_probe(). This is a problem because userspace still needs to be able to interact with the device via hidraw to perform the firmware flash. This commit adds a non-generic device definition for 056a:0094 which is used when devices are in "bootloader" mode. It marks the devices with a special BOOTLOADER type that is recognized by wacom_probe() and wacom_raw_event(). When we see this type we ensure a hidraw device is created and otherwise keep our hands off so that userspace is in full control. Signed-off-by: Jason Gerecke <[email protected]> Tested-by: Tatsunosuke Tobita <[email protected]> Cc: <[email protected]> Signed-off-by: Jiri Kosina <[email protected]> [[email protected]: Imported into input-wacom (1db1f39259)] Signed-off-by: Jason Gerecke <[email protected]>
1 parent bcc02aa commit 2232973

File tree

3 files changed

+13
-0
lines changed

3 files changed

+13
-0
lines changed

4.5/wacom_sys.c

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -193,6 +193,9 @@ static int wacom_raw_event(struct hid_device *hdev, struct hid_report *report,
193193
{
194194
struct wacom *wacom = hid_get_drvdata(hdev);
195195

196+
if (wacom->wacom_wac.features.type == BOOTLOADER)
197+
return 0;
198+
196199
if (size > WACOM_PKGLEN_MAX)
197200
return 1;
198201

@@ -2849,6 +2852,11 @@ static int wacom_probe(struct hid_device *hdev,
28492852
return error;
28502853
}
28512854

2855+
if (features->type == BOOTLOADER) {
2856+
hid_warn(hdev, "Using device in hidraw-only mode");
2857+
return hid_hw_start(hdev, HID_CONNECT_HIDRAW);
2858+
}
2859+
28522860
error = wacom_parse_and_register(wacom, false);
28532861
if (error)
28542862
return error;

4.5/wacom_wac.c

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4897,6 +4897,9 @@ static const struct wacom_features wacom_features_0x3dd =
48974897
static const struct wacom_features wacom_features_HID_ANY_ID =
48984898
{ "Wacom HID", .type = HID_GENERIC, .oVid = HID_ANY_ID, .oPid = HID_ANY_ID };
48994899

4900+
static const struct wacom_features wacom_features_0x94 =
4901+
{ "Wacom Bootloader", .type = BOOTLOADER };
4902+
49004903
#define USB_DEVICE_WACOM(prod) \
49014904
HID_DEVICE(BUS_USB, HID_GROUP_WACOM, USB_VENDOR_ID_WACOM, prod),\
49024905
.driver_data = (kernel_ulong_t)&wacom_features_##prod
@@ -4970,6 +4973,7 @@ const struct hid_device_id wacom_ids[] = {
49704973
{ USB_DEVICE_WACOM(0x84) },
49714974
{ USB_DEVICE_WACOM(0x90) },
49724975
{ USB_DEVICE_WACOM(0x93) },
4976+
{ USB_DEVICE_WACOM(0x94) },
49734977
{ USB_DEVICE_WACOM(0x97) },
49744978
{ USB_DEVICE_WACOM(0x9A) },
49754979
{ USB_DEVICE_WACOM(0x9F) },

4.5/wacom_wac.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -267,6 +267,7 @@ enum {
267267
MTTPC,
268268
MTTPC_B,
269269
HID_GENERIC,
270+
BOOTLOADER,
270271
MAX_TYPE
271272
};
272273

0 commit comments

Comments
 (0)