-
Notifications
You must be signed in to change notification settings - Fork 81
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
Michaero/v5.0.0 draft #96
Conversation
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.
Changes seem to reasonable. Did we test this PR somewhere in PULPissimo?
needs a changelog entry |
95efc08
to
6e07163
Compare
Added the changelog entry, tested in PULPissimo (see this WIP branch) |
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.
Looks good! Let me know once your ready we will remove the draft
from the PR title and merge.
I think the synth FPGA failures need to be addressed anyway on the PULPissimo level so these should not block us.
- Renamed fc_eu signals to fc_interrupts to make a clear distinction between events and interrupts. - Removed old APB crossbar and replaced it with integrated crossbar in AXI-lite to APB converter. Moved address definitions from peripheral_bud_defines to soc_mem_maps.svh
Was not connected anywhere in the first place
This follows the lowrisc guidelines and the practice of most PULP IPs.
470e10f
to
e6d223b
Compare
No description provided.