Skip to content

Adding Support for the arty z7 20 and zybo z7 20 fpga boards #50

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

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

Xelef2000
Copy link

I want to add support for the Arty Z7 20 FPGA board.
This PR adds the constraints, adjusts the build script, and the top module.

I couldn't test the modified build script with the genesys2 since I do not have one.

@Xelef2000 Xelef2000 changed the title Adding Support for the arty z7 20 fpga board Adding Support for the arty z7 20 and zybo z7 20 fpga boards Apr 17, 2025
@phsauter phsauter self-requested a review April 19, 2025 00:24
@Xelef2000 Xelef2000 force-pushed the feat/add_arty_support branch from abf2f37 to f946c05 Compare May 7, 2025 09:17
@phsauter
Copy link
Collaborator

phsauter commented May 9, 2025

Quick update: I am planning to test your FPGA configurations in the next two weeks on all three boards.

@Xelef2000
Copy link
Author

Xelef2000 commented May 15, 2025

It looks like the rebase on croc 1.1 broke the FPGA synthesis.

ERROR: [Synth 8-11216] the number of actual arguments '3' differs from the number of formal arguments '2' for macro 'ASSERT_INIT' [/home/felix/Documents/Projects/FPGA/vlsi2/croc/rtl/common_cells/fifo_v3.sv:143]
ERROR: [Synth 8-10939] macro 'ASSERT_INIT' is defined here [/home/felix/Documents/Projects/FPGA/vlsi2/croc/rtl/cve2/include/lowrisc_prim/prim_assert_dummy_macros.svh:10]
ERROR: [Synth 8-11216] the number of actual arguments '5' differs from the number of formal arguments '4' for macro 'ASSERT' [/home/felix/Documents/Projects/FPGA/vlsi2/croc/rtl/common_cells/fifo_v3.sv:146]
ERROR: [Synth 8-10939] macro 'ASSERT' is defined here [/home/felix/Documents/Projects/FPGA/vlsi2/croc/rtl/cve2/include/lowrisc_prim/prim_assert_dummy_macros.svh:13]
ERROR: [Synth 8-11216] the number of actual arguments '5' differs from the number of formal arguments '4' for macro 'ASSERT' [/home/felix/Documents/Projects/FPGA/vlsi2/croc/rtl/common_cells/fifo_v3.sv:149]
ERROR: [Synth 8-439] module 'croc_xilinx' not found

@Xelef2000
Copy link
Author

the last commit should fix the issue

@phsauter phsauter added the enhancement New feature or request label Jun 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants