[Bug 50321] New: Wine Linux-input joystick driver unable to detect joystick with id > 64
https://bugs.winehq.org/show_bug.cgi?id=50321 Bug ID: 50321 Summary: Wine Linux-input joystick driver unable to detect joystick with id > 64 Product: Wine Version: 6.0-rc2 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: directx-dinput Assignee: wine-bugs(a)winehq.org Reporter: messengerofdeath(a)gmail.com Distribution: --- Created attachment 68903 --> https://bugs.winehq.org/attachment.cgi?id=68903 Patch for scanning of /dev/input/event* My joystick is not visible in wine (wine control joy.cpl) as evdev device. Visible only as js device but js device works bad with my device. I found that wine only can support joystick via evdev that have id less then 64. According to joystick_linuxinput.c currently maximum scanned /dev/input/eventX file is the /dev/input/event64 but on my system i have joystick on event 265(throttle) and 266(stick) Possible solution is to extend MAX_JOYDEV to some big value (512 for example). Or another way - scan /dev/input folder. I've attached working patch for joystick_linuxinput.c that uses readdir functionality. Reproducible on 5.22, 6.0-rc1, 6.0-rc2 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=50321 Denis <messengerofdeath(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #68903|0 |1 is obsolete| | --- Comment #1 from Denis <messengerofdeath(a)gmail.com> --- Created attachment 68904 --> https://bugs.winehq.org/attachment.cgi?id=68904 Patch for scanning of /dev/input/event* Added exit condition when directory fully read -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=50321 Torge Matthies <openglfreak(a)googlemail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |openglfreak(a)googlemail.com --- Comment #2 from Torge Matthies <openglfreak(a)googlemail.com> --- Created attachment 69163 --> https://bugs.winehq.org/attachment.cgi?id=69163 Less broken version of the previous patch I fixed some things in the patch, please test this one @Denis -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=50321 --- Comment #3 from Denis <messengerofdeath(a)gmail.com> --- No, it doesn't work. I made small checks and corrected this version. Also I've added closedir at the end. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=50321 Denis <messengerofdeath(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #68904|0 |1 is obsolete| | --- Comment #4 from Denis <messengerofdeath(a)gmail.com> --- Created attachment 69164 --> https://bugs.winehq.org/attachment.cgi?id=69164 Patch for scanning of /dev/input/event* -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=50321 Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=50321 Rémi Bernon <rbernon(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |rbernon(a)codeweavers.com --- Comment #5 from Rémi Bernon <rbernon(a)codeweavers.com> --- The legacy joystick backends have been removed, and I believe winebus.sys doesn't have this limitation (it either uses SDL or udev for device enumeration). Please re-check if this is still happening with latest Wine HEAD. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (1)
-
WineHQ Bugzilla