Skip to content

ch32v: Fix DMA#833

Merged
mattnite merged 3 commits intomainfrom
wch_dma_fix
Jan 4, 2026
Merged

ch32v: Fix DMA#833
mattnite merged 3 commits intomainfrom
wch_dma_fix

Conversation

@Grazfather
Copy link
Collaborator

  • Fix mem2mem and periph2mem semantics
  • Improve DMA example

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

🔍 Lint Results

ℹ️ Additional issues on unchanged lines
The following 4 issue(s) exist but are not on lines changed in this PR:

port/wch/ch32v/src/hals/dma.zig:18: TODO style comments need to have a linked microzig issue on the same line.
port/wch/ch32v/src/hals/dma.zig:45: TODO style comments need to have a linked microzig issue on the same line.
port/wch/ch32v/src/hals/dma.zig:261: TODO style comments need to have a linked microzig issue on the same line.
port/wch/ch32v/src/hals/dma.zig:16: Suggestion: Rename `DmaRegs` to `DMA_Regs`, it _should_ be more in line with our [style guidelines](https://microzig.tech/docs/contributing/). This automation is not perfect so take it with a grain of salt.

@Grazfather Grazfather changed the title wch: Fix DMA ch32v: Fix DMA Jan 4, 2026
@mattnite mattnite merged commit d825979 into main Jan 4, 2026
52 checks passed
@mattnite mattnite deleted the wch_dma_fix branch January 4, 2026 23:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants