Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump to Chisel 6, migrate to svsim #122

Merged
merged 1 commit into from
Jan 31, 2024

Conversation

seldridge
Copy link
Member

Bump the template to Chisel 6. Drop the ChiselTest dependency and migrate to svsim.

The test is not as efficient as it could be right now. However, it works!

@seldridge
Copy link
Member Author

I think this needs to install Verilator in the image as this is working locally for me.

@schoeberl
Copy link
Contributor

LGTM. At least we see now svsim in action. Still very low lever, but I think we can build on that.

@seldridge
Copy link
Member Author

seldridge commented Jan 31, 2024

LGTM. At least we see now svsim in action. Still very low lever, but I think we can build on that.

Yep! Please start seeing if this would work for you and your teaching.

Note that this example can be made much, much cleaner. I don't see why we couldn't add the equivalent of enqueueSeq and dequeueSeq at a minimum. This can also land directly in Chisel's svsim area so it works for any decoupled. Fork and join are more tedious, but I don't see why this couldn't be made to work.

@seldridge
Copy link
Member Author

This works in CI with a manual Verilator install. The installation can be sped up by properly setting up ccache (which Verilator is configured to use by default). 🎊 It would also be worthwhile to see if upstream verilator would accept (or we implement) a GitHub action that just does the Verilator install reusably.

@schoeberl
Copy link
Contributor

LGTM. At least we see now svsim in action. Still very low lever, but I think we can build on that.

Yep! Please start seeing if this would work for you and your teaching.

Note that this example can be made much, much cleaner. I don't see why we couldn't add the equivalent of enqueueSeq and dequeueSeq at a minimum. This can also land directly in Chisel's svsim area so it works for any decoupled. Fork and join are more tedious, but I don't see why this couldn't be made to work.

Yes, to fork and join. It needs to be a bit nicer/polished UI for use in teaching. I will start playing around with it soon ;-)
However, I still think we should provide an ChiselTest version based on svsim, with a compatible API. Of course, not using SFC. Just the interface.

@seldridge seldridge enabled auto-merge (squash) January 31, 2024 23:14
Bump the template to Chisel 6.  Drop the ChiselTest dependency and migrate
to svsim.

Signed-off-by: Schuyler Eldridge <[email protected]>
@seldridge seldridge force-pushed the dev/seldridge/bump-to-chisel-6-drop-chiseltest branch from fb85240 to e179c02 Compare January 31, 2024 23:15
@seldridge seldridge merged commit 2a5cb5f into main Jan 31, 2024
2 checks passed
@seldridge seldridge deleted the dev/seldridge/bump-to-chisel-6-drop-chiseltest branch January 31, 2024 23:16
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