Skip to content
Snippets Groups Projects
Commit 6eeb0eb1 authored by Michael Schaffner's avatar Michael Schaffner Committed by Florian Zaruba
Browse files

testharness: Remove spurious warnings in Verilator

Modify warnings in testharness such that they only trigger on active
clock edge (resolves an issue with verilator)
parent 7c085cda
Branches
Tags
No related merge requests found
...@@ -52,18 +52,18 @@ endif ...@@ -52,18 +52,18 @@ endif
# Sources # Sources
# Package files -> compile first # Package files -> compile first
ariane_pkg := include/riscv_pkg.sv \ ariane_pkg := include/riscv_pkg.sv \
src/riscv-dbg/src/dm_pkg.sv \ src/riscv-dbg/src/dm_pkg.sv \
include/ariane_pkg.sv \ include/ariane_pkg.sv \
include/std_cache_pkg.sv \ include/std_cache_pkg.sv \
include/wt_cache_pkg.sv \ include/wt_cache_pkg.sv \
src/axi/src/axi_pkg.sv \ src/axi/src/axi_pkg.sv \
src/register_interface/src/reg_intf.sv \ src/register_interface/src/reg_intf.sv \
src/register_interface/src/reg_intf_pkg.sv \ src/register_interface/src/reg_intf_pkg.sv \
include/axi_intf.sv \ include/axi_intf.sv \
tb/ariane_soc_pkg.sv \ tb/ariane_soc_pkg.sv \
include/ariane_axi_pkg.sv \ include/ariane_axi_pkg.sv \
src/fpu/src/fpnew_pkg.sv \ src/fpu/src/fpnew_pkg.sv \
src/fpu/src/fpu_div_sqrt_mvp/hdl/defs_div_sqrt_mvp.sv src/fpu/src/fpu_div_sqrt_mvp/hdl/defs_div_sqrt_mvp.sv
ariane_pkg := $(addprefix $(root-dir), $(ariane_pkg)) ariane_pkg := $(addprefix $(root-dir), $(ariane_pkg))
# utility modules # utility modules
...@@ -368,7 +368,7 @@ verilate_command := $(verilator) ...@@ -368,7 +368,7 @@ verilate_command := $(verilator)
$(list_incdir) --top-module ariane_testharness \ $(list_incdir) --top-module ariane_testharness \
--Mdir $(ver-library) -O3 \ --Mdir $(ver-library) -O3 \
--exe tb/ariane_tb.cpp tb/dpi/SimDTM.cc tb/dpi/SimJTAG.cc \ --exe tb/ariane_tb.cpp tb/dpi/SimDTM.cc tb/dpi/SimJTAG.cc \
tb/dpi/remote_bitbang.cc tb/dpi/msim_helper.cc tb/dpi/remote_bitbang.cc tb/dpi/msim_helper.cc
# User Verilator, at some point in the future this will be auto-generated # User Verilator, at some point in the future this will be auto-generated
verilate: verilate:
......
...@@ -670,17 +670,17 @@ module ariane_testharness #( ...@@ -670,17 +670,17 @@ module ariane_testharness #(
// Simulation Helper Functions // Simulation Helper Functions
// ------------- // -------------
// check for response errors // check for response errors
always_comb begin always_ff @(posedge clk_i) begin : p_assert
if (axi_ariane_req.r_ready && if (axi_ariane_req.r_ready &&
axi_ariane_resp.r_valid && axi_ariane_resp.r_valid &&
axi_ariane_resp.r.resp inside {axi_pkg::RESP_DECERR, axi_pkg::RESP_SLVERR}) begin axi_ariane_resp.r.resp inside {axi_pkg::RESP_DECERR, axi_pkg::RESP_SLVERR}) begin
$warning("R Response Errored"); $warning("R Response Errored");
end end
if (axi_ariane_req.b_ready && if (axi_ariane_req.b_ready &&
axi_ariane_resp.b_valid && axi_ariane_resp.b_valid &&
axi_ariane_resp.b.resp inside {axi_pkg::RESP_DECERR, axi_pkg::RESP_SLVERR}) begin axi_ariane_resp.b.resp inside {axi_pkg::RESP_DECERR, axi_pkg::RESP_SLVERR}) begin
$warning("B Response Errored"); $warning("B Response Errored");
end end
end end
`ifdef AXI_SVA `ifdef AXI_SVA
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment