Page MenuHomeSolus

Initial inclusion of xone
ClosedPublic

Authored by infinitymdm on Sun, Nov 20, 8:55 PM.

Details

Summary

Initial inclusion of xone, a driver for Xbox Wireless recievers. Fixes T10185

Test Plan
  • With the dongle unplugged, run sudo xone-get-firmware.sh. After accepting the EULA, Microsoft's proprietary firmware will be downloaded.
  • Plug in the dongle.
  • Pair an Xbox controller as you normally would. See the "Connect your controller" section of this page.
  • Launch an application that uses the controller (I use the controller configuration window in Yuzu) and verify that all inputs work as expected.
  • Launch a game or application that uses controller force feedback (I use Yuzu for this as well) and ensure that force feedback works as expected.

Diff Detail

Repository
R5643 xone
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

infinitymdm created this revision.Sun, Nov 20, 8:55 PM
infinitymdm requested review of this revision.Sun, Nov 20, 8:55 PM

Note that you need an xbox wireless dongle to use this. I have the slim one shown on this page: https://www.xbox.com/en-US/accessories/adapters/wireless-adapter-windows. There is also an older, slightly chunkier one. I have not tested this software with the older model.

Note also that you'll need a peripheral device capable of using Microsoft's Game Input Protocol. I use an Xbox One controller (model 1708), but there are a variety of other devices (controllers, headsets, etc.) that use GIP wirelessly. For the full list of supported devices, see https://github.com/medusalix/xone#readme.

Add this to the bottom of the install phase please:

	    # Compress modules with zstd
	    find "$installdir" -name '*.ko' -exec strip --strip-debug {} \; -exec zstd {} \; -exec rm -v {} \;
ReillyBrogan requested changes to this revision.Sun, Nov 20, 9:09 PM
This revision now requires changes to proceed.Sun, Nov 20, 9:09 PM

Oh, you are also using the branch main. You'll need to use master instead.

  • Add zstd compression
  • Remove a bit of whitespace
  • Switch branch name to master

Oh, you are also using the branch main. You'll need to use master instead.

I gave it a git branch -m master, but I'm not sure if arcanist picked that up. Let me know if I need to take another look at it.

ReillyBrogan accepted this revision.Mon, Nov 21, 12:54 AM

LGTM, thanks!

This revision is now accepted and ready to land.Mon, Nov 21, 12:54 AM
This revision was automatically updated to reflect the committed changes.