-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Fix AXI4 Xbar elaboration error due to missing fields #3549
Conversation
|
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.
Can you rebase this PR on dev?
145356d
to
7f820d2
Compare
Unfortunately I need to wait a bit to get some feedback from my company on the CLA, so resolving that is going to take some time. |
On a recent
I worked around it by adding this line right above the in(i).w.bits.user := DontCare Not completely sure what the more principled way to solve this is, but I do see that |
7f820d2
to
1ec72d4
Compare
1ec72d4
to
2a3bd4e
Compare
Okay, I believe I've taken care of all the fields that need to be tied off in case they're not used by a particular destination. |
Checking on the status of this, anything else that I need to do before this can be merged @jerryz123 ? |
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.
Nope, this just slipped past me. Let's merge this
Thanks so much! |
Related issue: #3548
Type of change: bug report
Impact: no functional change
Development Phase: implementation
Release Notes