Skip to content

Commit

Permalink
[ibex_register_file_fpga] Drop two confusing comments
Browse files Browse the repository at this point in the history
These were noticed by someone responding to issue #2230. I think the
author's original logic was to point out that there's a path from e.g.
raddr_a_i to rdata_a_o which doesn't depend on any clock, so is
"asynchronous".

But that's the same in the other modes and also for the other register
file implementations, which don't have analogous comments.

Drop these ones.
  • Loading branch information
rswarbrick committed Jan 23, 2025
1 parent 591c381 commit aff062d
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions rtl/ibex_register_file_fpga.sv
Original file line number Diff line number Diff line change
Expand Up @@ -150,10 +150,7 @@ module ibex_register_file_fpga #(
assign rdata_a_o = (raddr_a_i == '0) ? WordZeroVal : mem_o_a;
assign rdata_b_o = (raddr_b_i == '0) ? WordZeroVal : mem_o_b;
end else begin : gen_no_rdata_mux_check
// async_read a
assign rdata_a_o = (raddr_a_i == '0) ? WordZeroVal : mem[raddr_a_i];

// async_read b
assign rdata_b_o = (raddr_b_i == '0) ? WordZeroVal : mem[raddr_b_i];

assign oh_raddr_a_err = 1'b0;
Expand Down

0 comments on commit aff062d

Please sign in to comment.