Skip to content

Commit

Permalink
stream_arbiter_flushable: Do not lock prio arbiter
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 <nwistoff@iis.ee.ethz.ch>
  • Loading branch information
niwis committed May 15, 2024
1 parent 0d67563 commit 350b613
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 350b613

Please sign in to comment.