Skip to content

Commit

Permalink
[rv_dm,dv] Fix condition for check in scoreboard
Browse files Browse the repository at this point in the history
Commit 9d5ec8c was supposed to weaken a check so that we don't require
debug access for an SBA to come out if there's a magic flag set. (This
is to handle fault injection tests).

But I got it wrong! And it is now checking that SBA transactions don't
happen at all. Fix the mistake.

Signed-off-by: Rupert Swarbrick <[email protected]>
  • Loading branch information
rswarbrick committed Aug 13, 2024
1 parent c082b89 commit c7ca134
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion hw/ip/rv_dm/dv/env/rv_dm_scoreboard.sv
Original file line number Diff line number Diff line change
Expand Up @@ -195,7 +195,7 @@ class rv_dm_scoreboard extends cip_base_scoreboard #(

// If the debugger is not enabled and cfg.sba_tl_tx_requires_debug=1 then we should not have
// seen the SBA item translate into a TL access.
if (cfg.sba_tl_tx_requires_debug) begin
if (cfg.sba_tl_tx_requires_debug && cfg.rv_dm_vif.lc_hw_debug_en != lc_ctrl_pkg::On) begin
`DV_CHECK(sba_tl_access_q.size() == 0)
end

Expand Down

0 comments on commit c7ca134

Please sign in to comment.