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

Serial sends messages at 1 Mbs but readme says 3 Mbs. #2

Open
Geekdude opened this issue Dec 18, 2021 · 3 comments
Open

Serial sends messages at 1 Mbs but readme says 3 Mbs. #2

Geekdude opened this issue Dec 18, 2021 · 3 comments

Comments

@Geekdude
Copy link

The serial demo sends the repeated message at 1 Mbs, but the readme says that it sends at 3 Mbs.

@peepo
Copy link

peepo commented Aug 4, 2023

indeed, did anyone get this working? using what baud rate?
or did you change the clock? 3mb is somewhat crazy as a starting point for serial....

@bwitherspoon
Copy link

indeed, did anyone get this working? using what baud rate? or did you change the clock? 3mb is somewhat crazy as a starting point for serial....

We are not using this exact UART but a very similar design on the ice40 up5k. We use 1 Mbps is a sane default. The only significant difference I see is that we sample at x8 or x16 the baud rate instead of x4. Hope that helps.

@Geekdude
Copy link
Author

indeed, did anyone get this working? using what baud rate?
or did you change the clock? 3mb is somewhat crazy as a starting point for serial....

3Mb/s UART does work on the up5k. It also seems to be about the max speed that you can transmit over UART, given the 48 MHz base clock.

This issue relates to the discrepancy between the README and the example code. The serial.v example uses 1Mb/s whereas the serial-echo.v example uses the 3Mb/s speed.

The serial.v example should be updated to 3Mb/s, or the README description should be changed to 1Mb/s.

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

No branches or pull requests

3 participants