Skip to content

Commit

Permalink
[prim,rtl] Remove ~under_rst from the expression for fifo_incr_wptr
Browse files Browse the repository at this point in the history
This term can only be necessary if wready_o is true. But wready_o is
driven by the following continuous assignment:

    assign wready_o = ~full_o & ~under_rst;

This commit doesn't add an assertion (to check the behaviour doesn't
change) because the behaviour is internal to the module.

Signed-off-by: Rupert Swarbrick <[email protected]>
  • Loading branch information
rswarbrick committed Jan 16, 2025
1 parent 140185f commit 8c54914
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion hw/ip/prim/rtl/prim_fifo_sync.sv
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,7 @@ module prim_fifo_sync #(
.depth_o,
.err_o
);
assign fifo_incr_wptr = wvalid_i & wready_o & ~under_rst;
assign fifo_incr_wptr = wvalid_i & wready_o;
assign fifo_incr_rptr = rvalid_o & rready_i & ~under_rst;

logic [Depth-1:0][Width-1:0] storage;
Expand Down

0 comments on commit 8c54914

Please sign in to comment.