chipsalliance / chipsalliance/rocket-chip
Periphery: stale workaround `NoPrefix` for debug
- Dominant language
- Scala
- Stars
- 3.9k
- Forks
- 1.3k
- Avg merge
- 5d 13m
- Merged PRs (30d)
- 1
Description
https://github.com/chipsalliance/rocket-chip/blob/b503f8ac28a497b2463ffbac84bfe66533ace0bb/src/main/scala/devices/debug/Periphery.scala#L115
This references a workaround for a chisel issue closed by https://github.com/chipsalliance/chisel3/pull/1660
My guess is this naming bug workaround can probably be removed now? I'll look into it later but documenting it for now as a low priority update.
Contributor guide
Research direction
Start at src/main/scala/devices/debug/Periphery.scala around line 115 and review the referenced Chisel pull request #1660. Determine whether the NoPrefix workaround is still required, then run the relevant Rocket Chip or debug-generation tests to confirm naming remains correct after any removal.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- scala
- Domain
- embedded-iot
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100