Skip to content
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

Add axi_burst_unwrap #326

Merged
merged 2 commits into from
Dec 18, 2024
Merged

Add axi_burst_unwrap #326

merged 2 commits into from
Dec 18, 2024

Conversation

niwis
Copy link
Contributor

@niwis niwis commented Dec 4, 2023

Add a module that converts wrapping bursts into up to two incremental bursts.

The first incremental burst covers the region from the start address to the wrap boundary. The second burst covers the region from the aligned base address to the start address. In the special case of an aligned start address, the wrapping burst can be trivially converted into a single incremental burst.

By splitting wrapping bursts that wrap around the wrap boundary into two incremental bursts, the ordering of the read/write transactions is preserved and no data buffering is required.

This module hasn't been properly verified yet.

Signed-off-by: Nils Wistoff <[email protected]>
@niwis niwis force-pushed the nw/axi_burst_unwrap branch from 0632fbb to 7f82d0b Compare December 4, 2023 20:35
@thommythomaso thommythomaso marked this pull request as ready for review December 18, 2024 08:23
@thommythomaso thommythomaso self-assigned this Dec 18, 2024
@thommythomaso thommythomaso self-requested a review December 18, 2024 08:24
@thommythomaso thommythomaso added the enhancement New feature or request label Dec 18, 2024
@thommythomaso thommythomaso merged commit bbe468e into master Dec 18, 2024
3 checks passed
@thommythomaso thommythomaso deleted the nw/axi_burst_unwrap branch December 18, 2024 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants