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 different routing channel width in X and and Y axis #1313

Merged
merged 4 commits into from
Aug 23, 2023

Conversation

tangxifan
Copy link
Collaborator

@tangxifan tangxifan commented Aug 22, 2023

Motivate of the pull request

  • To address an existing issue. If so, please provide a link to the issue:
  • Breaking new feature. If so, please describe details in the description part.

Describe the technical details

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

Currently, OpenFPGA has the following limitations:

  • VPR supports different routing channel width in X and Y axis. For example, X-direction routing channel width is 30 while Y-direction routing channel width is 20. This is very useful to alleviate routing congestion when some programmable blocks have a large number of pins to access routing on some sides.
  • VPR also support different wire distribution in X and Y axis routing channels. For example, X-direction routing channel width is 30 but contains 80% L1 and 20% L4, while Y-direction routing channel width is 20 but contains 80% L1 and 20% L8. This is very useful in optimizing routing architecture when some programmable block pins require different wire lengths.

However, these features are yet tested in OpenFPGA!!!

What does this pull request change?

This PR improves in the following aspects:

  • Update openfpga to support the two features above.
  • Added two testcases to validate the new features.

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.

@tangxifan tangxifan merged commit 2d57099 into master Aug 23, 2023
72 checks passed
@tangxifan tangxifan deleted the xt_chan_dist branch August 23, 2023 01:35
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