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

udev rule #4

Closed
wants to merge 7 commits into from
Closed

udev rule #4

wants to merge 7 commits into from

Conversation

ciolast
Copy link

@ciolast ciolast commented Sep 2, 2020

Hi, thanks for your work, it's amazing!

I installed your module, and now my T300RS works well.

But I had an issue on my Fedora 32. My wheel had a deadzone, and iIf I set up a udev rule, Fedora freeze at launch, and reboot again and again.

On Fedora it's not necessary to create a file on udev folder. Each time I want to use my wheel I write this on terminal

evdev-joystick --evdev /dev/input/by-id/usb-Thrustmaster_Thrustmaster_T300RS_Racing_wheel-event-joystick --axis 0 --deadzone 0

Now it's working very well, I tried it on Assetto Corsa Competizione, ET2, and KartKraft.

Thanks again for your work!

@Kimplul
Copy link
Owner

Kimplul commented Sep 2, 2020

Thank you for your kind words. This genuinely means a lot to me.

I'm glad you found a way around the issue, but that's really weird. I'll try installing Fedora on a virtual machine and see if I can reproduce it.

Quick question though, why did you open a pull request instead of an issue?

EDIT: Somewhat interesting, on Fedora I did get a warning, stating that having the rule on two lines made udev ignore the RUN+=... part, but I couldn't get it to freeze. Anycase, I updated the README with the format that didn't create any warnings.

I forgot that the module uses MODULE_LICENSE("GPL"), and it's probably for the best to use the same license in both the module and the repository.
@Kimplul
Copy link
Owner

Kimplul commented Sep 19, 2020

I'll be closing this then due to inactivity, thanks again for your kind words.

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.

None yet

2 participants