[QuickStart] Yosys/Parmys Update #2776
Conversation
There was a problem hiding this comment.
Thanks @ZohairZaidi !
I have some comments about the parmys flow (I think we've listed some unnecessary steps for it) that you should take a look at closely. Adding @amirarjmand93 and @KennethKent to review; in case @AlexandreSinger has time to review I've tagged him too
|
Thanks @vaughnbetz!, I have addressed your suggestions. Will wait for the added members to review as well. |
There was a problem hiding this comment.
@ZohairZaidi Thank you for these changes! I like how you are running Yosys. I think using the vtr_flow script just on the Yosys step is perfectly fine since running Yosys directly is such a pain. The main goal is to know how to make the .blif file.
I have left some comments below, but overall the documentation is very good.
|
Adding @soheilshahrouz in case he can help figure out the required flow ... we need to document this well in the quickstart as Mohamed and I were confused about it. |
|
I agree @vaughnbetz, I plan to work on it this reading week. Please let me know what you were confused about. |
|
Hi @vaughnbetz, let me know how this is. I changed the order and made it smoother for the user to follow. |
doc/src/quickstart/index.rst
Outdated
| blink --circuit_file $VTR_FLOW_DIR/temp/blink.pre-vpr.blif \ | ||
| --route_chan_width 100 \ | ||
| --analysis --disp on | ||
|
|
||
|
|
||
| Running VPR |
There was a problem hiding this comment.
I think we should rename this to "Running VPR Manually"
and then add a first sentence after the heading that says
"Sometimes you may wish to run only the vpr (placement, routing and timing analysis) parts of the flow rather than the full VTR flow (which includes synthesis). To show how to do this, let's now try taking ..."
|
Looks good, thanks! |
Description
This pull request updates the VTR Quick Start Guide by replacing Odin II as the primary synthesis tool with the Yosys/Parmys flow. I have also updated the graphic images
Related Issue
closes #2739
Checklist: