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

Remove xon/xoff flow control, now that we have more proper usb-level flow control #1370

Merged
merged 1 commit into from
Dec 14, 2023

Conversation

obra
Copy link
Member

@obra obra commented Dec 14, 2023

Now that @tlyu has fixed bugs in the usb-level flow control implementations for avr and gd32 cores, it looks like we don't need the (flawed) xon/xoff support.

This reverts commit 81851f7.
This reverts commit 601a023.
This reverts commit 92de6c0.
This reverts commit 3490984.

…flow control.

This reverts commit 81851f7.
This reverts commit 601a023.
This reverts commit 92de6c0.
This reverts commit 3490984.
@obra obra merged commit e08ec31 into master Dec 14, 2023
15 checks passed
@obra obra deleted the f/remove-xonoff branch December 14, 2023 03:48
@tlyu tlyu linked an issue Dec 14, 2023 that may be closed by this pull request
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.

xon/xoff flow control may be stalling out waiting for more input.
1 participant