Skip to content

Commit

Permalink
Re-enable Icarus tests
Browse files Browse the repository at this point in the history
Signed-off-by: Tomasz Gorochowik <[email protected]>
  • Loading branch information
tgorochowik committed Dec 16, 2024
1 parent 20c9c60 commit 7569076
Show file tree
Hide file tree
Showing 2 changed files with 27 additions and 18 deletions.
10 changes: 10 additions & 0 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,16 @@ jobs:
run: |
./install.sh
- name: Build Icarus
run: |
git clone https://github.com/steveicarus/iverilog.git -b 06077ed026b4d8498d9129a6acfb0b44d0b51f18
cd iverilog
sh autoconf.sh
./configure
make -j$(nproc)
sudo make install
iverilog -V
- name: Build Verilator
run: |
git clone https://github.com/verilator/verilator -b v5.024
Expand Down
35 changes: 17 additions & 18 deletions verification/cocotb/noxfile.py
Original file line number Diff line number Diff line change
Expand Up @@ -242,24 +242,23 @@ def ctrl_bus_monitor_verify(session, test_group, test_name, coverage):
verify_block(session, test_group, test_name, coverage)


# TODO: Install iverilog in CI and reenable
# @nox.session(tags=["tests", "ahb", "axi"])
# @nox.parametrize("test_group", ["i3c_phy_io"])
# @nox.parametrize(
# "simulator",
# [
# "icarus",
# ],
# )
# @nox.parametrize(
# "test_name",
# [
# "test_drivers",
# ],
# )
# @nox.parametrize("coverage", coverage_types)
# def i3c_phy_io_verify(session, test_group, test_name, coverage, simulator):
# verify_block(session, test_group, test_name, coverage, simulator)
@nox.session(tags=["tests", "ahb", "axi"])
@nox.parametrize("test_group", ["i3c_phy_io"])
@nox.parametrize(
"simulator",
[
"icarus",
],
)
@nox.parametrize(
"test_name",
[
"test_drivers",
],
)
@nox.parametrize("coverage", coverage_types)
def i3c_phy_io_verify(session, test_group, test_name, coverage, simulator):
verify_block(session, test_group, test_name, coverage, simulator)


@nox.session(reuse_venv=True)
Expand Down

0 comments on commit 7569076

Please sign in to comment.