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

Support IPIN connections from all four sides incoming wires of the GSB #1946

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

ganeshgore
Copy link
Collaborator

Motivate of the pull request

  • To address an existing issue. If so, please provide a link to the issue:
  • (Non) Breaking new feature. If so, please provide details in the description section.

To ensure the abutment during physical design, the codebase enforces CBX connections from top and bottom and CBY connections from left and right. However, as the tile-based (CLB+CBX+CBY+SB) methodology is being developed, this constraint doesn't seem mandatory.

Describe the technical details

What is currently done? (Provide issue link if applicable)

What does this pull request change?

This pull iterates through all 4 sides to find SrcNode connected to any IPIN, to achieve this additional loops are added and some assertions are removed which are available here gg_dev_openfpga on VTR repo

This PR also fixes the GSB writer to print a side for IPIN driving nodes.

Which part of the code base require a change

  • VPR
  • Tileable routing architecture generator
  • OpenFPGA libraries
  • FPGA-Verilog
  • FPGA-Bitstream
  • FPGA-SDC
  • FPGA-SPICE
  • Flow scripts
  • Architecture library
  • Cell library
  • Documentation
  • Regression tests
  • Continous Integration (CI) scripts

Impact of the pull request

  • Require a change on Quality of Results (QoR)
  • Break back-compatibility. If so, please list who may be influenced.

@ganeshgore
Copy link
Collaborator Author

Fixed all failures

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant