axi_jesd204: add missing ASYNC_REG #1881
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR Description
This patch resolves an "Asynchronous reset synchronized with missing ASYNC_REG property" CDC warning.
All the other reset synchronization chains in
axi_jesd204_[rx|tx]
haveASYNC_REG
set. Just looks likedevice_reset_vector_reg
was forgotten.I've only tested
axi_jesd204_tx
. I tried building one of the projects that use bothjesd204_rx
andjesd204_tx
, but they all seem to require Vivado licenses I don't have.PR Type
PR Checklist