HID: wacom: move allocation of inputs earlier
authorBenjamin Tissoires <benjamin.tissoires@redhat.com>
Tue, 23 Sep 2014 16:08:07 +0000 (12:08 -0400)
committerJiri Kosina <jkosina@suse.cz>
Wed, 1 Oct 2014 07:11:22 +0000 (09:11 +0200)
This allows to have the input devices ready in while parsing the reports
descriptor.

Signed-off-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
Acked-by: Jason Gerecke <killertofu@gmail.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
drivers/hid/wacom_sys.c

index 62f3c899ab9804a0684cbc6e0d7f741ca2374c1b..21ac2baa21be0b12ee8e30161b82132df283c137 100644 (file)
@@ -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: