Skip to content
/ xone Public

🔧 Fix compatibility issues with Linux 5.15+ (vmalloc, mmap, etc.)#62

Open
WildWestCyberSecurity wants to merge 5 commits intomedusalix:masterfrom
WildWestCyberSecurity:master
Open

🔧 Fix compatibility issues with Linux 5.15+ (vmalloc, mmap, etc.)#62
WildWestCyberSecurity wants to merge 5 commits intomedusalix:masterfrom
WildWestCyberSecurity:master

Conversation

@WildWestCyberSecurity
Copy link
Copy Markdown

This PR fixes multiple issues in the xone driver related to newer Linux kernels (5.15+):

  • Fixed gip_bus_match function signature to match kernel changes.
  • Replaced snd_pcm_lib_alloc_vmalloc_buffer() and snd_pcm_lib_free_vmalloc_buffer()
    with snd_pcm_set_managed_buffer() where applicable.
  • Removed deprecated .mmap and .page assignments from snd_pcm_ops.
  • Ensured compatibility with older kernels using #if LINUX_VERSION_CODE macros.

These changes have been tested on kernel 6.12.13.

Let me know if any further modifications are needed!

Btw thankyou so much for this project you da goat fr fr!

Comment thread bus/bus.c
Comment thread driver/headset.c
@SirCiorap335
Copy link
Copy Markdown

ArchLinux
for me it worked with content from this PR. first install failed with error related to gip_bus_match. then I ran uninstall.sh, copied content from this PR, then next install worked.
Thx!

@tricksan3
Copy link
Copy Markdown

Pop OS
The changes on this pr worked perfectly for me.

@andecker
Copy link
Copy Markdown

Can confirm that it works on Pop OS with the new kernel.

@gkatsandris
Copy link
Copy Markdown

Just did an apt upgrade and controller didn't turn on anymore because of the kernel switch. I pulled your master and did sudo ./uninstall.sh then sudo ./install.sh, unplugged and plugged in the controller, and it turned on. Thanks!

@WildWestCyberSecurity
Copy link
Copy Markdown
Author

Happy to help out my fellow gamers ✌️

@Silent-Hunter
Copy link
Copy Markdown

Thanks, after updating my Raspberry Pi 5 the other day, this no longer works, but your fork does.

@HarryT-123
Copy link
Copy Markdown

Tested and working on Elementary OS 8 (Ubuntu 24.04.2) - Thanks! :)

Installing without these changes caused the same error seen in #64

@Lawstorant
Copy link
Copy Markdown

Lawstorant commented Apr 11, 2025

author is MIA and project is dead, please open PRs to https://github.com/dlundqvist/xone

This is an active and maintained fork

@WildWestCyberSecurity
Copy link
Copy Markdown
Author

WildWestCyberSecurity commented Apr 12, 2025

author is MIA and project is dead, please open PRs to https://github.com/dlundqvist/xone

This is an active and maintained fork

I'm a patient man/creature, anyways I appreciate this nonetheless and do see dlundqvist has contributed to the project quite a long time as well. At time of writing past 2+ years.

I'll give author Medusalix some more time but if more breaking changes continue as time goes on I may consider. For now it fits my purposes and I know it's from the OG.

Thankyou for taking time to let me and others know about it!

Comment thread driver/headset.c Outdated
Comment thread driver/headset.c Outdated
Comment thread driver/headset.c Outdated
Comment thread driver/headset.c Outdated
@WildWestCyberSecurity
Copy link
Copy Markdown
Author

Thankyou so much Lawstorant for your careful eye! I fixed the slop up a bit as requested and as should be!

Looks much better now, I know it was just indentations/visual but I went ahead and re-tested to make sure nothing unexpected would happen. Works great, let me know if you have any other recommendations.

bd7pca pushed a commit to bd7pca/recalbox that referenced this pull request Feb 13, 2026
The patch comes from medusalix/xone#62
Thanks WildWestCyberSecurity
bd7pca pushed a commit to bd7pca/recalbox that referenced this pull request Feb 13, 2026
The patch comes from medusalix/xone#62
Thanks WildWestCyberSecurity
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.

9 participants