-
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
Add IO Connections for Custom User Field in TL Channels within Xbar #3637
base: master
Are you sure you want to change the base?
Conversation
module Please refer to the issue page below. ucb-bar/chipyard#1888
The committers listed above are authorized under a signed CLA. |
@jerryz123 Hi Jerry, I've made a PR in the rocket-chip for the issue I reported to the chipyard. Could you please review it? |
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.
Is removing the := DontCare
sufficient? The x.waiveAll :<>= y.waiveAll
pattern should connect the user fields
@jerryz123 I'm not sure what waiveAll does exactly, but when I remove Could you please guide me a better way to connect userfield than I used? |
@sequencer can you assist? |
Sure, will have a try this weekend. |
Thanks @sequencer for you help. Let me know if you have anything from me. |
Please refer to the issue page below.
ucb-bar/chipyard#1888
Related issue: ucb-bar/chipyard#1888
Type of change: bug report
Impact: Affect custom user field of TL Channels
Development Phase: proposal | implementation
Release Notes