Skip to content

Commit f73afc5

Browse files
[CSR] Clarify PMP config assignment
Functionally pmp_mseccfg and pmp_mseccfg_q are the same when PMP is enabled. However, the RTL is a bit more clear if both the MML and RLB register are referenced through the pmp_mseccfg_q register name. I thought of this clarification while analysing a question posed by Shutong Jin during the HACK@CHES'25 event.
1 parent b34d079 commit f73afc5

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

rtl/ibex_cs_registers.sv

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1183,7 +1183,7 @@ module ibex_cs_registers import ibex_pkg::*; #(
11831183
// When MSECCFG.MML is set cannot add new regions allowing M mode execution unless MSECCFG.RLB
11841184
// is set
11851185
assign pmp_cfg_wr_suppress[i] = pmp_mseccfg_q.mml &
1186-
~pmp_mseccfg.rlb &
1186+
~pmp_mseccfg_q.rlb &
11871187
is_mml_m_exec_cfg(pmp_cfg_wdata[i]);
11881188

11891189
// --------------------------

0 commit comments

Comments
 (0)