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

examples/usb/acm_serial does not meet timing requirements on FomuPVT (Max 46.82 MHz) #80

Closed
wants to merge 3 commits into from

Conversation

umarcor
Copy link

@umarcor umarcor commented Dec 23, 2020

Currently, the CI in this repo seems not to be executed. Workflows are triggered, but tox runs no commands. The first commit in this PR is a quick fix for that, which enables tests on py37, py38 and py39.

The second commit shows an issue when trying to build example usb/acm_serial for Fomu (PVT version). Timing is not met, but it's really close:

ERROR: Max frequency for clock              'car_clk': 46.82 MHz (FAIL at 48.00 MHz)

Was this tested before? Is the example expected to work on Fomu?

@umarcor umarcor marked this pull request as ready for review December 23, 2020 02:30
@umarcor umarcor marked this pull request as draft December 23, 2020 02:30
@ktemkin ktemkin self-assigned this Jan 6, 2021
Base automatically changed from master to main March 9, 2021 14:49
@umarcor
Copy link
Author

umarcor commented Jul 10, 2021

I rebased this PR. It seems that some synthesis/PnR steps are executed, but it cannot find the bitstream: https://github.com/umarcor/luna/runs/3037219991?check_suite_focus=true#step:4:315.

@straithe straithe requested review from miek and mossmann August 5, 2021 21:29
@straithe straithe added the enhancement potential new feature label Nov 4, 2021
@miek
Copy link
Member

miek commented Nov 16, 2021

The CI not running was fixed in 2a0116c and the timings were improved for iCE40 in #133. The acm_serial example on FomuPVT builds successfully for me now, so I'm closing this. Feel free to open an issue if you have any more trouble with it though, thanks!

@miek miek closed this Nov 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement potential new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants