search

STMicroelectronics / STM32CubeF7

STM32Cube MCU Full Package for the STM32F7 series - (HAL + LL Drivers, CMSIS Core, CMSIS Device, MW libraries plus a set of Projects running on all boards provided by ST (Nucleo, Evaluation and Discovery Kits))
Other
320 stars 191 forks source link

HAL_DMA_Init set invalid state when DMA_CheckFifoParam not return HAL_OK #87

Closed KamilDuljas closed 4 weeks ago

KamilDuljas commented 1 year ago

https://github.com/STMicroelectronics/STM32CubeF7/blob/6a3b9bb8d09839f365ba83a6cf5823ae5f5226db/Drivers/STM32F7xx_HAL_Driver/Src/stm32f7xx_hal_dma.c#L279

In related to documentation, The checking compatibility before FIFO threshold level and size of the memory burst is very fragile but after returned other state than HAL_OK the function set HAL_DMA_STATE_READY instead of ERROR or RESET.

MCU H7 is also affected: https://github.com/STMicroelectronics/STM32CubeH7/blob/b75d4ea0a368c0e2e7bb9c40c1258a39fff0919a/Drivers/STM32H7xx_HAL_Driver/Src/stm32h7xx_hal_dma.c#L341

HBOSTM commented 1 year ago

Hi @KamilDuljas ,

Thank you for this contribution, this point has been reported to our development teams. I will get back to you as soon as I have any updates.

With regards,

HBOSTM commented 1 year ago

ST Internal Reference: 149047

TOUNSTM commented 4 weeks ago

Fixed in 7f25a108012707cc7a643a6bee79947bd068e0ab