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

Make boot ROM contents configurable through top-level parameter #27

Merged
merged 1 commit into from Dec 13, 2016
Merged

Conversation

olofk
Copy link
Contributor

@olofk olofk commented Dec 12, 2016

This makes it easier for external tools to change boot ROM

@sam-falvo sam-falvo merged commit 764f849 into KestrelComputer:master Dec 13, 2016
@sam-falvo
Copy link
Contributor

Thanks! Parameterization is something I don't have much experience with, but will need to research going further.

I hear that Vivado does not support readmemh() though when instantiating block RAMs. This is a horrible design decision on Xilinx's part. What are they thinking?! I'll have to think of something else for a computer that is actually synthesizable.

@olofk
Copy link
Contributor Author

olofk commented Dec 13, 2016

Are you sure about that? I'm pretty convinced that works

@olofk olofk deleted the bootrom_fix branch December 13, 2016 07:39
@sam-falvo
Copy link
Contributor

Jan Gray had problems with his GRVI project, according to what he expressed during the RISC-V workshop. Dunno; I don't use Vivado.

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.

None yet

2 participants