-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
avoid latch #86
base: master
Are you sure you want to change the base?
avoid latch #86
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.
The failed test is due to flaky, shouldn't be a big problem.
@@ -131,6 +131,11 @@ def construct(s, NocPktType, DataType, data_mem_size_global, | |||
|
|||
@update | |||
def assemble_xbar_pkt(): | |||
for i in range(num_xbar_in_rd_ports): | |||
s.rd_pkt[i] @= TileSramXbarRdPktType(0,0,0) |
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.
(0,0,0)
- (0, 0, 0)
for i in range(num_xbar_in_rd_ports): | ||
s.rd_pkt[i] @= TileSramXbarRdPktType(0,0,0) | ||
for i in range(num_xbar_in_wr_ports): | ||
s.wr_pkt[i] @= TileSramXbarWrPktType(0,0,0) |
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.
ditto
Still wanna submit this PR @RuihongY? |
will do it after finish layout |
No description provided.