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

Now .act file is no longer required in openfpga_flow/task when power analysis option is off #737

Merged
merged 7 commits into from
Aug 2, 2022

Conversation

tangxifan
Copy link
Collaborator

@tangxifan tangxifan commented Aug 2, 2022

Motivate of the pull request

Describe the technical details

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

What does this pull request change?

This PR improves in the following aspects:

  • Removed the requirements where .act file is already checked when even power analysis option is off in a task configuration file.
  • Now variable ${TOP_MODULE} in template scripts of openfpga shell can be replaced by the top-level module name defined in task configuration file.
  • Added a dedicated testcase to validate the correctness

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 changed the title [WIP] now .act file is no longer required in openfpga_flow/task wh… [WIP] now .act file is no longer required in openfpga_flow/task when power analysis option is off Aug 2, 2022
@github-actions github-actions bot added the tests label Aug 2, 2022
@tangxifan tangxifan changed the title [WIP] now .act file is no longer required in openfpga_flow/task when power analysis option is off Now .act file is no longer required in openfpga_flow/task when power analysis option is off Aug 2, 2022
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.

Remove the hard requirement on signal activity file in the OpenFPGA vpr_blif flow
2 participants