commit dd87f4e85482c216bd8cdc555176a7a6a26e400d
author Weicai Yang <weicai@google.com> Tue Mar 30 17:33:39 2021 -0700
committer weicaiyang <49293026+weicaiyang@users.noreply.github.com> Thu Apr 01 13:39:25 2021 -0700
tree 3889b2dcbe897e2908e7aec229c8fbcbd7116bf5
parent 555451bb2b83399efdec11dca83916bf3ee207d8

[keymgr] Fix advance state when there is a HW input error

Update scb as #4899 is fixed

Signed-off-by: Weicai Yang <weicai@google.com>

hw/ip/keymgr/dv/env/keymgr_scoreboard.sv

diff --git a/hw/ip/keymgr/dv/env/keymgr_scoreboard.sv b/hw/ip/keymgr/dv/env/keymgr_scoreboard.sv
index 3584b05..547a77d 100644
--- a/hw/ip/keymgr/dv/env/keymgr_scoreboard.sv
+++ b/hw/ip/keymgr/dv/env/keymgr_scoreboard.sv
@@ -239,11 +239,12 @@
             if (get_fault_err()) begin
               current_state = keymgr_pkg::StDisabled;
               update_result = NotUpdate;
-            end else if (!get_op_err()) begin
+            end else if (get_op_err()) begin
+              update_result = NotUpdate;
+            end else begin
               // if it's StOwnerKey, it advacens to OpDisable. Key is just random value
               if (current_state == keymgr_pkg::StOwnerKey) update_result = NotUpdate;
               else                                         update_result = UpdateInternalKey;
-
               update_state(get_next_state(current_state));
               // set sw_binding_regwen after advance OP
               void'(ral.sw_binding_regwen.predict(.value(1)));