[PATCH v3] HID: autoload hid-multitouch as needed

From: benjamin.tissoires
Date: Wed Mar 28 2012 - 10:33:24 EST

From: Benjamin Tissoires <benjamin.tissoires@xxxxxxx>

When the generic hid parsing of the report descriptors in hid-core
detects that the device contains the field ContactID, the device should
be handled by hid-multitouch, and hid-core should release it.
This patch implements a temporary fix for hid-core to automatically
call the loading of hid-multitouch. A better solution would involve

Signed-off-by: Benjamin Tissoires <benjamin.tissoires@xxxxxxx>

Hi guys,

This is the v3 of the patch.
Changes since v2:
- make the working queue global to the module. As noted by Henrik,
there is no need to have a per-device working queue.
- removed the boolean that tells if the working queue has been initialized
and filled: we have only one wq that is initialized at the loading of the
module. I didn't put any threading guards: as far as I understood, working
queues are thread safe.

There is still one problem: this works for hotplugged devices. Coldplugged
ones (before booting then) are not loading hid-multitouch on my computer.
I don't think the previous implementation worked either. However, I don't
think it's a big issue as we may ask end-user to add hid-multitouch in this
case in their /etc/modules-load.d/.


drivers/hid/hid-core.c | 40 +++++++++++++++++++++++++++++++++++++++-
1 files changed, 39 insertions(+), 1 deletions(-)

diff --git a/drivers/hid/hid-core.c b/drivers/hid/hid-core.c
index 990fe19..f694aa1 100644
--- a/drivers/hid/hid-core.c
+++ b/drivers/hid/hid-core.c
@@ -1210,6 +1210,39 @@ static struct bin_attribute dev_bin_attr_report_desc = {

+#if defined(CONFIG_MODULES) && defined(MODULE)
+/* This is a temporary fix to make hid-multitouch loadable from
+ * the kernel before we come up with more robust attitude
+ * (with userspace involvement).
+ * In case we detect a multitouch device through the parsing of
+ * hid-core, we request hid-multitouch to be loaded. */
+static struct work_struct request_hid_mt_module_wk;
+static void hid_request_hid_mt_module_async(struct work_struct *work)
+ request_module("hid-multitouch");
+static inline void hid_request_hid_mt_module(void)
+ schedule_work(&request_hid_mt_module_wk);
+static inline void hid_flush_request_modules(void)
+ flush_work_sync(&request_hid_mt_module_wk);
+static inline void hid_init_request_modules(void)
+ INIT_WORK(&request_hid_mt_module_wk, hid_request_hid_mt_module_async);
+#define hid_request_hid_mt_module()
+#define hid_flush_request_modules()
+#define hid_init_request_modules()
+#endif /* CONFIG_MODULES */
int hid_connect(struct hid_device *hdev, unsigned int connect_mask)
static const char *types[] = { "Device", "Pointer", "Mouse", "Device",
@@ -1235,7 +1268,9 @@ int hid_connect(struct hid_device *hdev, unsigned int connect_mask)
hdev->claimed |= HID_CLAIMED_INPUT;
if (hdev->quirks & HID_QUIRK_MULTITOUCH) {
- /* this device should be handled by hid-multitouch, skip it */
+ /* this device should be handled by hid-multitouch, request
+ * for hid-multitouch to be loaded and leave the device to it */
+ hid_request_hid_mt_module();
return -ENODEV;

@@ -2210,6 +2245,8 @@ static int __init hid_init(void)


+ hid_init_request_modules();
return 0;
@@ -2219,6 +2256,7 @@ err:

static void __exit hid_exit(void)
+ hid_flush_request_modules();

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/