Skip to content

feat(axi): Add optional AXI cut on the DMA manager port - #295

Merged
creinwar merged 1 commit into
mainfrom
wiesep/axi_cut
Oct 14, 2026
Merged

creinwar merged 1 commit into
mainfrom
wiesep/axi_cut

Conversation

@Xeratec

@Xeratec Xeratec commented Aug 25, 2026 •

Copy link
Copy Markdown
Member

Adds an optional register cut on the DMA's AXI manager port, controlled by a new
DmaEnableAxiCut configuration bit.

Disabled by default (DmaEnableAxiCut : 0), which instantiates axi_cut with
Bypass = 1'b1 — a pure passthrough. No functional or timing change for any existing
configuration.

Changes

  • cheshire_pkg: new bit DmaEnableAxiCut in the DMA parameter group, defaulted to 0
    in DefaultCfg.
  • cheshire_idma_wrap: new EnableAxiCut parameter and an axi_cut between the iDMA
    backend and axi_mst_req_o/axi_mst_rsp_i, following the same
    .Bypass ( ~<cfg bit> ) idiom already used for the AMO post-cuts in cheshire_soc.
    The wrapper now also takes the individual axi_mst_*_chan_t types that axi_cut
    requires.
  • cheshire_soc: forwards Cfg.DmaEnableAxiCut and the channel types to the wrapper.
  • docs/um/arch.md: documents the new parameter in the iDMA table.

@Xeratec Xeratec self-assigned this Aug 25, 2026
@Xeratec Xeratec added the enhancement New feature or request label Aug 25, 2026
@Xeratec
Xeratec marked this pull request as ready for review August 25, 2026 13:31
@Xeratec
Xeratec requested review from Lore0599 and creinwar August 25, 2026 13:33

@creinwar creinwar left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@creinwar
creinwar merged commit 6c6dd5e into main Oct 14, 2026
18 checks passed
@creinwar
creinwar deleted the wiesep/axi_cut branch October 14, 2026 15:32
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