michaelh/zephyr
1
0
Fork 0
Code Releases Activity

drivers: dma: stm32 dma driver support repeated start/stop

Browse source 
To support the repeated start/stop, the stm32 dma driver is
returning 0 if the channel is already started/stopped.
This is not done at the dmamux level if any.

Signed-off-by: Francois Ramu <francois.ramu@st.com>
This commit is contained in:
Francois Ramu 2023-01-12 10:13:58 +01:00 • committed by Carles Cufí
commit 227226313a
4 changed files with 27 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

10
drivers/dma/dma_stm32.c
View file

@ -573,6 +573,11 @@ DMA_STM32_EXPORT_API int dma_stm32_start(const struct device *dev, uint32_t id)
return -EINVAL; return -EINVAL;
} }
/* Repeated start : return now if channel is already started */
if (stm32_dma_is_enabled_stream(dma, id)) {
return 0;
}
/* When starting the dma, the stream is busy before enabling */ /* When starting the dma, the stream is busy before enabling */
stream = &config->streams[id]; stream = &config->streams[id];
stream->busy = true; stream->busy = true;
@ -596,6 +601,11 @@ DMA_STM32_EXPORT_API int dma_stm32_stop(const struct device *dev, uint32_t id)
return -EINVAL; return -EINVAL;
} }
/* Repeated stop : return now if channel is already stopped */
if (!stm32_dma_is_enabled_stream(dma, id)) {
return 0;
}
#if !defined(CONFIG_DMAMUX_STM32) \ #if !defined(CONFIG_DMAMUX_STM32) \
|| defined(CONFIG_SOC_SERIES_STM32H7X) || defined(CONFIG_SOC_SERIES_STM32MP1X) || defined(CONFIG_SOC_SERIES_STM32H7X) || defined(CONFIG_SOC_SERIES_STM32MP1X)
LL_DMA_DisableIT_TC(dma, dma_stm32_id_to_stream(id)); LL_DMA_DisableIT_TC(dma, dma_stm32_id_to_stream(id));

1
drivers/dma/dma_stm32.h
View file

@ -81,6 +81,7 @@ void stm32_dma_clear_stream_irq(DMA_TypeDef *dma, uint32_t id);
bool stm32_dma_is_irq_happened(DMA_TypeDef *dma, uint32_t id); bool stm32_dma_is_irq_happened(DMA_TypeDef *dma, uint32_t id);
bool stm32_dma_is_unexpected_irq_happened(DMA_TypeDef *dma, uint32_t id); bool stm32_dma_is_unexpected_irq_happened(DMA_TypeDef *dma, uint32_t id);
void stm32_dma_enable_stream(DMA_TypeDef *dma, uint32_t id); void stm32_dma_enable_stream(DMA_TypeDef *dma, uint32_t id);
bool stm32_dma_is_enabled_stream(DMA_TypeDef *dma, uint32_t id);
int stm32_dma_disable_stream(DMA_TypeDef *dma, uint32_t id); int stm32_dma_disable_stream(DMA_TypeDef *dma, uint32_t id);
#if !defined(CONFIG_DMAMUX_STM32) #if !defined(CONFIG_DMAMUX_STM32)

8
drivers/dma/dma_stm32_v1.c
View file

@ -320,6 +320,14 @@ void stm32_dma_enable_stream(DMA_TypeDef *dma, uint32_t id)
LL_DMA_EnableStream(dma, dma_stm32_id_to_stream(id)); LL_DMA_EnableStream(dma, dma_stm32_id_to_stream(id));
} }
bool stm32_dma_is_enabled_stream(DMA_TypeDef *dma, uint32_t id)
{
if (LL_DMA_IsEnabledStream(dma, dma_stm32_id_to_stream(id)) == 1) {
return true;
}
return false;
}
int stm32_dma_disable_stream(DMA_TypeDef *dma, uint32_t id) int stm32_dma_disable_stream(DMA_TypeDef *dma, uint32_t id)
{ {
LL_DMA_DisableStream(dma, dma_stm32_id_to_stream(id)); LL_DMA_DisableStream(dma, dma_stm32_id_to_stream(id));

8
drivers/dma/dma_stm32_v2.c
View file

@ -291,6 +291,14 @@ void stm32_dma_enable_stream(DMA_TypeDef *dma, uint32_t id)
LL_DMA_EnableChannel(dma, dma_stm32_id_to_stream(id)); LL_DMA_EnableChannel(dma, dma_stm32_id_to_stream(id));
} }
bool stm32_dma_is_enabled_stream(DMA_TypeDef *dma, uint32_t id)
{
if (LL_DMA_IsEnabledChannel(dma, dma_stm32_id_to_stream(id)) == 1) {
return true;
}
return false;
}
int stm32_dma_disable_stream(DMA_TypeDef *dma, uint32_t id) int stm32_dma_disable_stream(DMA_TypeDef *dma, uint32_t id)
{ {
LL_DMA_DisableChannel(dma, dma_stm32_id_to_stream(id)); LL_DMA_DisableChannel(dma, dma_stm32_id_to_stream(id));