Skip to content

Conversation

@mariopaja
Copy link
Contributor

@mariopaja mariopaja commented Aug 30, 2025

This PR fixes SAI DMA configuration for STM32H7xx series

PeriphInc and MemInc should be as follows for both TX & RX:

hdma->Init.PeriphInc = DMA_PINC_DISABLE;
hdma->Init.MemInc = DMA_MINC_ENABLE;

MemDataAlignment should be set to DMA_MDATAALIGN_HALFWORD

hdma->Init.MemDataAlignment = DMA_MDATAALIGN_HALFWORD;

@mariopaja
Copy link
Contributor Author

@evgeniy-paltsev

@mariopaja mariopaja mentioned this pull request Aug 30, 2025
21 tasks
Copy link
Contributor

@etienne-lms etienne-lms left a comment

Choose a reason for hiding this comment

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

LGTM.
Worth to squash as a single commit drivers: i2s: stm32_sai: fix h7xx DMA configuration?

This change fixes STM32H7xx dma configuration.
PeriphInc and MemInc should be the same for both directions.
MemDataAlignment should be set to DMA_MDATAALIGN_HALFWORD

Signed-off-by: Mario Paja <[email protected]>
@mariopaja mariopaja force-pushed the fix_h7xx_sai_dma_conf branch from 7f01ef1 to 3924642 Compare August 30, 2025 15:20
@sonarqubecloud
Copy link

@kartben kartben merged commit 5af3adf into zephyrproject-rtos:main Sep 2, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants