Browse files

lm32: fix watchpoints

The wp_match_n vector is off by one. Which results in undefined states, at
least in simulation.

Signed-off-by: Michael Walle <>
  • Loading branch information...
1 parent 6453d76 commit 3fd23151ac95543345801a463dcc7297b9031c9a @mwalle mwalle committed with sbourdeauducq Nov 29, 2012
Showing with 1 addition and 1 deletion.
  1. +1 −1 cores/lm32/rtl/lm32_debug.v
2 cores/lm32/rtl/lm32_debug.v
@@ -167,7 +167,7 @@ wire [0:breakpoints-1]bp_match_n; // Indicates if a h/w instructio
reg [`LM32_WPC_C_RNG] wpc_c[0:watchpoints-1]; // Watchpoint enable
reg [`LM32_WORD_RNG] wp[0:watchpoints-1]; // Watchpoint address
-wire [0:watchpoints]wp_match_n; // Indicates if a h/w data watchpoint matched
+wire [0:watchpoints-1]wp_match_n; // Indicates if a h/w data watchpoint matched
wire debug_csr_write_enable; // Debug CSR write enable (from either a wcsr instruction of external debugger)
wire [`LM32_WORD_RNG] debug_csr_write_data; // Data to write to debug CSR

0 comments on commit 3fd2315

Please sign in to comment.