-
Notifications
You must be signed in to change notification settings - Fork 13
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
Midas integration #6
Conversation
…tDecoupled to be new HostPort, added new HostDecoupled. Importantly, these don't force hostBits to be Outputs like Chisel Decoupled does
@@ -1,10 +1,11 @@ | |||
package strober | |||
|
|||
import midas_widgets._ | |||
import dram_midas._ |
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.
Why do we need this dependency?
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.
Which? On dram_midas? Or on widgets?
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 mean dram_midas
.
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.
No we don't need it, it was just easier for the time being.
Do you think i should just pass some () => MemoryModel through the config? Or can you think of a better way to parameterize this in the short term?
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 just can't compile it without dram_midas
. Let's figure out how we can remove this dependency.
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.
Yeah, sounds good. The purpose of this PR was mainly to solicit feedback. We can talk about how we want to do that tomorrow.
…into midas_integration
71688bb
to
8d3917b
Compare
I confirm this works on FPGA. |
Midas integration
Opened this PR mainly for review.
I've pulled all the source from midas into midas_widgets and kept the history. I can remove the history if you prefer.