[rv_dm,rtl] Fix vendored RTL which causes a Verilator lint warning
Open
@andreaskurth is already working on this.
Since Apr 23, 2024.
Component:RTL
IP:rv_dm
Type:Cleanup
- Dominant language
- SystemVerilog
- Stars
- 3.6k
- Forks
- 1.1k
- Avg merge
- 2d 22h
- Merged PRs (30d)
- 141
Description
Description
The pulp code that we're vendoring for rv_dm contains some code that triggers Verilator warnings about address truncation. We're currently waiving the warning in pulp_riscv_dbg.vlt.
When reviewing the waiver in #22486, @andreaskurth pointed out that a better approach would be to fix the underlying RTL. This issue is to remind us to do so.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.