[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

Added create-task shortcut in openfpga shell scripts #635

Merged
merged 4 commits into from
May 3, 2022
Merged

Conversation

ganeshgore
Copy link
Collaborator

Motivate of the pull request

This enables users to create a sample OpenFPGA project quickly.

Describe the technical details

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

What does this pull request change?

I added a create_task shortcut in openfpga.sh file
This shortcut copies one of the template tasks defined in the openfpga_flow to the current directory.
There are two template projects created for now, vpr_blif and yosys_vpr,
these projects are portable and contain all the required files in the task directory

source openfpga.sh 
create_task _my_openfpga_task  # This will create  a example task default is yosys_vpr

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.

@ganeshgore Any regression tests required for this feature?

@ganeshgore
Copy link
Collaborator Author

Added

@tangxifan tangxifan merged commit e8a2614 into master May 3, 2022
@tangxifan tangxifan deleted the binder branch May 3, 2022 06:53
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

2 participants