-
Notifications
You must be signed in to change notification settings - Fork 565
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
Memory test fails on arty7 + Rocket #933
Comments
Hi @developandplay, this could be related to the |
VexRiscv was tested at 100Mhz when it passed without any errors. I now tested VexRiscv at 50Mhz and also encounter memory errors on the arty.
|
With the workaround from litex-hub/litex-boards#166 the memory test now succeeds using VexRiscV@50Mhz on the arty board.
|
With the workaround from litex-hub/litex-boards#166 the memory test now succeeds using Rocket@50Mhz on the arty board.
|
So what do we make from this now? 🤔 |
@developandplay: Thanks for the tests, now we know a little bit more what's going on. I'll try to improve this soon. |
Just tested Rocket@100Mhz with the workaround and it also works on the arty. |
At 100MHz, it should work with Rocket without the workaround no? |
Yes it did. Just wanted to add this workaround doesn't lead to regression at 100Mhz. |
On Tue, Jun 08, 2021 at 09:48:59AM -0700, developandplay wrote:
Yes it did. Just wanted to add this workaround doesn't lead to regression at
100Mhz.
Just curious, if you build litex+rocket at 100MHz, does Vivado think
the design passes timing? I remember getting timing failures from
Vivado (on the nexys4ddr) at anything significantly higher than 50MHz
(admittedly, after adding both --with-ethernet and --with-sdcard).
|
Yep it doesn't meet timing constraints at 100Mhz.
|
Note for my future self and others using Blackparrot:
the memory test succeeds again. |
This has been fixed with enjoy-digital/litedram@6f323f6 and tested on Rocket at 50Mhz with it:
|
I wanted to try version a064e9d of Litex with Rocket.
Unfortunately it failed the memory tests.
I went back a bit further and also tried 21273ff which failed them as well.
On the ulx3s the initialization works without any issues.
Additionally I tried vexriscv at 21273ff which passes the memory tests on both arty7 and ulx3s.
The text was updated successfully, but these errors were encountered: