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

Tile.getWireConnections() not consistent with Vivado #1096

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

eddieh-xlnx
Copy link
Collaborator

Javadoc for Tile.getWireConnections() states that it should be equivalent to:

get_nodes -downhill -of_objects [get_nodes -of [get_wires -of $tile/$wireName]]]

Applying it in the following way on a xcvp1002 in Vivado:

get_nodes -downhill -of [get_nodes -of [get_wires CLK_REBUF_VERT_VNOC_BAO_TILE_X30Y471/IF_WRAP_CLK_V_BOT_CLK_VDISTR21]]
WARNING: [Vivado 12-2683] No nodes matched 'get_nodes -downhill -of [get_nodes -of [get_wires CLK_REBUF_VERT_VNOC_BAO_TILE_X30Y471/IF_WRAP_CLK_V_BOT_CLK_VDISTR21]]'

returns nothing.

But this test demonstrates that RapidWright returns:

Expected :[]
Actual   :[CLK_REBUF_VERT_VNOC_CBO_TILE_X30Y375/IF_WRAP_CLK_V_TOP_CLK_VDISTR21, INVALID1_X30Y431/null]
  1. CLK_REBUF_VERT_VNOC_CBO_TILE_X30Y375/IF_WRAP_CLK_V_TOP_CLK_VDISTR21 seems to be a wire on the original input node:
get_nodes -of [get_wires CLK_REBUF_VERT_VNOC_CBO_TILE_X30Y375/IF_WRAP_CLK_V_TOP_CLK_VDISTR21]
CLK_REBUF_VERT_VNOC_BAO_TILE_X30Y471/IF_WRAP_CLK_V_BOT_CLK_VDISTR21
  1. Obviously returning INVALID1_X30Y431/null is not desirable.

@eddieh-xlnx
Copy link
Collaborator Author

Oh, the other interesting point is that doing a get_wires instead of a get_nodes gives:

get_wires -downhill -of [get_nodes -of [get_wires CLK_REBUF_VERT_VNOC_BAO_TILE_X30Y471/IF_WRAP_CLK_V_BOT_CLK_VDISTR21]]
CLK_REBUF_VERT_VNOC_BAO_TILE_X30Y471/IF_WRAP_CLK_V_BOT_CLK_VDISTR21 CLK_REBUF_VERT_VNOC_CBO_TILE_X30Y375/IF_WRAP_CLK_V_TOP_CLK_VDISTR21

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