Skip to content

Commit

Permalink
stream_arbiter_flushable: Do not lock prio arbiter (#222)
Browse files Browse the repository at this point in the history
Setting `LockIn` while `ExtPrio = 1'b1` is disallowed by the
`rr_arb_tree` and has no effect. Furthermore, locking a decision with a
possibly lower priority violates the intent of the priority arbiter.

Therefore, clear the `LockIn` parameter for the priority arbiter.

Signed-off-by: Nils Wistoff <[email protected]>
  • Loading branch information
niwis authored Jun 17, 2024
1 parent f4d6406 commit adda219
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/stream_arbiter_flushable.sv
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ module stream_arbiter_flushable #(
.DataType (DATA_T),
.ExtPrio (1'b1),
.AxiVldRdy (1'b1),
.LockIn (1'b1)
.LockIn (1'b0)
) i_arbiter (
.clk_i,
.rst_ni,
Expand Down

0 comments on commit adda219

Please sign in to comment.