[PATCH v2 1/1] winebus: Optionally split joysticks with more than 6 axes.
Timo Zuccarello
wine at gitlab.winehq.org
Tue Jun 7 18:08:10 CDT 2022
From: Timo Zuccarello <timo at zuccarello.eu>
Add registry option "Split Controllers" and optionally split joysticks
with more than 6 axes to improve compatibility with applications that
do not support more than 6 axes where on Windows a driver would be
installed that splits the device.
Signed-off-by: Timo Zuccarello <timo at zuccarello.eu>
---
dlls/winebus.sys/bus_sdl.c | 87 ++++++++++++++++++++++++++++++--------
dlls/winebus.sys/main.c | 2 +
dlls/winebus.sys/unixlib.h | 1 +
3 files changed, 73 insertions(+), 17 deletions(-)
diff --git a/dlls/winebus.sys/bus_sdl.c b/dlls/winebus.sys/bus_sdl.c
index a5621972493..627ee72a6ff 100644
--- a/dlls/winebus.sys/bus_sdl.c
+++ b/dlls/winebus.sys/bus_sdl.c
@@ -28,6 +28,7 @@
#include <fcntl.h>
#include <stdarg.h>
#include <stdlib.h>
+#include <stdio.h>
#include <unistd.h>
#include <sys/types.h>
#include <dlfcn.h>
@@ -143,6 +144,7 @@ struct sdl_device
int effect_ids[256];
int effect_state[256];
LONG effect_flags;
+ int axis_offset;
};
static inline struct sdl_device *impl_from_unix_device(struct unix_device *iface)
@@ -155,7 +157,20 @@ static struct sdl_device *find_device_from_id(SDL_JoystickID id)
struct sdl_device *impl;
LIST_FOR_EACH_ENTRY(impl, &device_list, struct sdl_device, unix_device.entry)
- if (impl->id == id) return impl;
+ if (impl->id == id && impl->axis_offset == 0) return impl;
+
+ return NULL;
+}
+
+static struct sdl_device *find_device_from_id_and_axis(SDL_JoystickID id, int axis)
+{
+ struct sdl_device *impl;
+
+ LIST_FOR_EACH_ENTRY(impl, &device_list, struct sdl_device, unix_device.entry)
+ if (impl->id == id &&
+ impl->axis_offset <= axis &&
+ impl->axis_offset + impl->unix_device.hid_device_state.abs_axis_count > axis)
+ return impl;
return NULL;
}
@@ -184,7 +199,8 @@ static BOOL descriptor_add_haptic(struct sdl_device *impl)
USHORT i, count = 0;
USAGE usages[16];
- if (!pSDL_JoystickIsHaptic(impl->sdl_joystick) ||
+ if (impl->axis_offset > 0 ||
+ !pSDL_JoystickIsHaptic(impl->sdl_joystick) ||
!(impl->sdl_haptic = pSDL_HapticOpenFromJoystick(impl->sdl_joystick)))
impl->effect_support = 0;
else
@@ -196,7 +212,7 @@ static BOOL descriptor_add_haptic(struct sdl_device *impl)
impl->effect_support |= WINE_SDL_HAPTIC_RUMBLE;
}
- if (pSDL_JoystickRumble && !pSDL_JoystickRumble(impl->sdl_joystick, 0, 0, 0))
+ if (impl->axis_offset == 0 && pSDL_JoystickRumble && !pSDL_JoystickRumble(impl->sdl_joystick, 0, 0, 0))
impl->effect_support |= WINE_SDL_JOYSTICK_RUMBLE;
if (impl->effect_support & EFFECT_SUPPORT_HAPTICS)
@@ -262,6 +278,7 @@ static NTSTATUS build_joystick_report_descriptor(struct unix_device *iface)
USAGE_AND_PAGE physical_usage;
axis_count = pSDL_JoystickNumAxes(impl->sdl_joystick);
+ if (options.split_controllers) axis_count = min(6, axis_count - impl->axis_offset);
if (axis_count > ARRAY_SIZE(absolute_usages))
{
FIXME("More than %zu absolute axes found, ignoring.\n", ARRAY_SIZE(absolute_usages));
@@ -275,8 +292,16 @@ static NTSTATUS build_joystick_report_descriptor(struct unix_device *iface)
ball_count = ARRAY_SIZE(relative_usages) / 2;
}
- hat_count = pSDL_JoystickNumHats(impl->sdl_joystick);
- button_count = pSDL_JoystickNumButtons(impl->sdl_joystick);
+ if (impl->axis_offset == 0)
+ {
+ hat_count = pSDL_JoystickNumHats(impl->sdl_joystick);
+ button_count = pSDL_JoystickNumButtons(impl->sdl_joystick);
+ }
+ else
+ {
+ hat_count = 0;
+ button_count = 0;
+ }
if (!pSDL_JoystickGetType) physical_usage = device_usage;
else switch (pSDL_JoystickGetType(impl->sdl_joystick))
@@ -902,8 +927,9 @@ static void sdl_add_device(unsigned int index)
SDL_JoystickID id;
SDL_JoystickGUID guid;
SDL_GameController *controller = NULL;
- const char *str;
- char guid_str[33];
+ const char *str, *product;
+ char guid_str[33], buffer[ARRAY_SIZE(desc.product)];
+ int axis_count, axis_offset;
if ((joystick = pSDL_JoystickOpen(index)) == NULL)
{
@@ -916,7 +942,12 @@ static void sdl_add_device(unsigned int index)
if (controller) str = pSDL_GameControllerName(controller);
else str = pSDL_JoystickName(joystick);
- if (str) ntdll_umbstowcs(str, strlen(str) + 1, desc.product, ARRAY_SIZE(desc.product));
+ if (str)
+ {
+ ntdll_umbstowcs(str, strlen(str) + 1, desc.product, ARRAY_SIZE(desc.product));
+ product = str;
+ }
+ else product = "Joystick";
id = pSDL_JoystickInstanceID(joystick);
@@ -936,10 +967,14 @@ static void sdl_add_device(unsigned int index)
pSDL_JoystickGetGUIDString(guid, guid_str, sizeof(guid_str));
ntdll_umbstowcs(guid_str, strlen(guid_str) + 1, desc.serialnumber, ARRAY_SIZE(desc.serialnumber));
- if (controller) desc.is_gamepad = TRUE;
+ if (controller)
+ {
+ desc.is_gamepad = TRUE;
+ axis_count = 6;
+ }
else
{
- int button_count, axis_count;
+ int button_count;
axis_count = pSDL_JoystickNumAxes(joystick);
button_count = pSDL_JoystickNumButtons(joystick);
@@ -948,13 +983,20 @@ static void sdl_add_device(unsigned int index)
TRACE("%s id %d, desc %s.\n", controller ? "controller" : "joystick", id, debugstr_device_desc(&desc));
- if (!(impl = hid_device_create(&sdl_device_vtbl, sizeof(struct sdl_device)))) return;
- list_add_tail(&device_list, &impl->unix_device.entry);
- impl->sdl_joystick = joystick;
- impl->sdl_controller = controller;
- impl->id = id;
-
- bus_event_queue_device_created(&event_queue, &impl->unix_device, &desc);
+ for (axis_offset = 0; axis_offset < axis_count; axis_offset += (options.split_controllers ? 6 : axis_count))
+ {
+ if (axis_offset) snprintf(buffer, ARRAY_SIZE(buffer), "%s %d", product, axis_offset / 6);
+ else snprintf(buffer, ARRAY_SIZE(buffer), "%s", product);
+ ntdll_umbstowcs(buffer, strlen(buffer) + 1, desc.product, ARRAY_SIZE(desc.product));
+ if (!(impl = hid_device_create(&sdl_device_vtbl, sizeof(struct sdl_device)))) return;
+ list_add_tail(&device_list, &impl->unix_device.entry);
+ impl->sdl_joystick = joystick;
+ impl->sdl_controller = controller;
+ impl->id = id;
+ impl->axis_offset = axis_offset;
+ bus_event_queue_device_created(&event_queue, &impl->unix_device, &desc);
+ if (options.split_controllers && axis_count > 6) TRACE("%s id %d, split for axis %d-%d.\n", controller ? "controller" : "joystick", id, axis_offset, axis_offset + min(6, axis_count - axis_offset));
+ }
}
static void process_device_event(SDL_Event *event)
@@ -975,6 +1017,17 @@ static void process_device_event(SDL_Event *event)
if (impl) bus_event_queue_device_removed(&event_queue, &impl->unix_device);
else WARN("failed to find device with id %d\n", id);
}
+ else if (event->type == SDL_JOYAXISMOTION && options.split_controllers)
+ {
+ id = ((SDL_JoyAxisEvent *)event)->which;
+ impl = find_device_from_id_and_axis(id, ((SDL_JoyAxisEvent *)event)->axis);
+ if (impl)
+ {
+ ((SDL_JoyAxisEvent *)event)->axis -= impl->axis_offset;
+ set_report_from_joystick_event(impl, event);
+ }
+ else WARN("failed to find device with id %d for axis %d\n", id, ((SDL_JoyAxisEvent *)event)->axis);
+ }
else if (event->type >= SDL_JOYAXISMOTION && event->type <= SDL_JOYBUTTONUP)
{
id = ((SDL_JoyButtonEvent *)event)->which;
diff --git a/dlls/winebus.sys/main.c b/dlls/winebus.sys/main.c
index 4588fee1b02..35db8a5fb9a 100644
--- a/dlls/winebus.sys/main.c
+++ b/dlls/winebus.sys/main.c
@@ -720,6 +720,8 @@ static NTSTATUS sdl_driver_init(void)
};
NTSTATUS status;
+ bus_options.split_controllers = check_bus_option(L"Split Controllers", 0);
+ if (bus_options.split_controllers) TRACE("SDL controller splitting enabled\n");
bus_options.map_controllers = check_bus_option(L"Map Controllers", 1);
if (!bus_options.map_controllers) TRACE("SDL controller to XInput HID gamepad mapping disabled\n");
sdl_bus_load_mappings(&bus_options);
diff --git a/dlls/winebus.sys/unixlib.h b/dlls/winebus.sys/unixlib.h
index a84a3331a25..ba7674d36a7 100644
--- a/dlls/winebus.sys/unixlib.h
+++ b/dlls/winebus.sys/unixlib.h
@@ -47,6 +47,7 @@ struct device_desc
struct sdl_bus_options
{
BOOL map_controllers;
+ BOOL split_controllers;
/* freed after bus_init */
UINT mappings_count;
char **mappings;
--
GitLab
https://gitlab.winehq.org/wine/wine/-/merge_requests/181
More information about the wine-devel
mailing list