commit fa9ed4b0125c8b5dc0eca8f3d092ba14d310c548
author Cindy Chen <chencindy@opentitan.org> Thu Oct 21 17:27:39 2021 -0700
committer cindychip <cindy.chen0316@gmail.com> Mon Oct 25 19:01:26 2021 -0700
tree 6a52ea9deb74b1a79db49b5e33090299594da64f
parent 3400408d95acfc5852285e1b91914773b674520c

[dv/chip_level] Disable signal integrity check

Because of lpg and alert init are not fully supported in top-level
environment with alert_esc_agent, we will disable the alert's signal
integrity check.
This should have minimal impact because prim_receiver's assertions can
still check signal integrity error.
This also fixes issue #8715

Signed-off-by: Cindy Chen <chencindy@opentitan.org>

hw/dv/sv/cip_lib/cip_base_scoreboard.sv

diff --git a/hw/dv/sv/cip_lib/cip_base_scoreboard.sv b/hw/dv/sv/cip_lib/cip_base_scoreboard.sv
index 0c2af21..dd6cb2a 100644
--- a/hw/dv/sv/cip_lib/cip_base_scoreboard.sv
+++ b/hw/dv/sv/cip_lib/cip_base_scoreboard.sv
@@ -23,6 +23,7 @@
 
   // alert checking related parameters
   bit do_alert_check = 1;
+  bit check_alert_sig_int_err = 1;
   local bit under_alert_handshake[string];
   local bit exp_alert[string];
   local bit is_fatal_alert[string];
@@ -179,8 +180,10 @@
           if (item.alert_esc_type == AlertEscSigTrans && !item.ping_timeout &&
               item.alert_handshake_sta inside {AlertReceived, AlertAckComplete}) begin
             process_alert(alert_name, item);
-          // IP level alert protocol does not drive any sig_int_err or ping response
-          end else if (item.alert_esc_type == AlertEscIntFail) begin
+          // IP level alert protocol does not drive any sig_int_err or ping response.
+          // However, `lpg_en` or `alert_init` will trigger signal integrity error, user can
+          // disable signal integrity checking via `check_alert_sig_int_err` flag.
+          end else if (check_alert_sig_int_err && item.alert_esc_type == AlertEscIntFail) begin
             `uvm_error(`gfn, $sformatf("alert %s has unexpected signal int error", alert_name))
           end else if (item.ping_timeout) begin
             `uvm_error(`gfn, $sformatf("alert %s has unexpected timeout error", alert_name))

hw/top_earlgrey/dv/env/chip_scoreboard.sv

diff --git a/hw/top_earlgrey/dv/env/chip_scoreboard.sv b/hw/top_earlgrey/dv/env/chip_scoreboard.sv
index 22a2df8..2a8e4ac 100644
--- a/hw/top_earlgrey/dv/env/chip_scoreboard.sv
+++ b/hw/top_earlgrey/dv/env/chip_scoreboard.sv
@@ -31,6 +31,9 @@
   endfunction
 
   task run_phase(uvm_phase phase);
+    // Disable alert signal integrity check to avoid false alert on low_power_group_en or
+    // alert_init. Alert signal integrity can be checked by assertions.
+    check_alert_sig_int_err = 0;
     super.run_phase(phase);
     fork
       process_jtag_fifo();