AXI4 top-level #30
Draft
AXI4 top-level #30
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds an AXI4 top-level wrapper and converters for BlackParrot. The primary difference from the existing bp_axi_top is converting the I/O interfaces from AXI-Lite to AXI. This is desirable when integrating BP into systems containing many peripheral devices with a mix of 32b and 64b interfaces. AXI-Lite simply cannot do both 32b and 64b accesses properly with a fixed 32b or 64b data channel width.
The implementation is low performance and requires that bedrock_fill_width == axi_data_width == 64-bits, and that all I/O transactions are at most 64-bits in size with one data transfer per transaction and naturally aligned addresses. The data width restriction should be relaxed to only requiring axi_data_width of 64-bits by using bedrock stream pumps to gearbox as needed.