From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from endrift.com (endrift.com [173.255.198.10]) (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 8393B31985C for ; Tue, 8 Sep 2026 03:23:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=173.255.198.10 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788837827; cv=none; b=diHa27TwWNVXCQ6IT9NE2G7q75Sn5Es43LfLtlfUjTzU6l5pJnIZjmgIrqnHhY3tBt0T2qveFk8rMJYg/VmBC7zPmaeHB+x+Bf3TO2A0NNDZCLrW5ZtU2ln0UruqMhP8zUWdQDyu9lhgB1G09qBIiYP23U5TPeM1FcCCtMfe+7k= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788837827; c=relaxed/simple; bh=qv7TfMnKzIap3dcxVf+ZsI5YRKwN/hJpbbuXrJ9iSl8=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=IFElrd4bppbuYDYlEhwhwAGDFgg7WAib/heAGO+MaXIkVxVSsX49mfk5+4xbO5U3pZjUM9hmGvTB+FGd4bFnTgiRYgDQN1IYG93qi5j/LW1kOzx96qGC4m/0pM4UpO7CEzM77MlybBMCxFo1ckyXe9RpbqyQqyzAq7q552lxtaM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=endrift.com; spf=pass smtp.mailfrom=endrift.com; dkim=pass (2048-bit key) header.d=endrift.com header.i=@endrift.com header.b=bEKBPQcW; arc=none smtp.client-ip=173.255.198.10 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=endrift.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=endrift.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=endrift.com header.i=@endrift.com header.b="bEKBPQcW" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=endrift.com; s=2020; t=1788837818; bh=qv7TfMnKzIap3dcxVf+ZsI5YRKwN/hJpbbuXrJ9iSl8=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=bEKBPQcWlEZ9uENsw6UvrK2AI3q4E8cv+c92v3EzyWNoIwPgDZOJlv1nBC/w2LQs3 xIwEbZXdhnVm3T3Gklxq2X3LdSJWRXdbyQMc4FzGykwgCBANHW56GhF9dZJQ1wd+K+ wpGd88PHjcLil1PSkHBG69WI5ZexrP2I+TcWil/elUBAna7MGt2N15CEZHtn8qFLIa uVG/CStn2nIXlHP+ZxtoOzeCERxeVX5g7b3Y53LDEuyiQzWmdp5AgBlYvhYwjRDGkM xRsiNsNsZZUQbzD2CQjgWwwC1h00M65h9p0jcPuwht6YaUPHz1Q/8BQQ/R1smJzfHM M2WAFQN6gDygg== Received: from microtis.vulpes.eutheria.net (71-212-73-87.tukw.qwest.net [71.212.73.87]) by endrift.com (Postfix) with ESMTPSA id 4CFB113208A; Mon, 07 Sep 2026 20:23:38 -0700 (PDT) From: Vicki Pfau To: Dmitry Torokhov , linux-input@vger.kernel.org Cc: Vicki Pfau Subject: [PATCH v6 08/12] Input: xbox_gip - Add arcade stick support Date: Mon, 7 Sep 2026 20:21:39 -0700 Message-ID: <20260908032145.2118234-9-vi@endrift.com> X-Mailer: git-send-email 2.54.0 In-Reply-To: <20260908032145.2118234-1-vi@endrift.com> References: <20260908032145.2118234-1-vi@endrift.com> Precedence: bulk X-Mailing-List: linux-input@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit This adds support for the arcade stick device type. Currently I'm only aware of one Xbox controller that uses this device type, the Razer Atrox, so testing coverage is limited. Signed-off-by: Vicki Pfau --- drivers/input/joystick/gip/Makefile | 2 + drivers/input/joystick/gip/gip-arcade-stick.c | 173 ++++++++++++++++++ drivers/input/joystick/gip/gip-core.c | 6 +- drivers/input/joystick/gip/gip.h | 4 +- 4 files changed, 181 insertions(+), 4 deletions(-) create mode 100644 drivers/input/joystick/gip/gip-arcade-stick.c diff --git a/drivers/input/joystick/gip/Makefile b/drivers/input/joystick/gip/Makefile index a5ad301a9063..9ed59caec4d9 100644 --- a/drivers/input/joystick/gip/Makefile +++ b/drivers/input/joystick/gip/Makefile @@ -2,3 +2,5 @@ obj-$(CONFIG_JOYSTICK_XBOX_GIP) += xbox-gip.o # Base driver xbox-gip-y := gip-core.o gip-drivers.o gip-security.o +# Additional device support +xbox-gip-y += gip-arcade-stick.o diff --git a/drivers/input/joystick/gip/gip-arcade-stick.c b/drivers/input/joystick/gip/gip-arcade-stick.c new file mode 100644 index 000000000000..11ededfab855 --- /dev/null +++ b/drivers/input/joystick/gip/gip-arcade-stick.c @@ -0,0 +1,173 @@ +// SPDX-License-Identifier: GPL-2.0-or-later +/* + * Drivers for GIP arcade sticks + * + * Copyright (c) 2025 Valve Software + * + * This driver is based on the Microsoft GIP spec at: + * https://aka.ms/gipdocs + * https://learn.microsoft.com/en-us/openspecs/windows_protocols/ms-gipusb/e7c90904-5e21-426e-b9ad-d82adeee0dbc + */ + +#include +#include "gip.h" + +enum gip_arcade_stick_vibration_motor_support { + GIP_VIBRATION_NO_MOTOR = 0, + GIP_VIBRATION_SINGLE_MOTOR = 1, + GIP_VIBRATION_DUAL_MOTOR = 2, +}; + +struct gip_arcade_stick_info { + uint8_t vibration_motor; + uint16_t actuator_bias; +}; + +struct gip_arcade_stick_static_coniguration { + uint8_t button_count; + uint8_t report_version; + uint8_t vibration_motor_support; + uint8_t actuator_bits; +}; + +static int gip_arcade_stick_probe(struct gip_attachment *attachment) +{ + struct gip_arcade_stick_info *info = kzalloc(sizeof(*info), GFP_KERNEL); + + if (!info) + return -ENOMEM; + + attachment->quirks |= GIP_QUIRK_SWAP_LB_RB; + attachment->driver_data = info; + + return 0; +} + +static void gip_arcade_stick_remove(struct gip_attachment *attachment) +{ + kfree(attachment->driver_data); + attachment->driver_data = NULL; +} + +static int gip_init_arcade_stick(struct gip_attachment *attachment) +{ + if (gip_supports_vendor_message(attachment, GIP_CMD_INITIAL_REPORTS_REQUEST, false)) { + uint8_t request = GIP_LL_STATIC_CONFIGURATION; + int rc = gip_send_vendor_message(attachment, GIP_CMD_INITIAL_REPORTS_REQUEST, 0, + &request, sizeof(request)); + + if (rc < 0) + return rc; + + return GIP_INIT_NO_INPUT; + } + + return 0; +} + +static int gip_setup_arcade_stick_input(struct gip_attachment *attachment, struct input_dev *input) +{ + struct gip_arcade_stick_info *info = attachment->driver_data; + int rc; + + if (!info) + return -ENODEV; + + rc = gip_driver_navigation.setup_input(attachment, input); + if (rc < 0) + return rc; + + if (info->actuator_bias > 0) { + input_set_abs_params(input, ABS_X, + -info->actuator_bias, info->actuator_bias - 1, 0, 0); + input_set_abs_params(input, ABS_Y, + -info->actuator_bias, info->actuator_bias - 1, 0, 0); + } + + if (attachment->extra_buttons >= 1) + input_set_capability(input, EV_KEY, BTN_TR2); + + if (attachment->extra_buttons >= 2) + input_set_capability(input, EV_KEY, BTN_TL2); + return 0; +} + +static int gip_handle_arcade_stick_report(struct gip_attachment *attachment, + struct input_dev *input, const uint8_t *bytes, int num_bytes) +{ + struct gip_arcade_stick_info *info = attachment->driver_data; + int rc; + + if (!info) + return -ENODEV; + + rc = gip_driver_navigation.handle_input_report(attachment, input, bytes, num_bytes); + if (rc < 0) + return rc; + + if (num_bytes < 6) { + gip_dbg(attachment, "Discarding too-short input report\n"); + return -EINVAL; + } + + if (info->actuator_bias > 0) { + input_report_abs(input, ABS_X, get_unaligned_le16(&bytes[2]) - info->actuator_bias); + input_report_abs(input, ABS_Y, get_unaligned_le16(&bytes[4]) - info->actuator_bias); + } + + if (num_bytes >= 19) { + /* Extra button 6 */ + input_report_key(input, BTN_TR2, bytes[18] & BIT(6)); + /* Extra button 7 */ + input_report_key(input, BTN_TL2, bytes[18] & BIT(7)); + } + + return 0; +} + +static int gip_handle_arcade_stick_ll_static_configuration(struct gip_attachment *attachment, + const struct gip_header *header, const uint8_t *bytes, int num_bytes) +{ + const struct gip_arcade_stick_static_coniguration *config = + (const struct gip_arcade_stick_static_coniguration *)bytes; + struct gip_arcade_stick_info *info = attachment->driver_data; + + if (!info) + return -ENODEV; + + if (num_bytes < 4) + return -EINVAL; + + attachment->extra_buttons = clamp(config->button_count, 6, 38) - 6; + if (config->actuator_bits) + info->actuator_bias = BIT(min(config->actuator_bits, 16) - 1); + else + info->actuator_bias = 0; + + if (config->vibration_motor_support == GIP_VIBRATION_NO_MOTOR) + attachment->features &= ~GIP_FEATURE_MOTOR_CONTROL; + + gip_dbg(attachment, "Arcade stick with report version %u, %u buttons, %u actuator bits\n", + config->report_version, config->button_count, config->actuator_bits); + + return gip_setup_input_device(attachment); +} + +const struct gip_driver gip_driver_arcade_stick = { + .types = (const char* const[]) { + "Windows.Xbox.Input.ArcadeStick", + "Microsoft.Xbox.Input.ArcadeStick", + NULL + }, + .guid = GUID_INIT(0x332054cc, 0xa34b, 0x41d5, 0xa3, 0x4a, + 0xa6, 0xa6, 0x71, 0x1e, 0xc4, 0xb3), + + .probe = gip_arcade_stick_probe, + .remove = gip_arcade_stick_remove, + .init = gip_init_arcade_stick, + .setup_input = gip_setup_arcade_stick_input, + .handle_input_report = gip_handle_arcade_stick_report, + .vendor_handlers = { + [GIP_LL_STATIC_CONFIGURATION] = gip_handle_arcade_stick_ll_static_configuration, + }, +}; diff --git a/drivers/input/joystick/gip/gip-core.c b/drivers/input/joystick/gip/gip-core.c index 157a22c80abd..9e43d9cba453 100644 --- a/drivers/input/joystick/gip/gip-core.c +++ b/drivers/input/joystick/gip/gip-core.c @@ -10,7 +10,8 @@ * - Raw character device * - Wheel support * - Flight stick support - * - Arcade stick support + * - More arcade stick testing + * - Arcade stick extra buttons * - Split into driver-per-attachment GIP-as-a-bus approach drivers * * This driver is based on the Microsoft GIP spec at: @@ -331,6 +332,7 @@ struct gip_direct_motor { static const struct gip_driver *base_drivers[] = { &gip_driver_navigation, &gip_driver_gamepad, + &gip_driver_arcade_stick, NULL /* Sentinel */ }; @@ -1582,7 +1584,7 @@ static int gip_handle_command_raw_report(struct gip_attachment *attachment, return 0; } -static int gip_setup_input_device(struct gip_attachment *attachment) +int gip_setup_input_device(struct gip_attachment *attachment) { struct input_dev *input; int rc; diff --git a/drivers/input/joystick/gip/gip.h b/drivers/input/joystick/gip/gip.h index d2ad10f77bc2..98add856ae21 100644 --- a/drivers/input/joystick/gip/gip.h +++ b/drivers/input/joystick/gip/gip.h @@ -406,6 +406,8 @@ static inline struct device *gip_security_dev(struct gip_security *security) return gip_attachment_dev(container_of(security, struct gip_attachment, security)); } +int gip_setup_input_device(struct gip_attachment *attachment); + bool gip_supports_vendor_message(struct gip_attachment *attachment, uint8_t command, bool upstream); int gip_send_system_message(struct gip_attachment *attachment, @@ -421,6 +423,4 @@ void gip_security_release(struct gip_security *security); extern const struct gip_driver gip_driver_navigation; extern const struct gip_driver gip_driver_gamepad; extern const struct gip_driver gip_driver_arcade_stick; -extern const struct gip_driver gip_driver_wheel; -extern const struct gip_driver gip_driver_flight_stick; #endif -- 2.54.0