Skip to content

Commit

Permalink
Generate waves to FST format by default
Browse files Browse the repository at this point in the history
This format is better supported by gtkwave and allows enums values
to keep their string name when represented.
  • Loading branch information
teobiton committed Jan 26, 2024
1 parent 63fc12b commit 557037b
Show file tree
Hide file tree
Showing 9 changed files with 9 additions and 8 deletions.
1 change: 1 addition & 0 deletions tb/.gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,5 @@ __pycache__/
sim_build/

*.vcd
*.fst
*.xml
2 changes: 1 addition & 1 deletion tb/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ CORE_SOURCES := $(shell cat $(CORE_DIRECTORY)/Flist.$(TOPLEVEL))
VERILOG_SOURCES = $(addprefix $(ROOT_DIRECTORY)/, $(CORE_SOURCES)) $(ITF_DIRECTORY)/simple_reg_interface.sv

# Verilator extra arguments to build waves
EXTRA_ARGS += --trace --trace-structs
EXTRA_ARGS += --trace --trace-structs --trace-fst

include $(shell cocotb-config --makefiles)/Makefile.sim

2 changes: 1 addition & 1 deletion tb/interface/makefile
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ VERILOG_SOURCES = $(ITF_DIRECTORY)/$(VERILOG_FILE)
ITERATIONS := 10 # Number of random tests ran

# Verilator extra arguments to build waves
EXTRA_ARGS += --trace --trace-structs
EXTRA_ARGS += --trace --trace-structs --trace-fst

include $(shell cocotb-config --makefiles)/Makefile.sim

Expand Down
2 changes: 1 addition & 1 deletion tb/interface/test_simple_interface.py
Original file line number Diff line number Diff line change
Expand Up @@ -253,7 +253,7 @@ def test_interface_regs(DataWidth, BlockWidth, ByteAlign, DigestWidth):
extra_args: List[str] = []

if SIM == "verilator" and WAVES == "1":
extra_args = ["--trace", "--trace-structs"]
extra_args = ["--trace", "--trace-structs", "--trace-fst"]

parameters: Dict[str, str] = {}

Expand Down
2 changes: 1 addition & 1 deletion tb/sha1/makefile
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,6 @@ MODULE = test_sha1_core
VERILOG_SOURCES = $(SHA_DIRECTORY)/sha1_core.sv

# Verilator extra arguments to build waves
EXTRA_ARGS += --trace --trace-structs
EXTRA_ARGS += --trace --trace-structs --trace-fst

include $(shell cocotb-config --makefiles)/Makefile.sim
2 changes: 1 addition & 1 deletion tb/sha1/test_sha1_core.py
Original file line number Diff line number Diff line change
Expand Up @@ -264,7 +264,7 @@ def test_sha1_core():
extra_args: List[str] = []

if SIM == "verilator" and WAVES == "1":
extra_args = ["--trace", "--trace-structs"]
extra_args = ["--trace", "--trace-structs", "--trace-fst"]

parameters: Dict[str, str] = {}

Expand Down
2 changes: 1 addition & 1 deletion tb/sha2/makefile
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ MODULE = test_sha2_cores
VERILOG_SOURCES = $(SHA_DIRECTORY)/sha256_core.sv

# Verilator extra arguments to build waves
EXTRA_ARGS += --trace --trace-structs
EXTRA_ARGS += --trace --trace-structs --trace-fst

include $(shell cocotb-config --makefiles)/Makefile.sim

2 changes: 1 addition & 1 deletion tb/sha2/test_sha2_cores.py
Original file line number Diff line number Diff line change
Expand Up @@ -287,7 +287,7 @@ def test_sha256_core(core, DigestWidth):
extra_args: List[str] = []

if SIM == "verilator" and WAVES == "1":
extra_args = ["--trace", "--trace-structs"]
extra_args = ["--trace", "--trace-structs", "--trace-fst"]

parameters: Dict[str, str] = {}

Expand Down
2 changes: 1 addition & 1 deletion tb/test_tops.py
Original file line number Diff line number Diff line change
Expand Up @@ -373,7 +373,7 @@ def test_ip(ip, DigestWidth):
extra_args: List[str] = []

if SIM == "verilator" and WAVES == "1":
extra_args = ["--trace", "--trace-structs"]
extra_args = ["--trace", "--trace-structs", "--trace-fst"]

parameters: Dict[str, str] = {}

Expand Down

0 comments on commit 557037b

Please # to comment.