HID: use hid_hw_request() instead of direct call to usbhid

This allows the hid drivers to be independent from the transport layer.

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

Finally, few drivers still depends on USB_HID. Many of them
are requiring the io wait callback. They are found in the next patch.

Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
Reviewed-by: Mika Westerberg <mika.westerberg@linux.intel.com>

For the sensor-hub part:
Tested-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>

1 files changed, 3 insertions, 4 deletions

diff --git a/drivers/hid/hid-sensor-hub.c b/drivers/hid/hid-sensor-hub.c
index 6679788bf75a..59d29f853ac0 100644
--- a/drivers/hid/hid-sensor-hub.c
+++ b/drivers/hid/hid-sensor-hub.c
@@ -18,7 +18,6 @@
  */
 #include <linux/device.h>
 #include <linux/hid.h>
-#include <linux/usb.h>
 #include "usbhid/usbhid.h"
 #include <linux/module.h>
 #include <linux/slab.h>
@@ -204,7 +203,7 @@ int sensor_hub_set_feature(struct hid_sensor_hub_device *hsdev, u32 report_id,
                 goto done_proc;
         }
         hid_set_field(report->field[field_index], 0, value);
-        usbhid_submit_report(hsdev->hdev, report, USB_DIR_OUT);
+        hid_hw_request(hsdev->hdev, report, HID_REQ_SET_REPORT);
         usbhid_wait_io(hsdev->hdev);
 
 done_proc:
@@ -227,7 +226,7 @@ int sensor_hub_get_feature(struct hid_sensor_hub_device *hsdev, u32 report_id,
                 ret = -EINVAL;
                 goto done_proc;
         }
-        usbhid_submit_report(hsdev->hdev, report, USB_DIR_IN);
+        hid_hw_request(hsdev->hdev, report, HID_REQ_GET_REPORT);
         usbhid_wait_io(hsdev->hdev);
         *value = report->field[field_index]->value[0];
 
@@ -262,7 +261,7 @@ int sensor_hub_input_attr_get_raw_value(struct hid_sensor_hub_device *hsdev,
                 spin_unlock_irqrestore(&data->lock, flags);
                 goto err_free;
         }
-        usbhid_submit_report(hsdev->hdev, report, USB_DIR_IN);
+        hid_hw_request(hsdev->hdev, report, HID_REQ_GET_REPORT);
         spin_unlock_irqrestore(&data->lock, flags);
         wait_for_completion_interruptible_timeout(&data->pending.ready, HZ*5);
         switch (data->pending.raw_size) {