-
Notifications
You must be signed in to change notification settings - Fork 18
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
Support for hid-nx #71
Comments
I created the repo git-developer/batocera-extra as proof-of-concept for the integration of batocera and joycond-cemuhook. Feel free to have a look and report feedback. |
Really nice, and since it all can be added without breaking what we already have I think it's a good addition. For now I've been overwhelmed with bugfixes in other project and busy overall, but as soon as I get more time I can work on this too. I think we could create a simple |
Great, thanks for your quick feedback! Just for the record, I see the following subtasks for a proper integration of joycond-cemuhook in batocera:
If I find the time and skills for any of these, I will open a PR. |
Update: I have a working branch here, but it's not ready to merge yet because
|
Now using |
Thanks for merging all relevant PRs, and thanks a lot for maintaining this project! |
Sorry I didn't help at all with the commits, but also congrats for figuring it all out! Once again thanks for the contributions, I imagine it will help a lot of people in the future :) |
The retro-gaming distribution batocera contains hid-nx, an alternative Linux kernel HID driver for Nintendo Switch controllers.
I managed to run joycond-cemuhook in this environment successfully, but I had to jump though a few hoops to achieve this.
Here is a list of the required changes:
hid_nx
instead ofhid_nintendo
.The checks in the
main
function have to be extended accordingly.Thus, the device names differ slightly. The currently supported name is Nintendo Switch Pro Controller. In my case, the name was Pro Controller when connected via Bluetooth and Nintendo Co., Ltd. Pro Controller when connected via USB.
These names requires an entry in the list of controller names and a configuration file
profiles/Pro Controller.json
/profiles/Nintendo Co., Ltd. Pro Controller.json
.UPower
object is read.My workaround was to simply suppress the respective call (effectively disabling battery monitoring).
Technically speaking, this is my patch to make it run:
With these changes, joycond-cemuhook comes up properly and serves motion data via UDP. Within batocera, Cemu and Citra can be configured to use cemuhook (although I had to customize a few things there, too).
I opened this issue to ask if we can add support for hid-nx so that it may work out-of-the-box in a future release. My experience in Python programming and the D-Bus interface is very basic, so I don't feel ready to provide a pull request here yet. Nevertheless I'd like to give active support, be it with further investigations or testing, just need a little assistance.
The text was updated successfully, but these errors were encountered: