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 HDIOB* site types to IOB types #1028

Merged
merged 1 commit into from
Jul 12, 2024
Merged

Adding HDIOB* site types to IOB types #1028

merged 1 commit into from
Jul 12, 2024

Conversation

clavin-xlnx
Copy link
Member

This resolves an issue whenever exporting a design via the Interchange which uses HDIOB* IO types in the design. For example, this error occurred while converting a VU19P design:

Exception in thread "main" java.lang.RuntimeException: Unable to process PORT cell at site IOB_X2Y212
	at com.xilinx.rapidwright.interchange.PhysNetlistWriter.extractIntraSiteRouting(PhysNetlistWriter.java:395)
	at com.xilinx.rapidwright.interchange.PhysNetlistWriter.buildNet(PhysNetlistWriter.java:321)
	at com.xilinx.rapidwright.interchange.PhysNetlistWriter.writePhysNetsRange(PhysNetlistWriter.java:248)
	at com.xilinx.rapidwright.interchange.PhysNetlistWriter.writePhysNets(PhysNetlistWriter.java:237)
	at com.xilinx.rapidwright.interchange.PhysNetlistWriter.writePhysNetlist(PhysNetlistWriter.java:714)
	at com.xilinx.rapidwright.interchange.DcpToInterchange.main(DcpToInterchange.java:50)
	at com.xilinx.rapidwright.MainEntrypoint.main(MainEntrypoint.java:223)

Signed-off-by: Chris Lavin <[email protected]>
@clavin-xlnx clavin-xlnx requested a review from eddieh-xlnx July 12, 2024 17:24
@clavin-xlnx clavin-xlnx merged commit a55fac3 into master Jul 12, 2024
14 checks passed
@clavin-xlnx clavin-xlnx deleted the hdiob branch July 12, 2024 23:08
eddieh-xlnx added a commit that referenced this pull request Jul 15, 2024
* [RWRoute] Do not assume Y = 0 has Laguna tiles, since it could be HBM device (#1026)

* Do not assume Y = 0 has Laguna tiles, since it could be HBM device

Signed-off-by: Eddie Hung <[email protected]>

* Add test for xcu50

Signed-off-by: Eddie Hung <[email protected]>

---------

Signed-off-by: Eddie Hung <[email protected]>

* [RWRoute] Do not NPE on encrypted netlists (#1025)

* Do not NPE on encrypted netlists

Signed-off-by: Eddie Hung <[email protected]>

* Apply suggestions from code review

Co-authored-by: Chris Lavin <[email protected]>
Signed-off-by: eddieh-xlnx <[email protected]>

---------

Signed-off-by: Eddie Hung <[email protected]>
Signed-off-by: eddieh-xlnx <[email protected]>
Co-authored-by: Chris Lavin <[email protected]>

* Test that PIP.getArbitraryPIP() sets the reversed flag correctly

Signed-off-by: Eddie Hung <[email protected]>

* Add TestNode.getAll{Uphill,Downhill}PIPsReversed() tests

Signed-off-by: Eddie Hung <[email protected]>

* Adding HDIOB types (#1028)

Signed-off-by: Chris Lavin <[email protected]>

* [EDIFTools] writeTclLoadScriptForPartialEncryptedDesigns abspath (#1029)

Write *_load.tcl into same dir as DCP rather than into current
working dir

Signed-off-by: Eddie Hung <[email protected]>

* Test that PIP.getArbitraryPIP() sets the reversed flag correctly

Signed-off-by: Eddie Hung <[email protected]>

* Add TestNode.getAll{Uphill,Downhill}PIPsReversed() tests

Signed-off-by: Eddie Hung <[email protected]>

---------

Signed-off-by: Eddie Hung <[email protected]>
Signed-off-by: eddieh-xlnx <[email protected]>
Signed-off-by: Chris Lavin <[email protected]>
Co-authored-by: Chris Lavin <[email protected]>
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