HID: use hid_hw_wait() instead of direct call to usbhid
authorBenjamin Tissoires <benjamin.tissoires@gmail.com>
Mon, 25 Feb 2013 10:31:47 +0000 (11:31 +0100)
committerJiri Kosina <jkosina@suse.cz>
Mon, 25 Feb 2013 12:26:41 +0000 (13:26 +0100)
This removes most of the dependencies between hid drivers and usbhid.

The patch was constructed by replacing all occurences of
usbhid_wait_io() by its hid_hw_wait() counterpart.
Then, drivers not requiring USB_HID anymore have their USB_HID
dependency cleaned in the Kconfig file.

As of today, few drivers are still requiring an explicit USB layer
dependency:
* ntrig (a patch is on its way)
* multitouch (one patch following and another on its way)
* lenovo tpkbd
* roccat
* sony

The last three are two deeply using direct calls to the usb subsystem
to be able to be cleaned right now.

Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
Reviewed-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
drivers/hid/Kconfig
drivers/hid/hid-ntrig.c
drivers/hid/hid-picolcd_fb.c
drivers/hid/hid-sensor-hub.c
drivers/hid/usbhid/hid-core.c
drivers/hid/usbhid/hid-pidff.c
drivers/hid/usbhid/hiddev.c
drivers/hid/usbhid/usbhid.h

index f0acf2754b692219c2e01101beb4d6b8975f5d40..c802b0716f48c63fa023034a5e6a7c0e89922f10 100644 (file)
@@ -479,7 +479,7 @@ config HID_PETALYNX
 
 config HID_PICOLCD
        tristate "PicoLCD (graphic version)"
-       depends on USB_HID
+       depends on HID
        ---help---
          This provides support for Minibox PicoLCD devices, currently
          only the graphical ones are supported.
index b926592e6c16da7c7a02c963192e6faac52c2214..ef95102515e4d499143dd02a831481902d674b1e 100644 (file)
@@ -119,7 +119,7 @@ static inline int ntrig_get_mode(struct hid_device *hdev)
                return -EINVAL;
 
        hid_hw_request(hdev, report, HID_REQ_GET_REPORT);
-       usbhid_wait_io(hdev);
+       hid_hw_wait(hdev);
        return (int)report->field[0]->value[0];
 }
 
@@ -937,7 +937,7 @@ static int ntrig_probe(struct hid_device *hdev, const struct hid_device_id *id)
        if (report) {
                /* Let the device settle to ensure the wakeup message gets
                 * through */
-               usbhid_wait_io(hdev);
+               hid_hw_wait(hdev);
                hid_hw_request(hdev, report, HID_REQ_GET_REPORT);
 
                /*
index 98f61de9a5e070a87d2d9fd5234f30fa9392ef4f..591f6b22aa947190a9e728b650caa40090cc9595 100644 (file)
@@ -19,7 +19,6 @@
 
 #include <linux/hid.h>
 #include <linux/vmalloc.h>
-#include "usbhid/usbhid.h"
 
 #include <linux/fb.h>
 #include <linux/module.h>
@@ -269,7 +268,7 @@ static void picolcd_fb_update(struct fb_info *info)
                                mutex_unlock(&info->lock);
                                if (!data)
                                        return;
-                               usbhid_wait_io(data->hdev);
+                               hid_hw_wait(data->hdev);
                                mutex_lock(&info->lock);
                                n = 0;
                        }
@@ -287,7 +286,7 @@ static void picolcd_fb_update(struct fb_info *info)
                spin_unlock_irqrestore(&fbdata->lock, flags);
                mutex_unlock(&info->lock);
                if (data)
-                       usbhid_wait_io(data->hdev);
+                       hid_hw_wait(data->hdev);
                return;
        }
 out:
index 59d29f853ac0a8a319a42bc40b28448651f8a25e..ca74981073271effdddc72ba1f173acea27ebbf9 100644 (file)
@@ -18,7 +18,6 @@
  */
 #include <linux/device.h>
 #include <linux/hid.h>
-#include "usbhid/usbhid.h"
 #include <linux/module.h>
 #include <linux/slab.h>
 #include <linux/mfd/core.h>
@@ -204,7 +203,7 @@ int sensor_hub_set_feature(struct hid_sensor_hub_device *hsdev, u32 report_id,
        }
        hid_set_field(report->field[field_index], 0, value);
        hid_hw_request(hsdev->hdev, report, HID_REQ_SET_REPORT);
-       usbhid_wait_io(hsdev->hdev);
+       hid_hw_wait(hsdev->hdev);
 
 done_proc:
        mutex_unlock(&data->mutex);
@@ -227,7 +226,7 @@ int sensor_hub_get_feature(struct hid_sensor_hub_device *hsdev, u32 report_id,
                goto done_proc;
        }
        hid_hw_request(hsdev->hdev, report, HID_REQ_GET_REPORT);
-       usbhid_wait_io(hsdev->hdev);
+       hid_hw_wait(hsdev->hdev);
        *value = report->field[field_index]->value[0];
 
 done_proc:
index da68687d2c7c39bb501fa8f42da6c5f8c836690d..420466bc481abb508f1f4a7b1167af206672ab32 100644 (file)
@@ -705,7 +705,7 @@ static int usb_hidinput_input_event(struct input_dev *dev, unsigned int type, un
        return 0;
 }
 
-int usbhid_wait_io(struct hid_device *hid)
+static int usbhid_wait_io(struct hid_device *hid)
 {
        struct usbhid_device *usbhid = hid->driver_data;
 
@@ -719,7 +719,6 @@ int usbhid_wait_io(struct hid_device *hid)
 
        return 0;
 }
-EXPORT_SYMBOL_GPL(usbhid_wait_io);
 
 static int hid_set_idle(struct usb_device *dev, int ifnum, int report, int idle)
 {
index 0f1efa39ec46195d2cf22db2af32d16bc1cc1238..10b6167027802abbc289466c00a74b6281e2b5f3 100644 (file)
@@ -471,13 +471,13 @@ static int pidff_request_effect_upload(struct pidff_device *pidff, int efnum)
 
        pidff->block_load[PID_EFFECT_BLOCK_INDEX].value[0] = 0;
        pidff->block_load_status->value[0] = 0;
-       usbhid_wait_io(pidff->hid);
+       hid_hw_wait(pidff->hid);
 
        for (j = 0; j < 60; j++) {
                hid_dbg(pidff->hid, "pid_block_load requested\n");
                hid_hw_request(pidff->hid, pidff->reports[PID_BLOCK_LOAD],
                                HID_REQ_GET_REPORT);
-               usbhid_wait_io(pidff->hid);
+               hid_hw_wait(pidff->hid);
                if (pidff->block_load_status->value[0] ==
                    pidff->status_id[PID_BLOCK_LOAD_SUCCESS]) {
                        hid_dbg(pidff->hid, "device reported free memory: %d bytes\n",
@@ -551,7 +551,7 @@ static int pidff_erase_effect(struct input_dev *dev, int effect_id)
                effect_id, pidff->pid_id[effect_id]);
        /* Wait for the queue to clear. We do not want a full fifo to
           prevent the effect removal. */
-       usbhid_wait_io(pidff->hid);
+       hid_hw_wait(pidff->hid);
        pidff_playback_pid(pidff, pid_id, 0);
        pidff_erase_pid(pidff, pid_id);
 
@@ -1159,18 +1159,18 @@ static void pidff_reset(struct pidff_device *pidff)
        pidff->device_control->value[0] = pidff->control_id[PID_RESET];
        /* We reset twice as sometimes hid_wait_io isn't waiting long enough */
        hid_hw_request(hid, pidff->reports[PID_DEVICE_CONTROL], HID_REQ_SET_REPORT);
-       usbhid_wait_io(hid);
+       hid_hw_wait(hid);
        hid_hw_request(hid, pidff->reports[PID_DEVICE_CONTROL], HID_REQ_SET_REPORT);
-       usbhid_wait_io(hid);
+       hid_hw_wait(hid);
 
        pidff->device_control->value[0] =
                pidff->control_id[PID_ENABLE_ACTUATORS];
        hid_hw_request(hid, pidff->reports[PID_DEVICE_CONTROL], HID_REQ_SET_REPORT);
-       usbhid_wait_io(hid);
+       hid_hw_wait(hid);
 
        /* pool report is sometimes messed up, refetch it */
        hid_hw_request(hid, pidff->reports[PID_POOL], HID_REQ_GET_REPORT);
-       usbhid_wait_io(hid);
+       hid_hw_wait(hid);
 
        if (pidff->pool[PID_SIMULTANEOUS_MAX].value) {
                while (pidff->pool[PID_SIMULTANEOUS_MAX].value[0] < 2) {
@@ -1183,7 +1183,7 @@ static void pidff_reset(struct pidff_device *pidff)
                        hid_dbg(pidff->hid, "pid_pool requested again\n");
                        hid_hw_request(hid, pidff->reports[PID_POOL],
                                          HID_REQ_GET_REPORT);
-                       usbhid_wait_io(hid);
+                       hid_hw_wait(hid);
                }
        }
 }
index 430d2a9e4521f897aaf4c0e7c19771cd5f444dc8..2f1ddca6f2e0a3a4712bce434d5b29e1556e216c 100644 (file)
@@ -706,7 +706,7 @@ static long hiddev_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
                        break;
 
                hid_hw_request(hid, report, HID_REQ_GET_REPORT);
-               usbhid_wait_io(hid);
+               hid_hw_wait(hid);
 
                r = 0;
                break;
@@ -725,7 +725,7 @@ static long hiddev_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
                        break;
 
                hid_hw_request(hid, report, HID_REQ_SET_REPORT);
-               usbhid_wait_io(hid);
+               hid_hw_wait(hid);
 
                r = 0;
                break;
index aa1d5ff1208fa4cde5d34d0e9a5318992f87de18..dbb6af6991350d4e09b2c7ca484411065afdc75c 100644 (file)
@@ -34,7 +34,6 @@
 #include <linux/input.h>
 
 /*  API provided by hid-core.c for USB HID drivers */
-int usbhid_wait_io(struct hid_device* hid);
 void usbhid_close(struct hid_device *hid);
 int usbhid_open(struct hid_device *hid);
 void usbhid_init_reports(struct hid_device *hid);