Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

xpadneo, hidraw: Also work around SDL2 hidraw mode conflicts #320

Merged
merged 1 commit into from
Nov 13, 2021

Conversation

kakra
Copy link
Collaborator

@kakra kakra commented Nov 13, 2021

Also fixes Steam Input which apparently also uses SDL2.

The bug shows up when SDL identifies the controller as being on USB
instead of Bluetooth (SDL Gamepad ID starts with 03 instead of 05)
which then messes with SDL's expectations for the contents of hidraw.

This fixes two SDL problems:

  • Wrong mappings in hidraw (buttons shifted to other buttons)
  • SDL2 doesn't properly throttle rumble commands (endless rumble)

See-also: #286
Maybe-fixes: #303
Maybe-fixes: #311
Maybe-fixes: #314
Signed-off-by: Kai Krakow [email protected]

Also fixes Steam Input which apparently also uses SDL2.

The bug shows up when SDL identifies the controller as being on USB
instead of Bluetooth (SDL Gamepad ID starts with `03` instead of `05`)
which then messes with SDL's expectations for the contents of hidraw.

This fixes two SDL problems:

  * Wrong mappings in hidraw (buttons shifted to other buttons)
  * SDL2 doesn't properly throttle rumble commands (endless rumble)

See-also: atar-axis#286
Maybe-fixes: atar-axis#303
Maybe-fixes: atar-axis#311
Maybe-fixes: atar-axis#314
Signed-off-by: Kai Krakow <[email protected]>
@codeclimate
Copy link

codeclimate bot commented Nov 13, 2021

Code Climate has analyzed commit 8747e7b and detected 0 issues on this pull request.

View more on Code Climate.

@kakra kakra merged commit 5aee029 into atar-axis:master Nov 13, 2021
@kakra kakra deleted the bugfix/hidraw-sdl2-steaminput branch November 13, 2021 20:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant