Skip to content

SInput: Mapping updates for GCUltimate/ProGCC #13698

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

Merged
merged 2 commits into from
Aug 12, 2025

Conversation

mitchellcairns
Copy link
Contributor

@mitchellcairns mitchellcairns commented Aug 6, 2025

  • In previous firmware for my gamepads, the buttons were sent in a static order for ABXY according to button letter rather than physical position.
  • To be more 'transparent' to the driver and save myself from future fragmentation, I am updating the mappings to be consistent and always expect the buttons in a given cardinal format, rather than a glyph format.
  • This better aligns with the existing 'fallback' behavior of a generic device and will help gamepad designers achieve better consistency.
  • Updated to use the SEWN cardinal layout to better reflect standard gamepads.
  • Coincides with firmware library update: HandHeldLegend/HOJA-LIB-RP2040@d45810e

- In previous firmwares for my gamepads, the buttons were sent in a static order for ABXY.
- To better be 'transparent' to the driver and save myself from future fragmentation, I am updating the mappings to be consistent, where the HID report should always expect to receive the button inputs based on cardinal button directions rather than button labels.
- This better aligns with the existing 'fallback' behavior of a generic device.
- Coincides with firmware library update: HandHeldLegend/HOJA-LIB-RP2040#30
@mitchellcairns mitchellcairns marked this pull request as ready for review August 6, 2025 02:12
@slouken slouken merged commit 62b9ac3 into libsdl-org:main Aug 12, 2025
41 checks passed
@mitchellcairns mitchellcairns deleted the sinput_gcult_parity branch August 12, 2025 22:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants