From 189dfd62e2385aa2f36c6283628b8d9285c49647 Mon Sep 17 00:00:00 2001 From: "Pavel V. Shatov (Meister)" Date: Tue, 19 Nov 2019 13:05:02 +0300 Subject: Removed the latch accidentally created while pipelining the uOP engine module. The FSM previously had four states encoded using two bits, so the next state logic didn't have a default case, since all the possible states were used. Addition of the fifth state required one more state bit, so the FSM now has five states out eight possible and a default case is thus necessary. --- rtl/modexpng_uop_engine.v | 1 + 1 file changed, 1 insertion(+) diff --git a/rtl/modexpng_uop_engine.v b/rtl/modexpng_uop_engine.v index 1119fbd..2046db7 100644 --- a/rtl/modexpng_uop_engine.v +++ b/rtl/modexpng_uop_engine.v @@ -678,6 +678,7 @@ module modexpng_uop_engine UOP_FSM_STATE_DECODE_1: uop_fsm_state_next = UOP_FSM_STATE_DECODE_2 ; UOP_FSM_STATE_DECODE_2: uop_fsm_state_next = uop_opcode_is_stop ? UOP_FSM_STATE_IDLE : UOP_FSM_STATE_BUSY; UOP_FSM_STATE_BUSY: uop_fsm_state_next = uop_exit_from_busy ? UOP_FSM_STATE_FETCH : UOP_FSM_STATE_BUSY; + default: uop_fsm_state_next = UOP_FSM_STATE_IDLE ; endcase // end -- cgit v1.2.3