-
Notifications
You must be signed in to change notification settings - Fork 10
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
nextpnr-xilinx: add computation of needed data files #89
Conversation
Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
585b6bd
to
003f761
Compare
Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I get this error during the nextpnr-xilinx build:
This did not happen previously. Probably due to the addition of pypy3? |
Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
@mithro the package now contains the required files:
|
@acomodi - Should there be xc7a200t and other similar files there too? |
Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
…wrong version Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
@mithro Added the remaining devices. Unfortunately 200t cannot be used as the submodule to prjxray-db is pointing to an old version, that needs to change upstream. |
65ba014
to
5a5f810
Compare
- yosys | ||
- pypy3.6 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The dependency on pypy3.6
worries me a little. Let's merge and log an issue to follow up on that.
The contents output are now;
|
Signed-off-by: Alessandro Comodi acomodi@antmicro.com
This PR adds the computation of the .bba and .bin files for nextpnr-xilinx