[ot] hw/opentitan: ot_lc_ctrl: Set LC_STATE to POST_TRANSITION after a transition

From <https://opentitan.org/book/hw/ip/lc_ctrl/doc/programmers_guide.html>:

"Note that all life cycle state transition increments the LC_TRANSITION_CNT
and moves the life cycle state into the temporary POST_TRANSITION state -
even if the transition was unsuccessful"

This is checked by manufacturing tests (on the host side).

Signed-off-by: Luís Marques <[email protected]>

Author: luismarques

hw/opentitan/ot_lc_ctrl.c

@@ -1285,6 +1285,7 @@ static void ot_lc_ctrl_handle_otp_ack(void *opaque, bool ack)
             s->regs[R_STATUS] |= R_STATUS_OTP_ERROR_MASK;
         }
         LC_FSM_CHANGE_STATE(s, ST_POST_TRANS);
+        s->lc_state = LC_STATE_POST_TRANSITION;
         break;
     default:
         g_assert_not_reached();
@@ -1304,6 +1305,7 @@ static void ot_lc_ctrl_program_otp(OtLcCtrlState *s, unsigned lc_tcount,
                       __func__);
         s->regs[R_STATUS] |= R_STATUS_OTP_ERROR_MASK;
         LC_FSM_CHANGE_STATE(s, ST_POST_TRANS);
+        s->lc_state = LC_STATE_POST_TRANSITION;
         return;
     }
 
@@ -1317,6 +1319,7 @@ static void ot_lc_ctrl_program_otp(OtLcCtrlState *s, unsigned lc_tcount,
         trace_ot_lc_ctrl_error(s->ot_id, "OTP program request rejected");
         s->regs[R_STATUS] |= R_STATUS_STATE_ERROR_MASK;
         LC_FSM_CHANGE_STATE(s, ST_POST_TRANS);
+        s->lc_state = LC_STATE_POST_TRANSITION;
         return;
     }
 }
@@ -1364,12 +1367,14 @@ static void ot_lc_ctrl_start_transition(OtLcCtrlState *s)
                                        "Invalid volatile unlock token");
                 s->regs[R_STATUS] |= R_STATUS_TOKEN_ERROR_MASK;
                 LC_FSM_CHANGE_STATE(s, ST_POST_TRANS);
+                s->lc_state = LC_STATE_POST_TRANSITION;
             }
         } else {
             trace_ot_lc_ctrl_error(s->ot_id,
                                    "Invalid state(s) for volatile unlock");
             s->regs[R_STATUS] |= R_STATUS_TRANSITION_ERROR_MASK;
             LC_FSM_CHANGE_STATE(s, ST_POST_TRANS);
+            s->lc_state = LC_STATE_POST_TRANSITION;
         }
         return;
     }
@@ -1409,6 +1414,7 @@ static void ot_lc_ctrl_start_transition(OtLcCtrlState *s)
         trace_ot_lc_ctrl_error(s->ot_id, "Max transition count reached");
         s->regs[R_STATUS] |= R_STATUS_TRANSITION_COUNT_ERROR_MASK;
         LC_FSM_CHANGE_STATE(s, ST_POST_TRANS);
+        s->lc_state = LC_STATE_POST_TRANSITION;
         return;
     }
 
@@ -1456,10 +1462,12 @@ static void ot_lc_ctrl_resume_transition(OtLcCtrlState *s)
         trace_ot_lc_ctrl_error(s->ot_id, "Invalid transition");
         s->regs[R_STATUS] |= R_STATUS_TRANSITION_ERROR_MASK;
         LC_FSM_CHANGE_STATE(s, ST_POST_TRANS);
+        s->lc_state = LC_STATE_POST_TRANSITION;
     } else if (!ot_lc_ctrl_match_token(s, token)) {
         trace_ot_lc_ctrl_error(s->ot_id, "Invalid OTP token");
         s->regs[R_STATUS] |= R_STATUS_TOKEN_ERROR_MASK;
         LC_FSM_CHANGE_STATE(s, ST_POST_TRANS);
+        s->lc_state = LC_STATE_POST_TRANSITION;
     } else {
         trace_ot_lc_ctrl_info(s->ot_id, "Valid token");