Skip to content

Commit

Permalink
[conn,alert_esc] Check connectivity of escalation sender/receiver clk…
Browse files Browse the repository at this point in the history
… and rst_n

The escalation sender and receiver pairs need to connect to the same clock and
reset signals, or they can malfunction.

This adds a connectivity configuration for these tests, but it is expected to
fail due to the mis-connection of pwrmgr rst_ni described in lowRISC#23888.

Fixes lowRISC#14887

Signed-off-by: Guillermo Maturana <[email protected]>
  • Loading branch information
matutem committed Jul 2, 2024
1 parent 09b4736 commit 9cda769
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 1 deletion.
3 changes: 2 additions & 1 deletion hw/top_earlgrey/formal/chip_conn_cfg.hjson
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,8 @@

bbox_cmd: "[list aon_osc io_osc sys_osc usb_osc]"
conn_csvs_dir: "{proj_root}/hw/top_earlgrey/formal/conn_csvs"
conn_csvs: ["{conn_csvs_dir}/analog_sigs.csv",
conn_csvs: ["{conn_csvs_dir}/alert_handler_esc.csv",
"{conn_csvs_dir}/analog_sigs.csv",
"{conn_csvs_dir}/aon_timer_rst.csv",
"{conn_csvs_dir}/ast_infra.csv",
"{conn_csvs_dir}/ast_entropy_src_cfg.csv",
Expand Down
19 changes: 19 additions & 0 deletions hw/top_earlgrey/formal/conn_csvs/alert_handler_esc.csv
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
# Copyright lowRISC contributors (OpenTitan project).
# Licensed under the Apache License, Version 2.0, see LICENSE for details.
# SPDX-License-Identifier: Apache-2.0
#
# Run these checks with:
# ./util/dvsim/dvsim.py hw/top_earlgrey/formal/chip_conn_cfg.hjson

,NAME,SRC BLOCK,SRC SIGNAL,DEST BLOCK,DEST SIGNAL,,,,,,

# escalation sender and receivers
CONNECTION, ALERT_HANDLER_PWRMGR_ESC_CLK, top_earlgrey.u_alert_handler, clk_i, top_earlgrey.u_pwrmgr_aon.u_esc_rx, clk_i
# TODO(lowrisc/opentitan#23888) Enable this check once this issue is fixed.
# CONNECTION, ALERT_HANDLER_PWRMGR_ESC_RST, top_earlgrey.u_alert_handler, rst_ni, top_earlgrey.u_pwrmgr_aon.u_esc_rx, rst_ni
CONNECTION, ALERT_HANDLER_LC_CTRL_ESC0_CLK, top_earlgrey.u_alert_handler, clk_i, top_earlgrey.u_lc_ctrl.u_prim_esc_receiver0, clk_i
CONNECTION, ALERT_HANDLER_LC_CTRL_ESC0_RST, top_earlgrey.u_alert_handler, rst_ni, top_earlgrey.u_lc_ctrl.u_prim_esc_receiver0, rst_ni
CONNECTION, ALERT_HANDLER_LC_CTRL_ESC1_CLK, top_earlgrey.u_alert_handler, clk_i, top_earlgrey.u_lc_ctrl.u_prim_esc_receiver1, clk_i
CONNECTION, ALERT_HANDLER_LC_CTRL_ESC1_RST, top_earlgrey.u_alert_handler, rst_ni, top_earlgrey.u_lc_ctrl.u_prim_esc_receiver1, rst_ni
CONNECTION, ALERT_HANDLER_RV_CORE_IBEX_ESC_CLK, top_earlgrey.u_alert_handler, clk_i, top_earlgrey.u_rv_core_ibex.u_prim_esc_receiver, clk_i
CONNECTION, ALERT_HANDLER_RV_CORE_IBEX_ESC_CLK, top_earlgrey.u_alert_handler, rst_ni, top_earlgrey.u_rv_core_ibex.u_prim_esc_receiver, rst_ni

0 comments on commit 9cda769

Please sign in to comment.