-
-
Notifications
You must be signed in to change notification settings - Fork 725
Add USB send timeout (take 2) #154
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
Add USB send timeout (take 2) #154
Conversation
Hi @RamonRibes, Could you please add this to your testing list with the WiFi101 web server example, it's a replacement for #152. You should not see any delays with the web server now if you open and close the Serial monitor on Windows. So far I've tested with my Mac running OS X 10.11.3 and a Windows 10 VM. |
Hi, @sandeepmistry,
...and is working as expected. |
Timeout is checked on the next send attempt. Initial timeout is set to 70ms, after a time out occurs and the previous transfer is still not complete a send will report as failed. If a send times out, a ZLP will be in the send queue to check for future timeouts.
It’s no longer needed with the timeout mechanism in USBCore.
f2aab04
to
b2462a5
Compare
@ArduinoBot is misbehaving a bit, however here's a new build: ✅ Build completed. ⬇️ Build URL: ℹ️ To test this build:
|
Working OK for me 👍 |
ac0a30e
to
b2462a5
Compare
hallowing m4 final UART fix
Replacement for #152.
This time I've used Paul Stoffregen's work on Teensy as inspiration.
The initial send timeout is 70ms, however if a send times out the next one will fail immediately if the previous write is not complete. A ZLP will be attempted to send if a timeout occurs, to detect the endpoint is ready to receive.
I've also removed the check for line state in
Serial_::write
. After opening the port the user or application can manually disable DTR or RTS.