[PATCH v2] dinput: Report controllers as gamepads, not joysticks
Roderick Colenbrander
thunderbird2k at gmail.com
Wed Apr 17 10:09:16 CDT 2019
What happens on traditional joysticks on windows? Are they really setting
the gamepad flag on those too? I would have guessed they used some
heuristics for setting such flag.
Thanks,
Roderick
On Wed, Apr 17, 2019, 7:12 AM Andrew Eikum <aeikum at codeweavers.com> wrote:
> This matches what Windows does with common controllers like xbox 360.
>
> Signed-off-by: Andrew Eikum <aeikum at codeweavers.com>
> ---
>
> This allows controllers to work in some games like Telltale games and
> Yakuza Kiwami.
>
> dlls/dinput/joystick_linux.c | 9 +++------
> dlls/dinput/joystick_linuxinput.c | 9 +++------
> dlls/dinput/joystick_osx.c | 7 +++++--
> 3 files changed, 11 insertions(+), 14 deletions(-)
>
> diff --git a/dlls/dinput/joystick_linux.c b/dlls/dinput/joystick_linux.c
> index 963e62f017c..736a06e2b27 100644
> --- a/dlls/dinput/joystick_linux.c
> +++ b/dlls/dinput/joystick_linux.c
> @@ -316,9 +316,9 @@ static void
> fill_joystick_dideviceinstanceW(LPDIDEVICEINSTANCEW lpddi, DWORD ver
> lpddi->guidProduct = joystick_devices[id].guid_product;
> /* we only support traditional joysticks for now */
> if (version >= 0x0800)
> - lpddi->dwDevType = DI8DEVTYPE_JOYSTICK |
> (DI8DEVTYPEJOYSTICK_STANDARD << 8);
> + lpddi->dwDevType = DI8DEVTYPE_GAMEPAD |
> (DI8DEVTYPEJOYSTICK_STANDARD << 8);
> else
> - lpddi->dwDevType = DIDEVTYPE_JOYSTICK |
> (DIDEVTYPEJOYSTICK_TRADITIONAL << 8);
> + lpddi->dwDevType = DIDEVTYPE_JOYSTICK |
> (DIDEVTYPEJOYSTICK_GAMEPAD << 8);
>
> /* Assume the joystick as HID if it is attached to USB bus and has a
> valid VID/PID */
> if (joystick_devices[id].bus_type == BUS_USB &&
> @@ -326,10 +326,7 @@ static void
> fill_joystick_dideviceinstanceW(LPDIDEVICEINSTANCEW lpddi, DWORD ver
> {
> lpddi->dwDevType |= DIDEVTYPE_HID;
> lpddi->wUsagePage = 0x01; /* Desktop */
> - if (lpddi->dwDevType == DI8DEVTYPE_JOYSTICK || lpddi->dwDevType
> == DIDEVTYPE_JOYSTICK)
> - lpddi->wUsage = 0x04; /* Joystick */
> - else
> - lpddi->wUsage = 0x05; /* Game Pad */
> + lpddi->wUsage = 0x05; /* Game Pad */
> }
>
> MultiByteToWideChar(CP_ACP, 0, joystick_devices[id].name, -1,
> lpddi->tszInstanceName, MAX_PATH);
> diff --git a/dlls/dinput/joystick_linuxinput.c
> b/dlls/dinput/joystick_linuxinput.c
> index 910e755d3ed..cea669b90cb 100644
> --- a/dlls/dinput/joystick_linuxinput.c
> +++ b/dlls/dinput/joystick_linuxinput.c
> @@ -355,9 +355,9 @@ static void
> fill_joystick_dideviceinstanceW(LPDIDEVICEINSTANCEW lpddi, DWORD ver
> lpddi->guidFFDriver = GUID_NULL;
>
> if (version >= 0x0800)
> - lpddi->dwDevType = DI8DEVTYPE_JOYSTICK |
> (DI8DEVTYPEJOYSTICK_STANDARD << 8);
> + lpddi->dwDevType = DI8DEVTYPE_GAMEPAD |
> (DI8DEVTYPEJOYSTICK_STANDARD << 8);
> else
> - lpddi->dwDevType = DIDEVTYPE_JOYSTICK |
> (DIDEVTYPEJOYSTICK_TRADITIONAL << 8);
> + lpddi->dwDevType = DIDEVTYPE_JOYSTICK |
> (DIDEVTYPEJOYSTICK_GAMEPAD << 8);
>
> /* Assume the joystick as HID if it is attached to USB bus and has a
> valid VID/PID */
> if (joydevs[id].bus_type == BUS_USB &&
> @@ -365,10 +365,7 @@ static void
> fill_joystick_dideviceinstanceW(LPDIDEVICEINSTANCEW lpddi, DWORD ver
> {
> lpddi->dwDevType |= DIDEVTYPE_HID;
> lpddi->wUsagePage = 0x01; /* Desktop */
> - if (lpddi->dwDevType == DI8DEVTYPE_JOYSTICK || lpddi->dwDevType
> == DIDEVTYPE_JOYSTICK)
> - lpddi->wUsage = 0x04; /* Joystick */
> - else
> - lpddi->wUsage = 0x05; /* Game Pad */
> + lpddi->wUsage = 0x05; /* Game Pad */
> }
>
> MultiByteToWideChar(CP_ACP, 0, joydevs[id].name, -1,
> lpddi->tszInstanceName, MAX_PATH);
> diff --git a/dlls/dinput/joystick_osx.c b/dlls/dinput/joystick_osx.c
> index 0701b9f42cc..daef1a12ed5 100644
> --- a/dlls/dinput/joystick_osx.c
> +++ b/dlls/dinput/joystick_osx.c
> @@ -969,9 +969,12 @@ static HRESULT joydev_enum_deviceA(DWORD dwDevType,
> DWORD dwFlags, LPDIDEVICEINS
> lpddi->guidProduct.Data1 = make_vid_pid(device);
> /* we only support traditional joysticks for now */
> if (version >= 0x0800)
> - lpddi->dwDevType = DI8DEVTYPE_JOYSTICK |
> (DI8DEVTYPEJOYSTICK_STANDARD << 8);
> + lpddi->dwDevType = DI8DEVTYPE_GAMEPAD |
> (DI8DEVTYPEJOYSTICK_STANDARD << 8);
> else
> - lpddi->dwDevType = DIDEVTYPE_JOYSTICK |
> (DIDEVTYPEJOYSTICK_TRADITIONAL << 8);
> + lpddi->dwDevType = DIDEVTYPE_JOYSTICK |
> (DIDEVTYPEJOYSTICK_GAMEPAD << 8);
> + lpddi->dwDevType |= DIDEVTYPE_HID;
> + lpddi->wUsagePage = 0x01; /* Desktop */
> + lpddi->wUsage = 0x05; /* Game Pad */
> sprintf(lpddi->tszInstanceName, "Joystick %d", id);
>
> /* get the device name */
> --
> 2.21.0
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20190417/eeb78461/attachment.html>
More information about the wine-devel
mailing list