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

Known failing test for Tile.getSites() result different to Vivado #745

Merged
merged 2 commits into from
Jan 12, 2024

Conversation

eddieh-xlnx
Copy link
Collaborator

On VU3P:

get_sites -of [get_tiles LAG_LAG_X30Y250]
WARNING: [Vivado 12-3731] No sites matched '*'

On VU5P:

get_sites -of [get_tiles LAG_LAG_X30Y250]
LAGUNA_X6Y140 LAGUNA_X6Y141 LAGUNA_X7Y140 LAGUNA_X7Y141

whereas RapidWright currently returns sites for all four testcases (the first three of which is incorrect).

Discovered during the analysis of #738 which uses the VU3P (not that nextpnr should place a regular FDRE onto a Laguna BEL, even if there was a valid site)

@eddieh-xlnx eddieh-xlnx changed the title Tile.getSites() result different to Vivado Known failing test for Tile.getSites() result different to Vivado Jan 12, 2024
@eddieh-xlnx eddieh-xlnx marked this pull request as ready for review January 12, 2024 20:01
@eddieh-xlnx eddieh-xlnx merged commit bd560a1 into master Jan 12, 2024
14 checks passed
@eddieh-xlnx eddieh-xlnx deleted the sites_of_tile branch January 12, 2024 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants