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

feat: Add xone and xpad-noone drivers #314

Merged
merged 1 commit into from
Aug 27, 2023
Merged

feat: Add xone and xpad-noone drivers #314

merged 1 commit into from
Aug 27, 2023

Conversation

KyleGospo
Copy link
Member

No description provided.

@KyleGospo KyleGospo marked this pull request as ready for review August 27, 2023 18:03
Copy link
Contributor

@fiftydinar fiftydinar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait, you should not remove xpadneo as you will cut Bluetooth Xbox One gamepad support than.

We should combine all 3 of these: xpadneo, xone & xpad-noone

@KyleGospo
Copy link
Member Author

@bsherman @castrojo Needs real-world testing, modules build/mages look good.

@KyleGospo
Copy link
Member Author

Wait, you should not remove xpadneo as you will cut Bluetooth Xbox One gamepad support than.

We should combine all 3 of these: xpadneo, xone & xpad-noone

Does xpadneo not conflict with xone/xpad?

@fiftydinar
Copy link
Contributor

Wait, you should not remove xpadneo as you will cut Bluetooth Xbox One gamepad support than.
We should combine all 3 of these: xpadneo, xone & xpad-noone

Does xpadneo not conflict with xone/xpad?

Xone conflicted with xpad before, now it doesn't since you added xpad-noone

Xpadneo had no issues

I don't have those gamepads to test, I'm just looking at their github. Only xpadneo covers Bluetooth Xbox One gamepads as far as I see.

@KyleGospo KyleGospo changed the title feat: Replace xpadneo with xone and xpad-noone drivers feat: Add xone and xpad-noone drivers Aug 27, 2023
@KyleGospo
Copy link
Member Author

Wait, you should not remove xpadneo as you will cut Bluetooth Xbox One gamepad support than.
We should combine all 3 of these: xpadneo, xone & xpad-noone

Does xpadneo not conflict with xone/xpad?

Xone conflicted with xpad before, now it doesn't since you added xpad-noone

Xpadneo had no issues

I don't have those gamepads to test, I'm just looking at their github. Only xpadneo covers Bluetooth Xbox One gamepads as far as I see.

It's restored now

Copy link
Contributor

@bsherman bsherman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great! I can test anytime... I'm willing to merge first and test on an official build if others are.

@castrojo
Copy link
Member

+1 to that, let's try it.

@castrojo castrojo added this pull request to the merge queue Aug 27, 2023
@bsherman
Copy link
Contributor

I'll be testing on my AMD machine so I can test as soon as this main repo main branch build completes :-)

@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 27, 2023
@KyleGospo KyleGospo added this pull request to the merge queue Aug 27, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 27, 2023
@KyleGospo KyleGospo merged commit 940a231 into main Aug 27, 2023
13 checks passed
@KyleGospo KyleGospo deleted the xpad-xone branch August 27, 2023 19:25
@throwaway43
Copy link

throwaway43 commented Sep 3, 2023

Unfortunately, I have not been able to get my xbox one controller working (wired) since this has been merged. As prescribed, I made sure to install the latest drivers on Windows (through Xbox-Accessories). I even went so far as to reinstall Silverblue entirely in hopes of remedying the issue, but alas. Even rebasing from my custom image (startingpoint) to silverblue-main, bazzite-gnome and bluefin didn't solve it. Eventually, I got somewhat desperate and decided to brute-force it. The following has finally resolved it for me, but I can't shake off how wrong this feels:

  1. Fork ublue-os/main
  2. Remove the following lines in main-install.sh:
    /tmp/akmods-rpms/kmods/*xpadneo*.rpm
    /tmp/akmods-rpms/kmods/*xone*.rpm
    /tmp/akmods-rpms/kmods/*xpad-noone*.rpm
  3. (Ensuring the actions work again and even customize it so that it only creates the one image I use (silverblue-main)). Make changes to Containerfile and recipe.yml to change the base image to your forked one.

I tried simply uninstalling xone etc on my device, but to no avail. That's why I succumbed to the aforementioned inelegant solution. I'm sure a far more elegant solution exists. But this is my workaround till then 😅.

@bsherman
Copy link
Contributor

bsherman commented Sep 3, 2023

Unfortunately, I have not been able to get my xbox one controller working (wired) since this has been merged. As prescribed, I made sure to install the latest drivers on Windows (through Xbox-Accessories). I even went so far as to reinstall Silverblue entirely in hopes of remedying the issue, but alas.

That IS frustrating, so thank you for coming back to report your workaround.

Did you ever get any logs about the problem? Running dmesg -T -w in a terminal, then plugging in the controller should show what's happening.

Without logs the primary culprit I would expect here is SecureBoot enabled on an AMD machine which didn't otherwise require loading the universal blue MOK with just enroll-secure-boot-key.

@throwaway43
Copy link

Running dmesg -T -w in a terminal, then plugging in the controller should show what's happening.

[Sun Sep 3 17:51:19 2023] usb 1-2: new full-speed USB device number 6 using xhci_hcd
[Sun Sep 3 17:51:20 2023] usb 1-2: New USB device found, idVendor=045e, idProduct=02ea, bcdDevice= 5.11
[Sun Sep 3 17:51:20 2023] usb 1-2: New USB device strings: Mfr=1, Product=2, SerialNumber=3
[Sun Sep 3 17:51:20 2023] usb 1-2: Product: Controller
[Sun Sep 3 17:51:20 2023] usb 1-2: Manufacturer: Microsoft
[Sun Sep 3 17:51:20 2023] usb 1-2: SerialNumber: 3032363030353438393736363234
[Sun Sep 3 17:51:20 2023] Loading of module with unavailable key is rejected

Without logs the primary culprit I would expect here is SecureBoot enabled on an AMD machine which didn't otherwise require loading the universal blue MOK with just enroll-secure-boot-key.

Bullseye 👏 ! Thank you for your input! Much appreciated! I'll revert my image back to how it used to be 😄 ! As the kernel modules are enabled by default, is it perhaps worthwhile to mention the importance of the just enroll-secure-boot-key command more prominently?

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.

5 participants