-
Notifications
You must be signed in to change notification settings - Fork 620
PR #1817 may have broken AXI-lite peripheral interconnect #1831
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
Comments
Thanks for the feedback @Dolu1990, I'll look at it. |
@Dolu1990: I started looking at it and can see the issue by directly comparing the generated verilog with/without the changes. I have to stop for now, will continue later or tomorrow. |
Great, thanks :) |
This should be fixed with upstream (tested it on hardware on a Genesys2). |
Thanks, will try :) |
Hi, I forgot about closing that issue, but it seems it doesn't fix it all. What were the fixes which were pushed to #1817 ? |
Hi @Dolu1990, thanks for the feedback, I'll look at it. Would you mind sharing your build command for the test on hardware? I should be able to identify the issue by directly looking at the verilog before and after the PR as I did for the MMC. |
Thanks ^^ |
@Dolu1990: Sorry for the delay, I'm looking at it now. I've been able to have a simple repro, just noting it here if I need it later: Netboot working: Netboot failing: So Ethernet currently seems to be broken when used with axi-lite interface. I'll try to fix this now. |
No worries :) |
Netboot on Arty with AXI-Lite bus is now working with acd66f1. The address shift was the opposite of what it should be in this case. (Copy paste from m2s case just before). This should also fix the NaxRiscv build/design. |
I just tested, it works all good now :D Thanks |
Great, thanks for the feedback and test! |
Hi,
I update Litex and tested NaxRiscv, but it seems that #1817 did break something related to the peripheral interconnect.
UART work enough for the bootloader, but both spi and mmc are broken (can't run the sdcard bootloader).
I tracked it down to #1817.
The previous main branch commit works, while it seems that everything since #1817 isn't.
To visualize the issue, i did run :
But the issue is that i do not know which signals represent the memory bus which goes to the sdcard controller.
Any idea ^^ ?
Thanks
The text was updated successfully, but these errors were encountered: