Skip to content

Commit

Permalink
[rv_dm,dv] Slightly broaden dependency in rv_dm_env.core
Browse files Browse the repository at this point in the history
When the core file originally got a dependency on the IP itself (in
1f3a0e5), the tests were only actually depending on the debug
module package. But that's not true any more! In particular,
rv_dm_env_pkg now depends on rv_dm_reg_pkg (since commit 840a245).

Depend on hw:ip:rv_dm to make this explicit (rather than depending on
the ordering fusesoc happens to choose, which doesn't necessarily get
what we need).

Signed-off-by: Rupert Swarbrick <rswarbrick@lowrisc.org>
  • Loading branch information
rswarbrick committed Oct 10, 2024
1 parent 5018ada commit 9cd5326
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions hw/ip/rv_dm/dv/env/rv_dm_env.core
Original file line number Diff line number Diff line change
Expand Up @@ -7,15 +7,13 @@ description: "RV_DM DV UVM environment"
filesets:
files_dv:
depend:
- lowrisc:ip:rv_dm
- lowrisc:dv:ralgen
- lowrisc:dv:cip_lib
- lowrisc:dv:dv_base_reg
- lowrisc:dv:jtag_agent
- lowrisc:dv:jtag_dmi_agent
- lowrisc:opentitan:bus_params_pkg
# Note: This core pulls in an rv_dm implementation. We actually
# just need the package (dm_pkg.sv) for DV here.
- pulp-platform:riscv-dbg:0.1
files:
- rv_dm_env_pkg.sv
- rv_dm_if.sv
Expand Down

0 comments on commit 9cd5326

Please sign in to comment.