[PATCH v2 2/4] dmaengine: Add STM32 DMA driver
M'boumba Cedric Madianga
cedric.madianga at gmail.com
Wed Oct 14 08:26:04 PDT 2015
2015-10-14 16:24 GMT+02:00 Daniel Thompson <daniel.thompson at linaro.org>:
>
> Hmnnn...
>
> The dmaengine framework will WARN_ONCE() if an dmaengine is removed whilst
> it is active and also works hard to ensure dmaengine modules are not removed
> whilst there are active drivers using the framework.
>
> How do we get into this function whilst there is still an active DMA
> channels?
For example, when a user try "rmmod stm32-dma" in uart console.
It will enter in stm32_dma_remove while there is potentially still active DMA.
>
>
>> But even with this improvement, I think I have to disable the clock here.
>
>
> As above, I think the dmaengine framework work to protect you from this sort
> problem. However even if I am wrong about that then unconditionally calling
> clk_disable_unprepare() can not be used to reliably manage the clocks. You
> don't know what the counts are!
Ok got it. Thanks for the clarification.
Cedric
More information about the linux-arm-kernel
mailing list