[dv] Address 2 comments in sec_cm update PR
For PR #10539
1. removed unused CntStyle
2. moved sec_cm_alert_name check to the top of the seq
Signed-off-by: Weicai Yang <weicai@google.com>
diff --git a/hw/dv/sv/cip_lib/seq_lib/cip_base_vseq__sec_cm_fi.svh b/hw/dv/sv/cip_lib/seq_lib/cip_base_vseq__sec_cm_fi.svh
index f18c02c..e2e9739 100644
--- a/hw/dv/sv/cip_lib/seq_lib/cip_base_vseq__sec_cm_fi.svh
+++ b/hw/dv/sv/cip_lib/seq_lib/cip_base_vseq__sec_cm_fi.svh
@@ -2,8 +2,11 @@
// Licensed under the Apache License, Version 2.0, see LICENSE for details.
// SPDX-License-Identifier: Apache-2.0
-// TODO, only support testing hardened count for now
virtual task run_sec_cm_fi_vseq(int num_times = 1);
+ `DV_CHECK_FATAL(cfg.sec_cm_alert_name inside {cfg.list_of_alerts},
+ $sformatf("sec_cm_alert_name (%s) is not inside %p",
+ cfg.sec_cm_alert_name, cfg.list_of_alerts))
+
pre_run_sec_cm_fi_vseq();
for (int trans = 1; trans <= num_times; trans++) begin
`uvm_info(`gfn, $sformatf("Running run_sec_cm_fi_vseq %0d/%0d", trans, num_times), UVM_LOW)
@@ -25,10 +28,6 @@
// - Verify any operations that follow fail (as applicable).
// refer to ip/keymgr/dv/env/seq_lib/keymgr_common_vseq.sv as an example
virtual task check_sec_cm_fi_resp(sec_cm_base_if_proxy if_proxy);
- `DV_CHECK_FATAL(cfg.sec_cm_alert_name inside {cfg.list_of_alerts},
- $sformatf("sec_cm_alert_name (%s) is not inside %p",
- cfg.sec_cm_alert_name, cfg.list_of_alerts))
-
`uvm_info(`gfn, $sformatf("expected fatal alert is triggered for %s", if_proxy.sec_cm_type.name),
UVM_LOW)
