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

Add Support for Xilinx KCU116 #322

Merged
merged 6 commits into from
Mar 9, 2023
Merged

Conversation

mander1000
Copy link
Contributor

Hey

I added support for the Xilinx KCU116 board (Kintex Ultrascale Plus).

I tested it to flash both SPI flash chips on the board (QSPI), which worked fine.

Feel free to comment and let me know if you need anything else.

mander

Comment on lines 135 to 149
tool_options = {'part': part + '-1-e'}
parameters["secondaryflash"]= {
'datatype': 'int',
'paramtype': 'vlogdefine',
'description': 'secondary flash',
'default': 1}
elif part == "xcvu37p-fsvh2892":
tool_options = {'part': part + '-2L-e'}
elif part == "xcku5p-ffvb676":
tool_options = {'part': part + '-2-e'}
parameters["secondaryflash"]= {
'datatype': 'int',
'paramtype': 'vlogdefine',
'description': 'secondary flash',
'default': 1}
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not familiar about speedgrade for ultrascale+ but -1 or -2 are similar to equivalent for artix (for example)?
If so is it possible to use a low speed grade with a FPGA with highest grade?
The idea is to merge xcvu and xcku in the same test.
If I have true could you modify it (no need to rebuild the bitstream).
Thanks

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The naming for the speed grade is the same on all UltraScale devices (and I think even on all Xilinx devices). -1 is the slowest. So sure, I changed that. I combined the family name to Xilinx UltraScale as it checks for Virtex and Kintex devices. Hope that is ok.

@trabucayre
Copy link
Owner

LGTM!
Just this question/remark.
Another remark: could you rebase your PR: there is a conflict.
Thanks!

@mander1000
Copy link
Contributor Author

should all be done now and ready to merge. :)

@trabucayre trabucayre merged commit 4161c79 into trabucayre:master Mar 9, 2023
@trabucayre
Copy link
Owner

Applied. Thanks!

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

Successfully merging this pull request may close these issues.

2 participants