-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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 bulk devices on Windows and Mac #13008
Support for bulk devices on Windows and Mac #13008
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some general suggestions. I've no USB Bulk device to test this.
eb3973b
to
3bd6945
Compare
I took the initiative on expending the scope of this PR to also include Mac support as it seems it needs the same changes. This was tested with S4 Mk3 screens |
You need also to enabled BULK support in our Windows build setups: mixxx/tools/windows_buildenv.bat Line 216 in 99406cb
mixxx/.github/workflows/build.yml Line 96 in 99406cb
|
5447ce6
to
945e023
Compare
945e023
to
5e162f8
Compare
Just a heads up here - there seems to be reports of issues on Zulip about Mac and that feature. Let's hold on merging till we have further details and potentially fix the usecase first. I did some testing on MacOS M1 (Sonoma I believe) and it worked but there might be edge case other arch/OS version |
I wonder what the kernel driver is, that needs to be detached, a generic one, that comes with the OS like WinUSB, or a Native Instruments driver for the S4 Mk3. |
Yes, I think your theory is correct. |
I'm not sure, but it would be good to test this without the NI driver package as well, to get a better understanding. |
It's worth to mention I had to change the kernel driver from the NI to WinUSB one, so I guess this usecase isn't possible, or is it? I'm not sure how we could allow both kernels (And services) to be toggled on demand, and since I don't have much experience on Windows (nor the setup, and the interest to be honest), it's hard for me to work on a robust comprehensive solution here. |
Could we consider merging this as-is as it is improving the overall bulk support on non-Linux platform (especially Mac, allowing bulk to work out of the box on MacOS 14). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code is fine now, and I will merge it therefore!
But this functionality is not enough, that a normal Windows user can use USB Bulk devices as on Linux. This is because the WinUSB driver registration for the device must be done manually by an Administrator using a tool like Zadig.
Thank you, for addressing this long pending limitation! |
Allow bulk devices to be opened on Windows when using
WinUSB
and Mac.