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 Negative Edge Flip-flop #647

Merged
merged 16 commits into from
May 9, 2022
Merged

Support Negative Edge Flip-flop #647

merged 16 commits into from
May 9, 2022

Conversation

tangxifan
Copy link
Collaborator

@tangxifan tangxifan commented May 9, 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:

  • No examples about how to use negative edge triggered flip-flops

What does this pull request change?

This PR improves in the following aspects:

  • Added a new circuit design of multi-mode flip-flop which supports negative-edge and positive edge
  • Added a regression test for compiling verilog files in cell library
  • Added an architecture which supported both positive edge and negative edge triggered flip-flops
  • Added a counter design which is triggered by negative edge
  • Added a test case to validate the newly added benchmark and architecture

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
Copy link
Collaborator Author

Waveform of the test case:

image

@tangxifan tangxifan merged commit 2b05238 into master May 9, 2022
@tangxifan tangxifan deleted the negedge_ff branch May 9, 2022 11:33
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