-
Notifications
You must be signed in to change notification settings - Fork 39
initial merge b4 rebase. csr can now write to mtvec, added fence as a nop #67
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
Open
nganjehloo
wants to merge
60
commits into
master
Choose a base branch
from
system_csr
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
60 commits
Select commit
Hold shift + click to select a range
2a5cf75
csr implemented, needs testing
684d5a3
csr implemented, needs testing, cleaned verilog stuff
f470ef2
Delete Top.DualPortedMemory.memory.v
nganjehloo efcbabf
Delete firrtl_black_box_resource_files.f
nganjehloo 466ebd9
resolved test file conflict again
a753229
resolved test file conflict 3rd time
c37e759
resolved lab merge conflicts
8a808d4
corrected csr to regfile wiring, enable mie by default
29f3755
added mret in InsTest
374a9fc
corrected csrrwi reg enable flag, addes all csrrx tests
fb8eb48
added incomplete misaligned mem exception handeling
d510b17
csr and system calls finsihed and tested, fully working, commented co…
561b99e
unaligned mem access code wasn't push for some reason
666880f
1st step cleanup
c839878
reverted test binary
159b1c6
more cleanup
f39aed7
forgot to remove merge tag in lab2test
e318c68
removed csr test params from lab2test grader, and fivecycle option
951de83
refactored csr to avoid using utils, restored helpers.scala
e987292
refactored csr to avoid using utils, restored helpers.scala
0cc9d93
refactored csr to avoid using utils, restored helpers.scala, added co…
336d7f3
Removed CSR stuff for time being until we find a good way of testing it
a6e1b2a
removed csr from grading tests
9508e1d
corrected a missing line in CPUTesterDriver
1b6a645
fixed some tests failing
381b251
added comment to credit rocket chip for some implementation details
4ad7ac5
csr unit class created
1245fb5
Delete firrtl_black_box_resource_files.f
nganjehloo 9fe73e6
added mret in InsTest
8909899
corrected csrrwi reg enable flag, addes all csrrx tests
0f45753
added incomplete misaligned mem exception handeling
5d03a23
csr and system calls finsihed and tested, fully working, commented co…
28142a5
unaligned mem access code wasn't push for some reason
3167ed1
removed csr test params from lab2test grader, and fivecycle option
8e8afe4
added comment to credit rocket chip for some implementation details
491c563
cleaned generated verilog and verilator
8c9df1a
Delete Top.DualPortedMemory.memory.v
nganjehloo b749de0
removed useless headers in csr, rebuilt binaries for testing
08d41c1
resolved merge csr testing changes with old testing
bda6e07
added csr debug to single cycle, added test for csrrc instruction
bd6d381
corrected csr to regfile wiring, enable mie by default
96582af
moved verilator testbench to different folder, deleted more verilog f…
352295a
reverted test binary to use original compiled versions
2e5f72d
removed initializing registers through add instruction
a3923fc
forgot to remove merge tag in lab2test
ebb25aa
refactored csr to avoid using utils, restored helpers.scala
2c0af03
removed util header from csr
dab43b1
added comments for widecounter class, removed mcsr object using UInt …
731cf67
Removed CSR from testing for time being until we find a good way of t…
059e4d4
removed csr from grading tests
10481e0
corrected a missing line in CPUTesterDriver that initializes registers
4cbf05e
fixed some tests failing
1c642ef
Merge branch 'system_csr' of https://github.com/jlpteaching/dinocpu i…
08f734d
added support to write to mtvec register for better traps
e1e9871
added support to write to mtvec register for better traps
0073f26
Merge branch 'master' of https://github.com/jlpteaching/dinocpu
877f676
Added fence instruction, and added support to write mtvec csr register
bdcf521
added support to write to mtvec register for better traps
9e02910
initial merge b4 rebase. csr can now write to mtvec, added fence as a…
3b9c7b4
Forgot to remove merge symbols
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't you just say
reg_mtvec := wdata.asTypeOf(new MTVec())?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suppose I could. I just wanted to be explicit in case of any weird optimization trickery. Should I change it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's up to you.