From 494078b0bb578c4cf1e00275dd3224d793013488 Mon Sep 17 00:00:00 2001 From: Benjamin Tissoires Date: Tue, 23 Sep 2014 12:08:07 -0400 Subject: [PATCH] HID: wacom: move allocation of inputs earlier This allows to have the input devices ready in while parsing the reports descriptor. Signed-off-by: Benjamin Tissoires Acked-by: Jason Gerecke Signed-off-by: Jiri Kosina --- drivers/hid/wacom_sys.c | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/drivers/hid/wacom_sys.c b/drivers/hid/wacom_sys.c index 62f3c899ab98..21ac2baa21be 100644 --- a/drivers/hid/wacom_sys.c +++ b/drivers/hid/wacom_sys.c @@ -1357,6 +1357,12 @@ static int wacom_probe(struct hid_device *hdev, mutex_init(&wacom->lock); INIT_WORK(&wacom->work, wacom_wireless_work); + if (!(features->quirks & WACOM_QUIRK_NO_INPUT)) { + error = wacom_allocate_inputs(wacom); + if (error) + goto fail_allocate_inputs; + } + /* set the default size in case we do not get them from hid */ wacom_set_default_phy(features); @@ -1429,8 +1435,7 @@ static int wacom_probe(struct hid_device *hdev, } if (!(features->quirks & WACOM_QUIRK_NO_INPUT)) { - error = wacom_allocate_inputs(wacom) || - wacom_register_inputs(wacom); + error = wacom_register_inputs(wacom); if (error) goto fail_register_inputs; } @@ -1464,7 +1469,6 @@ static int wacom_probe(struct hid_device *hdev, return 0; fail_hw_start: - wacom_clean_inputs(wacom); if (hdev->bus == BUS_BLUETOOTH) device_remove_file(&hdev->dev, &dev_attr_speed); fail_register_inputs: @@ -1473,6 +1477,8 @@ fail_register_inputs: fail_battery: wacom_remove_shared_data(wacom_wac); fail_shared_data: + wacom_clean_inputs(wacom); +fail_allocate_inputs: fail_type: fail_pktlen: fail_parse: -- 2.20.1