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

Re-integrate UART hardware flow control #540

Closed
stnolting opened this issue Mar 8, 2023 Discussed in #537 · 1 comment · Fixed by #541
Closed

Re-integrate UART hardware flow control #540

stnolting opened this issue Mar 8, 2023 Discussed in #537 · 1 comment · Fixed by #541
Labels
enhancement New feature or request HW Hardware-related

Comments

@stnolting
Copy link
Owner

Discussed in #537

Originally posted by vhdlnerd March 8, 2023
I'm disappointed with the recent removal of the UART hardware handshaking lines. This breaks all my neorv32 based designs. I'm a little shocked that functionality would be removed from a basic interface. I use the UARTs at 921600 baud sending commands to the neorv32 that take time to process -- hardware handshake lines are required to avoid dropping commands.

The two workarounds mentioned in the change notes (adding a non-crippled UART as a custom function or a wishbone module) are equally poor. Either way means having a custom bootloader and worst not being able to use any of neov32's SW library functions that uses UARTs (neorv32_uart_printf, ...).

Brian

@stnolting stnolting added enhancement New feature or request HW Hardware-related labels Mar 8, 2023
@stnolting stnolting changed the title Recent UART Changes Re-integrate UARt hardware flow control Mar 8, 2023
@stnolting stnolting changed the title Re-integrate UARt hardware flow control Re-integrate UART hardware flow control Mar 8, 2023
@vhdlnerd
Copy link
Contributor

vhdlnerd commented Mar 8, 2023

Thanks for adding this back in!

@stnolting stnolting linked a pull request Mar 8, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request HW Hardware-related
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants