Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver

From	"M'boumba Cedric Madianga" <cedric.madianga@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver
Date	2015-10-14 17:50 +0200
Message-ID	<qjwts-2XK-29@gated-at.bofh.it> (permalink)
References	(5 earlier) <qjuhX-8cu-17@gated-at.bofh.it> <qjuBl-8B-25@gated-at.bofh.it> <qjve2-17o-23@gated-at.bofh.it> <qjwa6-2yi-23@gated-at.bofh.it> <qjwa6-2yi-21@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

2015-10-14 17:28 GMT+02:00 Daniel Thompson <daniel.thompson@linaro.org>:
> On 14/10/15 16:26, M'boumba Cedric Madianga wrote:
>>
>> 2015-10-14 16:24 GMT+02:00 Daniel Thompson <daniel.thompson@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.
>
>
> Check dmaengine.c for yourself but I think in this case the dmaengine
> framework will hold references to the module and prevent the remove from
> taking place.

Yes I did it.
As far I understand, the dmaengine framework will print a warning
message but doesn't stop removing operation if there are some active
clients.

Cedric.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

[PATCH v2 0/4] Add support for STM32 DMA M'boumba Cedric Madianga <cedric.madianga@gmail.com> - 2015-10-13 16:10 +0200
  [PATCH v2 2/4] dmaengine: Add STM32 DMA driver M'boumba Cedric Madianga <cedric.madianga@gmail.com> - 2015-10-13 16:10 +0200
    Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver Daniel Thompson <daniel.thompson@linaro.org> - 2015-10-13 16:40 +0200
      Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver "M'boumba Cedric Madianga" <cedric.madianga@gmail.com> - 2015-10-14 10:00 +0200
        Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver "M'boumba Cedric Madianga" <cedric.madianga@gmail.com> - 2015-10-14 11:00 +0200
        Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver Daniel Thompson <daniel.thompson@linaro.org> - 2015-10-14 11:00 +0200
        Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver "M'boumba Cedric Madianga" <cedric.madianga@gmail.com> - 2015-10-14 15:20 +0200
          Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver Daniel Thompson <daniel.thompson@linaro.org> - 2015-10-14 15:30 +0200
            Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver "M'boumba Cedric Madianga" <cedric.madianga@gmail.com> - 2015-10-14 15:50 +0200
              Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver Daniel Thompson <daniel.thompson@linaro.org> - 2015-10-14 16:30 +0200
                Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver Daniel Thompson <daniel.thompson@linaro.org> - 2015-10-14 17:30 +0200
                Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver "M'boumba Cedric Madianga" <cedric.madianga@gmail.com> - 2015-10-14 17:50 +0200
                Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver Vinod Koul <vinod.koul@intel.com> - 2015-10-15 06:10 +0200
                Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver "M'boumba Cedric Madianga" <cedric.madianga@gmail.com> - 2015-10-14 17:30 +0200
    Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver Vinod Koul <vinod.koul@intel.com> - 2015-10-14 13:20 +0200
      Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver "M'boumba Cedric Madianga" <cedric.madianga@gmail.com> - 2015-10-14 15:10 +0200
        Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver Vinod Koul <vinod.koul@intel.com> - 2015-10-14 16:20 +0200
  [PATCH v2 4/4] ARM: configs: Add STM32 DMA support in STM32 defconfig M'boumba Cedric Madianga <cedric.madianga@gmail.com> - 2015-10-13 16:10 +0200
  [PATCH v2 3/4] ARM: dts: Add STM32 DMA support for STM32F429 MCU M'boumba Cedric Madianga <cedric.madianga@gmail.com> - 2015-10-13 16:10 +0200
  [PATCH v2 1/4] dt-bindings: Document the STM32 DMA bindings M'boumba Cedric Madianga <cedric.madianga@gmail.com> - 2015-10-13 16:10 +0200
    Re: [PATCH v2 1/4] dt-bindings: Document the STM32 DMA bindings "M'boumba Cedric Madianga" <cedric.madianga@gmail.com> - 2015-10-14 11:00 +0200

csiph-web