From 7a1e4cb66b21ba0e0f7c18d2b4e7b1afd6776d95 Mon Sep 17 00:00:00 2001 From: Michael Walle Date: Fri, 30 Nov 2012 00:19:44 +0100 Subject: [PATCH] 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 --- verilog/lm32/lm32_debug.v | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/verilog/lm32/lm32_debug.v b/verilog/lm32/lm32_debug.v index e7f73cb16..5a22a2e0c 100644 --- a/verilog/lm32/lm32_debug.v +++ b/verilog/lm32/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