Skip to content

Commit cecf4fd

Browse files
committed
[ibex_register_file_fpga] Drop two confusing comments
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.
1 parent 591c381 commit cecf4fd

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

rtl/ibex_register_file_fpga.sv

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -150,10 +150,7 @@ module ibex_register_file_fpga #(
150150
assign rdata_a_o = (raddr_a_i == '0) ? WordZeroVal : mem_o_a;
151151
assign rdata_b_o = (raddr_b_i == '0) ? WordZeroVal : mem_o_b;
152152
end else begin : gen_no_rdata_mux_check
153-
// async_read a
154153
assign rdata_a_o = (raddr_a_i == '0) ? WordZeroVal : mem[raddr_a_i];
155-
156-
// async_read b
157154
assign rdata_b_o = (raddr_b_i == '0) ? WordZeroVal : mem[raddr_b_i];
158155

159156
assign oh_raddr_a_err = 1'b0;

0 commit comments

Comments
 (0)