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

Updated Alveo U250 Constraints for Timing Issue #104

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aaqdas
Copy link

@aaqdas aaqdas commented Jan 7, 2025

Description

📝 Resolves the timing violation in synthesis for Alveo U250. The pin constraints for the reference clock gt0_refclk_p showed that reference clock is directly connected to a 156.25 MHz Clock. However, the clock is constrained to 322MHz in Timing Constraints.

Type of change

  • Bug fix
  • New feature
  • Documentation update
  • A new research paper code implementation
  • Other

Tests & Results

📝 Static Bitstream Generated and Linked with rdma_perf example shell. Timing report verified shows no violation. I can share the timing report separately if required for verification by maintainers

Checklist

  • I have commented my code and made corresponding changes to the documentation.
  • I have added tests/results that prove my fix is effective or that my feature works.
  • My changes generate no new warnings or errors & all tests successfully pass.

@bo3z
Copy link
Collaborator

bo3z commented Jan 8, 2025

Thanks @aaqdas for this. The change looks good to me; we will try to merge in the near future after we have done some more tests with the u250. We are currently focusing on some improvement functionality on the u280 / u55c.

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