-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
fix: linux baudRate and latency errors #2253
Conversation
@GazHank I don't have a linux machine handy, would you be able to test this branch? |
With the parentheses change per above, I'm getting successful results:
I've tested on:
I should call out that I'm not really able to test that low latency mode works, only that it doesn't seem to break stuff anymore :-) Additionally I've tried sending some dodgy values (0 and very large numbers) to the update baudrate, and so far I've not managed to trigger any error codes (I double checked and this is the case with or without the change to linuxSetCustomBaudRate, so I don't think we have broken anything. |
I think dodgy baudrates are a serial driver concern less a linux kernel concern and they seem to have chosen to silently ignore them in general. Hense we have this bolted on readBaudrate method, which maybe setBaudrate should be calling to check it's work. |
@reconbot let me know if there are any extra tests you'd like me to run. I'm normally on Win10, but can fire up Linux for some testing if I can help |
Thank you =) |
- linuxGetLowLatencyMode would incorrectly get an error - linuxSetLowLatencyMode would incorrectly get an error - linuxGetSystemBaudRate would incorrectly get an error - linuxSetCustomBaudRate would incorrectly get an error
3340a40
to
0e97964
Compare
#1676 already exists 🤷 |
- linuxGetLowLatencyMode would incorrectly get an error - linuxSetLowLatencyMode would incorrectly get an error - linuxGetSystemBaudRate would incorrectly get an error - linuxSetCustomBaudRate would incorrectly get an error
Thanks to @GazHank for highlighting work was needed in this area