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
vendor.intel: reset synchronisers #246
Conversation
Codecov Report
@@ Coverage Diff @@
## vendor.altera #246 +/- ##
=================================================
+ Coverage 82.33% 82.52% +0.18%
=================================================
Files 33 33
Lines 5514 5498 -16
Branches 1180 1180
=================================================
- Hits 4540 4537 -3
+ Misses 839 824 -15
- Partials 135 137 +2
Continue to review full report at Codecov.
|
nmigen/vendor/intel.py
Outdated
def get_reset_sync(self, reset_sync): | ||
m = Module() | ||
reset = Signal() | ||
reset_clk = ClockSignal(reset_sync._domain) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you can just use ClockSignal
and ResetSignal
directly in instance ports.
0b1afa2
to
ddcff57
Compare
I couldn't actually find any documentation on this, so I'm working based on the simulation model for this, and hoping for the best.