Skip to content

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 added 3 commits July 15, 2021 11:02
Signed-off-by: gatecat <gatecat@ds0.me>
Signed-off-by: gatecat <gatecat@ds0.me>
Signed-off-by: gatecat <gatecat@ds0.me>
gatecat added 2 commits July 15, 2021 12:31
Signed-off-by: gatecat <gatecat@ds0.me>
Signed-off-by: gatecat <gatecat@ds0.me>
@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