Skip to content

Commit 7a4697f

Browse files
committed
[rtl,cheriot] Qualify CheriotWaddrMSBClear assertion with we_a_i
Signed-off-by: Samuel Riedel <sriedel@lowrisc.org>
1 parent b0c13b8 commit 7a4697f

3 files changed

Lines changed: 3 additions & 3 deletions

File tree

‎rtl/ibex_register_file_ff.sv‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,7 @@ module ibex_register_file_ff import ibex_pkg::*; #(
221221

222222
// In CHERIoT mode all register addresses are 4-bit (implicit E extension). Hence, the MSB must
223223
// never be set. The bank-select bit is anyway forced to 0.
224-
`ASSERT(CheriotWaddrMSBClear, cheriot_enabled |-> !waddr_a_i[4])
224+
`ASSERT(CheriotWaddrMSBClear, (cheriot_enabled && we_a_i) |-> !waddr_a_i[4])
225225
`ASSERT(CheriotRaddrAMSBClear, cheriot_enabled |-> !raddr_a_i[4])
226226
`ASSERT(CheriotRaddrBMSBClear, cheriot_enabled |-> !raddr_b_i[4])
227227

‎rtl/ibex_register_file_fpga.sv‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ module ibex_register_file_fpga import ibex_pkg::*; #(
128128

129129
// In CHERIoT mode all register addresses are 4-bit (implicit E extension). Hence, the MSB must
130130
// never be set. The bank-select bit is anyway forced to 0.
131-
`ASSERT(CheriotWaddrMSBClear, cheriot_enabled |-> !waddr_a_i[4])
131+
`ASSERT(CheriotWaddrMSBClear, (cheriot_enabled && we_a_i) |-> !waddr_a_i[4])
132132
`ASSERT(CheriotRaddrAMSBClear, cheriot_enabled |-> !raddr_a_i[4])
133133
`ASSERT(CheriotRaddrBMSBClear, cheriot_enabled |-> !raddr_b_i[4])
134134

‎rtl/ibex_register_file_latch.sv‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -270,7 +270,7 @@ module ibex_register_file_latch import ibex_pkg::*; #(
270270

271271
// In CHERIoT mode all register addresses are 4-bit (implicit E extension). Hence, the MSB must
272272
// never be set. The bank-select bit is anyway forced to 0.
273-
`ASSERT(CheriotWaddrMSBClear, cheriot_enabled |-> !waddr_a_i[4])
273+
`ASSERT(CheriotWaddrMSBClear, (cheriot_enabled && we_a_i) |-> !waddr_a_i[4])
274274
`ASSERT(CheriotRaddrAMSBClear, cheriot_enabled |-> !raddr_a_i[4])
275275
`ASSERT(CheriotRaddrBMSBClear, cheriot_enabled |-> !raddr_b_i[4])
276276

0 commit comments

Comments
 (0)