NutShell master branch re-adaption for FPGA platforms#206
Merged
poemonsense merged 8 commits intoOSCPU:masterfrom Aug 26, 2024
Merged
NutShell master branch re-adaption for FPGA platforms#206poemonsense merged 8 commits intoOSCPU:masterfrom
poemonsense merged 8 commits intoOSCPU:masterfrom
Conversation
…RTL file routine in tcl script
poemonsense
approved these changes
Aug 24, 2024
Collaborator
poemonsense
left a comment
There was a problem hiding this comment.
Look good to me
@sashimi-yzh Do you have time to take a look?
This was
linked to
issues
Aug 24, 2024
Collaborator
|
Thank you for the contribution. Please help add a CI test (simply build the verilog from Chisel?) for FPGA verilog to .github/workflows/main.yml if possible. |
Collaborator
|
I'm merging this now. If there're issues after yzh review, we can further fix them |
markaulunGH
pushed a commit
to markaulunGH/NutShell
that referenced
this pull request
Oct 26, 2024
NutShell master branch re-adaption for FPGA platforms
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The master branch of NutShell can not generate an FPGA project that provides correct functionality(#205 #204 #200 ). Therefore I made serval changes to the NutShell and the FPGA project tcl.
nutcoretrapin CSR as a signal to dump the statistic during the simulation, this signal is connected by the boringUtil. However, the source of this connection is missing when using the FPGA platform. I setnutcoretrapsignal to zero when using FPGA.sedcommand in Makefile to convert the Chisel name to the specific form of AXI Signals. However, after using--spilt-verilogoption, not all the signals were converted by the command.--spilt-verilog, vivado project tcl only addsTopMain.vto the project, therefore other RTL source files are missing in the vivado project.mvcommand to rename the top file name fromSimTop.svtoTopMain.sv, which will cause an error to the FPGA platform as the `SimTop.sv`` does not exist.The issues mentioned above have been resolved in the pull request. These modifications have been tested with sidewinder and PYNQ.