RWRoute RouteHelper fails to find the INT tile entry node from a BITSLICE_CONTROL site pin #558
Labels
Comments
clavin-xlnx
added a commit
that referenced
this issue
Oct 12, 2022
Signed-off-by: Chris Lavin <[email protected]>
eddieh-xlnx
pushed a commit
that referenced
this issue
Jan 12, 2024
Signed-off-by: Chris Lavin <[email protected]>
eddieh-xlnx
added a commit
that referenced
this issue
Jan 12, 2024
* Test for #558 Signed-off-by: Chris Lavin <[email protected]> * Adding comment header Signed-off-by: Chris Lavin <[email protected]> * Convert to known failing test Signed-off-by: Eddie Hung <[email protected]> --------- Signed-off-by: Chris Lavin <[email protected]> Signed-off-by: Eddie Hung <[email protected]> Co-authored-by: Eddie Hung <[email protected]>
eddieh-xlnx
added a commit
that referenced
this issue
Oct 10, 2024
Signed-off-by: Eddie Hung <[email protected]>
eddieh-xlnx
added a commit
that referenced
this issue
Oct 10, 2024
Signed-off-by: Eddie Hung <[email protected]>
eddieh-xlnx
added a commit
that referenced
this issue
Oct 10, 2024
…1081) * [RouterHelper] projectOutputPinToINTNode() to breadth-first-search Much like projectInputPinToINTNode() as this seems necessary for Versal I/Os Signed-off-by: Eddie Hung <[email protected]> * Missing import Signed-off-by: Eddie Hung <[email protected]> * Fix returned node Signed-off-by: Eddie Hung <[email protected]> * Fix projectInputPinToINTNode Signed-off-by: Eddie Hung <[email protected]> * Update src/com/xilinx/rapidwright/rwroute/RouterHelper.java Signed-off-by: eddieh-xlnx <[email protected]> * Inadvertently fixes #558 Signed-off-by: Eddie Hung <[email protected]> * projectOutputPinToINTNode() to skip nodes in clocking tiles Signed-off-by: Eddie Hung <[email protected]> * Reduce project{Input,Output}PinToINTNode() watchdog to 20 Signed-off-by: Eddie Hung <[email protected]> * Rework projectInputPinToINTNode(), ignore clocking nodes too Signed-off-by: Eddie Hung <[email protected]> --------- Signed-off-by: Eddie Hung <[email protected]> Signed-off-by: eddieh-xlnx <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The method
RouteHelper.projectOutputPinToINTNode()has not been tested in RWRoute forBITSLICE_CONTROLsite pins and does not find the INT tile entry node for the beginning of a route. For example, the site pinTX_T_OUTwhen queried, the method returnsnullwhen it should return the nodeINT_INTF_L_CMT_X182Y90/LOGIC_OUTS_R19. In Vivado, we can see that the INT entry node (shown in green) does exist:The method needs to account for the pip connection between the two nodes as shown in blue.
EDIT: For future contributors who will work on this -- a test demonstrating this failure is at #559
The text was updated successfully, but these errors were encountered: