-
Notifications
You must be signed in to change notification settings - Fork 15
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
Wishbone test #33
Wishbone test #33
Conversation
Signed-off-by: Massimiliano Giacometti <[email protected]>
Signed-off-by: Massimiliano Giacometti <[email protected]>
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.
please rebase from wb_slave_address_fix branch
Tried this out with the fix in wb_slave_address_fix branch, and both RTL and GL failed |
Signed-off-by: Massimiliano Giacometti <[email protected]>
According to #24 the size of the wishbone area is 0x10000000, but in the caravel_mgmt_soc_litex/litex/caravel.py Line 186 in 48361c7
The test has been adapted to be compatible with the branch, but I am not sure it is what you really want. |
Thank you everyone for your efforts on this. At this time, due to the age of this issue and other changes that have happened since, we have decided to close this PR without merging it. In some cases this also means there is now a better or other recommended approach, or possible the PR otherwise relates to a deprecated mode of use. Reviewed by @marwaneltoukhy @DavidRLindley |
Issue #24