Skip to content

Conversation

@mariopaja
Copy link
Contributor

@mariopaja mariopaja commented Oct 15, 2025

This PR enables SAI on STM32F7xx series by:

  1. Adding SAI1 A/B nodes
  2. Update DMA configurations for STM32F7xx
  3. Adding nucleo_f767zi in samples/drivers/i2s/output
stm32f7xx_sai

Audio Test:
2CH, 16bit, 44.1KHz, PCM5102a

stm32f7xx_sai.mp4

@mariopaja mariopaja changed the title Stm32f7xx sai drivers: i2s: stm32 sai add support for stm32f7xx series Oct 15, 2025
@mariopaja mariopaja mentioned this pull request Oct 15, 2025
21 tasks
@mariopaja mariopaja force-pushed the stm32f7xx_sai branch 4 times, most recently from f2d530b to 848b2d9 Compare October 23, 2025 05:02
This change introduces SAI1 A/B nodes to STM32F7xx series

Signed-off-by: Mario Paja <mariopaja@hotmail.com>
@mariopaja
Copy link
Contributor Author

i2s_stm32_sai driver cleanup will come soon after F7xx is merged

@anangl anangl assigned erwango and unassigned anangl Oct 23, 2025
gautierg-st
gautierg-st previously approved these changes Oct 23, 2025
static void dma_callback(const struct device *dma_dev, void *arg, uint32_t channel, int status)
{
DMA_HandleTypeDef *hdma = arg;

Copy link
Contributor

Choose a reason for hiding this comment

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

Please restore the empty line.
It's expected between variable definitions and instructions (which ARG_UNUSED() is when in debug mode).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done :)

STM32F7xx series shares several DMA configurations with
the other platforms. These changes aim to enable platform
specific DMA configuration and align them to other platforms.

Signed-off-by: Mario Paja <mariopaja@hotmail.com>
Add nucleo_f767zi board in samples/drivers/i2s/output

Signed-off-by: Mario Paja <mariopaja@hotmail.com>
@sonarqubecloud
Copy link

@erwango erwango added this to the v4.3.0 milestone Oct 23, 2025
@jhedberg jhedberg merged commit 2290b95 into zephyrproject-rtos:main Oct 25, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants