From nobody Fri Oct 3 13:23:47 2025 Received: from hognose1.porkbun.com (hognose1.porkbun.com [35.82.102.206]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id CBA0016A395; Sat, 30 Aug 2025 05:29:27 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=35.82.102.206 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756531769; cv=none; b=ZyG0ZRlbMEVs3MKTIqSeMztURir85luhFjx0DdjKBCYN6tdKjdWKZYXn3gTX/Awqb1VjaRDDk4cI/PA8e5bEXWgYSa7gAgQF+RHVV6BwG5ofHzEVVM2mTXJ4JMvpBo/8c0XsPhBWAgt0psInNm/kevFQVRAXnjmGaXUL7z0SlqY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756531769; c=relaxed/simple; bh=Q5daGjJ/YXbXYY8U7XevbYytlvZ8JW6mh9q6e73x/kc=; h=Date:From:To:Cc:Subject:Message-ID:MIME-Version:Content-Type: Content-Disposition; b=RJQF/HyB9xgRwaD+O1HxEql5xPUSinjuUONbfZuVKIOcvBdekwcaadPYpst9R4HeuGNvXdkXpCCHB51wx0WTK+0pZZZyrZBWsBxxaqleui1Br+b9V6wPxR6PPtHwU9gJbEFTv4h7469st7ulh2ACKCBv+Q+pSOY6ptACH4keCQg= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=altimeter.info; spf=pass smtp.mailfrom=altimeter.info; dkim=pass (1024-bit key) header.d=altimeter.info header.i=@altimeter.info header.b=ogPzQXJw; arc=none smtp.client-ip=35.82.102.206 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=altimeter.info Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=altimeter.info Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=altimeter.info header.i=@altimeter.info header.b="ogPzQXJw" Received: from altimeter-info (unknown [45.55.225.183]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) (Authenticated sender: linux-kernel@altimeter.info) by hognose1.porkbun.com (Postfix) with ESMTPSA id 9CAD54537E; Sat, 30 Aug 2025 05:21:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=altimeter.info; s=default; t=1756531267; bh=HnV5VLDf2K/1ocXH/EITt37CP9a3TzSH0gmHqKu3CYg=; h=Date:From:To:Cc:Subject; b=ogPzQXJwqLlZ5CJyMwL2QqHEvdlq3yOwgW7C11lqVNyFb3pVt7DL3wnJPL2xuOzDQ dY4qoqb63rFT3NmYRC1PsuRhBhoQp/NdJqRGkPSKg204SRGK1jXmnD/J2ZG4ZrlKvp zDDqZQPoQYVsn78QLQHugYaLjoBZOnbqSsReNEF8= Date: Sat, 30 Aug 2025 05:21:02 +0000 From: Ivan Gorinov To: Jiri Kosina Cc: linux-input@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] Input: Improve WinWing Orion2 throttle support Message-ID: <20250830052102.GA28400@altimeter-info> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Disposition: inline User-Agent: Mutt/1.5.24 (2015-08-30) Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Add support for Orion2 throttle configurations with more than 32 button cod= es on the grip handle (this means the device reports more than 80 button codes= ). Map additional button codes to KEY_MACRO1 .. KEY_MACRO28. Make the module simpler, removing report descriptor fixup. Signed-off-by: Ivan Gorinov --- drivers/hid/Kconfig | 2 + drivers/hid/hid-winwing.c | 167 +++++++++++++++++++++++--------------- 2 files changed, 104 insertions(+), 65 deletions(-) diff --git a/drivers/hid/Kconfig b/drivers/hid/Kconfig index a57901203aeb..3317981e65dc 100644 --- a/drivers/hid/Kconfig +++ b/drivers/hid/Kconfig @@ -1309,6 +1309,8 @@ config HID_WINWING help Support for WinWing Orion2 throttle base with the following grips: =20 + * TGRIP-15E + * TGRIP-15EX * TGRIP-16EX * TGRIP-18 =20 diff --git a/drivers/hid/hid-winwing.c b/drivers/hid/hid-winwing.c index d4afbbd27807..54e0f43163a8 100644 --- a/drivers/hid/hid-winwing.c +++ b/drivers/hid/hid-winwing.c @@ -38,6 +38,7 @@ struct winwing_drv_data { __u8 *report_buf; struct mutex lock; unsigned int num_leds; + int too_many_buttons; struct winwing_led leds[]; }; =20 @@ -81,12 +82,10 @@ static int winwing_init_led(struct hid_device *hdev, int ret; int i; =20 - size_t data_size =3D struct_size(data, leds, 3); - - data =3D devm_kzalloc(&hdev->dev, data_size, GFP_KERNEL); + data =3D hid_get_drvdata(hdev); =20 if (!data) - return -ENOMEM; + return -EINVAL; =20 data->report_buf =3D devm_kmalloc(&hdev->dev, MAX_REPORT, GFP_KERNEL); =20 @@ -106,6 +105,7 @@ static int winwing_init_led(struct hid_device *hdev, "%s::%s", dev_name(&input->dev), info->led_name); + if (!led->cdev.name) return -ENOMEM; =20 @@ -114,14 +114,96 @@ static int winwing_init_led(struct hid_device *hdev, return ret; } =20 - hid_set_drvdata(hdev, data); - return ret; } =20 +static int winwing_map_button(int button, int too_many_buttons) +{ + if (button < 1) + return KEY_RESERVED; + + if (button > 112) + return KEY_RESERVED; + + if (button <=3D 16) { + /* + * Grip buttons [1 .. 16] are mapped to + * key codes BTN_TRIGGER .. BTN_DEAD + */ + return (button - 1) + BTN_JOYSTICK; + } + + if (button >=3D 65) { + /* + * Base buttons [65 .. 112] are mapped to + * key codes BTN_TRIGGER_HAPPY17 .. KEY_MAX + */ + return (button - 65) + BTN_TRIGGER_HAPPY17; + } + + if (too_many_buttons) { + if (button <=3D 48) { + /* + * Grip buttons [17 .. 44] are mapped to + * KEY_MACRO1 .. KEY_MACRO28; + * button numbers [45 .. 48] are not used. + */ + return (button - 17) + KEY_MACRO1; + } else { + /* + * Grip buttons [49 .. 64] are mapped to + * BTN_TRIGGER_HAPPY1 .. BTN_TRIGGER_HAPPY16 + */ + return (button - 49) + BTN_TRIGGER_HAPPY1; + } + } else { + if (button <=3D 32) { + /* + * Grip buttons [17 .. 32] are mapped to + * BTN_TRIGGER_HAPPY1 .. BTN_TRIGGER_HAPPY16 + */ + return (button - 17) + BTN_TRIGGER_HAPPY1; + } + /* Not mapping button numbers [33 .. 64] */ + } + + return KEY_RESERVED; +} + +static int winwing_input_mapping(struct hid_device *hdev, + struct hid_input *hi, struct hid_field *field, struct hid_usage *usage, + unsigned long **bit, int *max) +{ + struct winwing_drv_data *data; + int code =3D KEY_RESERVED; + int button =3D 0; + + data =3D hid_get_drvdata(hdev); + + if (!data) + return -EINVAL; + + if ((usage->hid & HID_USAGE_PAGE) !=3D HID_UP_BUTTON) + return 0; + + if (field->application !=3D HID_GD_JOYSTICK) + return 0; + + /* Button numbers start with 1 */ + button =3D usage->hid & HID_USAGE; + + code =3D winwing_map_button(button, data->too_many_buttons); + + hid_map_usage(hi, usage, bit, max, EV_KEY, code); + + return 1; +} + static int winwing_probe(struct hid_device *hdev, const struct hid_device_id *id) { + struct winwing_drv_data *data; + size_t data_size =3D struct_size(data, leds, 3); int ret; =20 ret =3D hid_parse(hdev); @@ -130,6 +212,15 @@ static int winwing_probe(struct hid_device *hdev, return ret; } =20 + data =3D devm_kzalloc(&hdev->dev, data_size, GFP_KERNEL); + + if (!data) + return -ENOMEM; + + data->too_many_buttons =3D id->driver_data; + + hid_set_drvdata(hdev, data); + ret =3D hid_hw_start(hdev, HID_CONNECT_DEFAULT); if (ret) { hid_err(hdev, "hw start failed\n"); @@ -152,64 +243,11 @@ static int winwing_input_configured(struct hid_device= *hdev, return ret; } =20 -static const __u8 original_rdesc_buttons[] =3D { - 0x05, 0x09, 0x19, 0x01, 0x29, 0x6F, - 0x15, 0x00, 0x25, 0x01, 0x35, 0x00, - 0x45, 0x01, 0x75, 0x01, 0x95, 0x6F, - 0x81, 0x02, 0x75, 0x01, 0x95, 0x01, - 0x81, 0x01 -}; - -/* - * HID report descriptor shows 111 buttons, which exceeds maximum - * number of buttons (80) supported by Linux kernel HID subsystem. - * - * This module skips numbers 32-63, unused on some throttle grips. - */ - -static const __u8 *winwing_report_fixup(struct hid_device *hdev, __u8 *rde= sc, - unsigned int *rsize) -{ - int sig_length =3D sizeof(original_rdesc_buttons); - int unused_button_numbers =3D 32; - - if (*rsize < 34) - return rdesc; - - if (memcmp(rdesc + 8, original_rdesc_buttons, sig_length) =3D=3D 0) { - - /* Usage Maximum */ - rdesc[13] -=3D unused_button_numbers; - - /* Report Count for buttons */ - rdesc[25] -=3D unused_button_numbers; - - /* Report Count for padding [HID1_11, 6.2.2.9] */ - rdesc[31] +=3D unused_button_numbers; - - hid_info(hdev, "winwing descriptor fixed\n"); - } - - return rdesc; -} - -static int winwing_raw_event(struct hid_device *hdev, - struct hid_report *report, u8 *raw_data, int size) -{ - if (size >=3D 15) { - /* Skip buttons 32 .. 63 */ - memmove(raw_data + 5, raw_data + 9, 6); - - /* Clear the padding */ - memset(raw_data + 11, 0, 4); - } - - return 0; -} - static const struct hid_device_id winwing_devices[] =3D { - { HID_USB_DEVICE(0x4098, 0xbe62) }, /* TGRIP-18 */ - { HID_USB_DEVICE(0x4098, 0xbe68) }, /* TGRIP-16EX */ + { HID_USB_DEVICE(0x4098, 0xbd65), .driver_data =3D 1 }, /* TGRIP-15E */ + { HID_USB_DEVICE(0x4098, 0xbd64), .driver_data =3D 1 }, /* TGRIP-15EX */ + { HID_USB_DEVICE(0x4098, 0xbe68), .driver_data =3D 0 }, /* TGRIP-16EX */ + { HID_USB_DEVICE(0x4098, 0xbe62), .driver_data =3D 0 }, /* TGRIP-18 */ {} }; =20 @@ -218,10 +256,9 @@ MODULE_DEVICE_TABLE(hid, winwing_devices); static struct hid_driver winwing_driver =3D { .name =3D "winwing", .id_table =3D winwing_devices, + .input_mapping =3D winwing_input_mapping, .probe =3D winwing_probe, .input_configured =3D winwing_input_configured, - .report_fixup =3D winwing_report_fixup, - .raw_event =3D winwing_raw_event, }; module_hid_driver(winwing_driver); =20 --=20 2.34.1