[go: nahoru, domu]

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

[Engine] Now GSB output file contains segments name and pin name in SB module #618

Merged
merged 2 commits into from
Apr 10, 2022

Conversation

tangxifan
Copy link
Collaborator
@tangxifan tangxifan commented Apr 10, 2022

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:

  • RR GSB only outputs rr_graph attributes, which is hard for PD team to sign-off.

What does this pull request change?

This PR improves in the following aspects:

  • Now GSB XML contains routing segment name from architecture file for X-direction and Y-direction routing tracks
  • Now GSB XML contains pin name (seen in SB module) from architecture file for X-direction and Y-direction routing tracks
  • Now GSB XML contains pin name (seen in SB module) from architecture file for output pins

An example output:
image

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 d018d07 into master Apr 10, 2022
@tangxifan tangxifan deleted the gsb_xml branch April 10, 2022 14:47
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.

None yet

1 participant