diff options
| author |   Benjamin Tissoires <benjamin.tissoires@redhat.com> | 2018-04-24 10:04:33 +0200 |
|---|---|---|
| committer |   Jiri Kosina <jkosina@suse.cz> | 2018-04-26 14:17:31 +0200 |
| commit | f07b3c1da92db108662f99417a212fc1eddc44d1 (patch) | |
| tree | d4ad0e580dfab744d34460c80a361e78c7ff47dd /drivers/hid/hid-gfrm.c | |
| parent | HID: store the full list of reports in the hidinput (diff) | |
| download | linux-dev-f07b3c1da92db108662f99417a212fc1eddc44d1.tar.xz linux-dev-f07b3c1da92db108662f99417a212fc1eddc44d1.zip | |
HID: generic: create one input report per application type
It is not a good idea to try to fit all types of applications in the
same input report. There are a lot of devices that are needing
the quirk HID_MULTI_INPUT but this quirk doesn't match the actual HID
description as it is based on the report ID.
Given that most devices with MULTI_INPUT I can think of split nicely
the devices inputs into application, it is a good thing to split the
devices by default based on this assumption.
Also make hid-multitouch following this rule, to not have to deal
with too many input created.
While we are at it, fix some checkpatch complaints about converting
'unsigned' to 'unsigned int'.
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Diffstat (limited to 'drivers/hid/hid-gfrm.c')
| -rw-r--r-- | drivers/hid/hid-gfrm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/hid/hid-gfrm.c b/drivers/hid/hid-gfrm.c index 075b1c020846..cf477f8c8f4c 100644 --- a/drivers/hid/hid-gfrm.c +++ b/drivers/hid/hid-gfrm.c @@ -116,7 +116,7 @@ static int gfrm_probe(struct hid_device *hdev, const struct hid_device_id *id) * those reports reach gfrm_raw_event() from hid_input_report(). */ if (!hid_register_report(hdev, HID_INPUT_REPORT, - GFRM100_SEARCH_KEY_REPORT_ID)) { + GFRM100_SEARCH_KEY_REPORT_ID, 0)) { ret = -ENOMEM; goto done; } |