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

Enable bitstream generation with flat routing #1866

Merged
merged 1 commit into from
Oct 31, 2024

Conversation

fkosar-ql
Copy link
Collaborator

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)

What does this pull request change?

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.

Copy link
Collaborator

@tangxifan tangxifan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fkosar-ql Good work on simplifying the codes. I have cleaned up the code conflicts with VTR latest and update the submodule to the latest.

I believe this PR can be merged after code formatting is resolved. The rest of the flat router can be in another PR.

Let me know if this works for you.

@fkosar-ql
Copy link
Collaborator Author

I think we can merge this

@tangxifan tangxifan merged commit fff30d5 into lnis-uofu:master Oct 31, 2024
38 checks passed
@tangxifan
Copy link
Collaborator

@fkosar-ql Merged. Please continue the effort on a follow-up PR. Thank you for the contribution.

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.

3 participants