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

arty_a7: support both the 35T and 100T SKUs #157

Merged
merged 1 commit into from May 30, 2021

Conversation

StephanvanSchaik
Copy link
Contributor

Similar to the Arty S7, the Arty A7 comes in two variants: one with the Xilinx Artix A7 XC7A35TI SKU and the other with the Xilinx Artix A7 XC7A100TI SKU. I have a board with the latter, and ran the Blinky test to be sure that the following commit does not contain any regressions.

Basically the commit follows the same structure as the arty_s7 board by changing ArtyA7Platform to _ArtyA7Platform and not setting the device constant. Then it introduces ArtyA7_35Platform and ArtyA7_100Platform for the two different variants, inheriting _ArtyA7Platform and setting the device constant to the appropriate SKU for that platform. Finally, the test has been changed to default to running Blinky on the ArtyA7_35Platform.

@whitequark whitequark merged commit e8611d4 into amaranth-lang:master May 30, 2021
@whitequark
Copy link
Member

Thanks!

@StephanvanSchaik StephanvanSchaik deleted the arty_a7 branch May 31, 2021 01:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants