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

Adding nextpnr routing timing support #111

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

gatecat
Copy link
Contributor

@gatecat gatecat commented Jul 15, 2021

This currently imports a simplified set of routing delays to nextpnr. Remaining TODOs:

  • site input timing
  • populating the cell pin timing structures

@gatecat gatecat requested a review from acomodi July 15, 2021 11:56
@gatecat gatecat marked this pull request as ready for review July 19, 2021 14:12
@gatecat gatecat changed the title [WIP] Adding nextpnr timing support Adding nextpnr routing timing support Jul 19, 2021
@gatecat
Copy link
Contributor Author

gatecat commented Jul 19, 2021

@acomodi even though cell delays aren't implemented yet, what do you think about merging just the routing delay part so we get some degree of timing in nextpnr?

Depending on the outcome of chipsalliance/fpga-interchange-schema#61 there might be a bit more work needed to get cell delays working (and we don't have the data for them for 7-series in any case).

@tmichalak
Copy link
Contributor

@gatecat I am in favor of merging the routing delay part first. BTW how did you test the current implementation?

@gatecat
Copy link
Contributor Author

gatecat commented Jul 29, 2021

BTW how did you test the current implementation?

Currently, by using --sdf with nextpnr to write an SDF file, and manually comparing against the delays in the Vivado GUI.

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