-
Notifications
You must be signed in to change notification settings - Fork 32
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
IO bitstream generation #1548
IO bitstream generation #1548
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1548 +/- ##
==========================================
+ Coverage 13.74% 13.85% +0.10%
==========================================
Files 473 473
Lines 60651 60767 +116
Branches 24029 24107 +78
==========================================
+ Hits 8339 8417 +78
- Misses 51343 51349 +6
- Partials 969 1001 +32 ☔ View full report in Codecov by Sentry. |
@NadeemYaseen , all macos failed because of python version - https://github.com/os-fpga/FOEDAG/actions/runs/8823274431/job/24223371267?pr=1548. But my changes does not change anything on workflows. |
@NadeemYaseen @chungshien-chai , We need to resolve the MacOS failures before merge |
It's related to Python version support in latest macOS on CI. Unless @nadeemyaseen-rs has a solution to use the version we need, we should pin the macOS to previous one for time being. |
This #1549 will hopefully fix the MacOS issue |
Changes include:
a. Provide API to validate parameter(s) before using it. (Use case: for example PLL need to validate the multiply and division params)
b. Prepare to support clock routing to gearbox. In the progress understanding RTL what to set in bits to support.
Testing:
a. ported code (together with os-fpga/yosys_verific_rs#626) to latest NS Raptor manually
b. run batch, batch_gen2 and batch_gen3 test.