diff options
author | Jiri Kosina <jkosina@suse.cz> | 2010-01-04 12:20:56 +0100 |
---|---|---|
committer | Jiri Kosina <jkosina@suse.cz> | 2010-01-04 12:20:56 +0100 |
commit | cf2f765f1896064e34c6f0f2ef896ff058dd5c06 (patch) | |
tree | 5bb51af0bfcd5f72a740dee0c8b97ae358643d05 /drivers/hid/hid-input.c | |
parent | HID: make Stantum driver standalone config option (diff) | |
download | linux-cf2f765f1896064e34c6f0f2ef896ff058dd5c06.tar.xz linux-cf2f765f1896064e34c6f0f2ef896ff058dd5c06.zip |
HID: handle joysticks with large number of buttons
Current HID code doesn't properly handle HID joysticks which have
larger number of buttons than what fits into current range reserved
for BTN_JOYSTICK.
One such joystick reported to not work properly is Saitek X52 Pro
Flight System.
We can't extend the range to fit more buttons in, because of backwards
compatibility reasons.
Therefore this patch introduces a new BTN_TRIGGER_HAPPY range, and
uses these to map the buttons which are over BTN_JOYSTICK limit.
Acked-by: Dmitry Torokhov <dmitry.torokhov@gmail.com> [for the input.h part]
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Diffstat (limited to 'drivers/hid/hid-input.c')
-rw-r--r-- | drivers/hid/hid-input.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/drivers/hid/hid-input.c b/drivers/hid/hid-input.c index 5862b0f3b55d..dad7aae9c975 100644 --- a/drivers/hid/hid-input.c +++ b/drivers/hid/hid-input.c @@ -198,7 +198,12 @@ static void hidinput_configure_usage(struct hid_input *hidinput, struct hid_fiel switch (field->application) { case HID_GD_MOUSE: case HID_GD_POINTER: code += 0x110; break; - case HID_GD_JOYSTICK: code += 0x120; break; + case HID_GD_JOYSTICK: + if (code <= 0xf) + code += BTN_JOYSTICK; + else + code += BTN_TRIGGER_HAPPY; + break; case HID_GD_GAMEPAD: code += 0x130; break; default: switch (field->physical) { |