Skip to content

Conversation

@milovanovic
Copy link
Contributor

This adds a JTAG-to-memory-mapped bus master bridge, right now only for TL and AXI4, but could be potentially extended to other buses like AHB or APB. The main idea is to be able to test (in hardware) memory-mapped peripherals (e.g., a DspBlock) by reading from and writing to their status and configuration registers without having to invoke a complete processor core like Rocket. Portions of the code are reused from chisel-jtag and/or Rocket Chip JTAG. The same PR is recently opened against testchipip. Feel free to give improvement suggestions and to change the license to whatever you believe is the most suitable one.

@chick
Copy link
Contributor

chick commented Apr 19, 2021

@milovanovic I'd like to help get this merged but I'm a little unsure of the best approach. Changes to chisel-testers and updating scalatest has created a lot of errors. I have fixed these locally. I could push them as a PR to your master branch, or maybe there's a better way. Do you have any ideas.

@milovanovic
Copy link
Contributor Author

Thanks @chick for getting involved. Whatever you think is the best approach we'll do. I invited you as a collaborator to local forks so if it's easier, take the shortcut. Otherwise just place a PR to the master branch. The bigger problem I see is that the current rocket-dsptools subproject is not in CI, it doesn't build locally, neither its tests pass. Migration from Travis to Actions is perhaps a good time to add fixed rocket-dsptools into it also. If the rocket gets settled on your side, I believe we'll find a way to polish the JTAG2MM and integrate it.

@chick
Copy link
Contributor

chick commented Apr 20, 2021 via email

milovanovic and others added 3 commits April 20, 2021 19:07
- Driver now uses iotesters compatibility stuff
- Tests change to used last scala tests
  - Add `Any` prefix to scalatest class names
  - Matchers import has changed
- `withClockAndReset` moved out of experimental package
@chick
Copy link
Contributor

chick commented May 18, 2021

@milovanovic This looks good, and it seems to have all my changes. Any reasons not to merge it now?

@chick chick merged commit 7369edc into ucb-bar:master May 19, 2021
@sequencer
Copy link
Contributor

sequencer added a commit to sequencer/dsptools that referenced this pull request May 20, 2021
@sequencer
Copy link
Contributor

Should fix in #231

chick added a commit that referenced this pull request Jun 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants