Skip to content

rp2xxx/ch32v: Fix dma slice handling#827

Merged
mattnite merged 1 commit intomainfrom
dma_fix
Jan 3, 2026
Merged

rp2xxx/ch32v: Fix dma slice handling#827
mattnite merged 1 commit intomainfrom
dma_fix

Conversation

@Grazfather
Copy link
Collaborator

Noticed that the anytype fields didn't work when making changes to the ch32v dma hal. These were yanked from the rp2xxx version.

The example doesn't catch the issue because the compiler seems to use array fields when it can, I had to call with an intermediate function to force the compiler to create a version where the pointer was a slice type.

@Grazfather Grazfather requested a review from vesim987 January 3, 2026 21:33
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 5 issue(s) exist but are not on lines changed in this PR:

port/raspberrypi/rp2xxx/src/hal/dma.zig:120: TODO style comments need to have a linked microzig issue on the same line.
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:98: TODO style comments need to have a linked microzig issue on the same line.
port/wch/ch32v/src/hals/dma.zig:303: 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.

@mattnite mattnite merged commit 941ae75 into main Jan 3, 2026
52 checks passed
@mattnite mattnite deleted the dma_fix branch January 3, 2026 22:03
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