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 VCU118 block design #287

Merged
merged 17 commits into from
Jan 24, 2025
Merged

Add support for VCU118 block design #287

merged 17 commits into from
Jan 24, 2025

Conversation

CyrilKoe
Copy link
Contributor

@CyrilKoe CyrilKoe commented Jan 17, 2025

  • Added VCU118 block design and device tree
  • Removed half-done hyperram support (kept in other working branch) (pads are still in the Carfield IP but unconnected in the flow)
  • Added BD constraints for VCU118 (including ext_jtag)
  • Added CI boot for VCU118
  • Migrated CI to the new bordcomputer server

Tests:

  • Bare metal VCU118 (UART, JTAG, DDR, SPI)
  • Boot VCU118 (No ethernet [see docs]; but boot over SPI)

This PR requires the nonfree MR https://iis-git.ee.ethz.ch/carfield/carfield-nonfree/-/merge_requests/51

@CyrilKoe CyrilKoe self-assigned this Jan 17, 2025
@CyrilKoe CyrilKoe marked this pull request as ready for review January 18, 2025 10:21
@CyrilKoe CyrilKoe requested a review from alex96295 as a code owner January 18, 2025 10:21
@CyrilKoe CyrilKoe changed the title Draft: Add support for VCU118 block design Add support for VCU118 block design Jan 18, 2025
@CyrilKoe
Copy link
Contributor Author

Ready for review. Both FPGA boot in CI
https://iis-git.ee.ethz.ch/github-mirror/carfield/-/pipelines/96336

carfield.mk Show resolved Hide resolved
docs/tg/xilinx.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@alex96295 alex96295 left a comment

Choose a reason for hiding this comment

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

I added some minor comments, good job and thank you for the PR

As a side note, let's make sure we merge the nonfree branch before this

Copy link
Collaborator

@alex96295 alex96295 left a comment

Choose a reason for hiding this comment

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

LGTM

@CyrilKoe CyrilKoe merged commit 7908f99 into main Jan 24, 2025
7 checks passed
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