diff mbox

[v2,4/4] DMA: PL330: unregister dma_device in module's remove function

Message ID 1349432276-22919-5-git-send-email-inderpal.singh@linaro.org
State New
Headers show

Commit Message

Inderpal Singh Oct. 5, 2012, 10:17 a.m. UTC
unregister dma_device in module's remove function.

Signed-off-by: Inderpal Singh <inderpal.singh@linaro.org>
---
 drivers/dma/pl330.c |    2 ++
 1 file changed, 2 insertions(+)

Comments

Vinod Koul Oct. 24, 2012, 4:19 a.m. UTC | #1
On Fri, 2012-10-05 at 15:47 +0530, Inderpal Singh wrote:
> unregister dma_device in module's remove function.
> 
> Signed-off-by: Inderpal Singh <inderpal.singh@linaro.org>
> ---
>  drivers/dma/pl330.c |    2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/dma/pl330.c b/drivers/dma/pl330.c
> index 4b7a34d..e7dc040 100644
> --- a/drivers/dma/pl330.c
> +++ b/drivers/dma/pl330.c
> @@ -3017,6 +3017,8 @@ static int __devexit pl330_remove(struct amba_device *adev)
>  			return -EBUSY;
>  	}
>  
> +	dma_async_device_unregister(&pdmac->ddma);
> +
>  	amba_set_drvdata(adev, NULL);
>  
>  	list_for_each_entry_safe(pch, _p, &pdmac->ddma.channels,

Ok with this one :)

Tried applying but didn't work out. You would need to regenerate this
one.

Thanks
Inderpal Singh Oct. 25, 2012, 11:05 a.m. UTC | #2
On 24 October 2012 09:49, Vinod Koul <vkoul@infradead.org> wrote:
> On Fri, 2012-10-05 at 15:47 +0530, Inderpal Singh wrote:
>> unregister dma_device in module's remove function.
>>
>> Signed-off-by: Inderpal Singh <inderpal.singh@linaro.org>
>> ---
>>  drivers/dma/pl330.c |    2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/drivers/dma/pl330.c b/drivers/dma/pl330.c
>> index 4b7a34d..e7dc040 100644
>> --- a/drivers/dma/pl330.c
>> +++ b/drivers/dma/pl330.c
>> @@ -3017,6 +3017,8 @@ static int __devexit pl330_remove(struct amba_device *adev)
>>                       return -EBUSY;
>>       }
>>
>> +     dma_async_device_unregister(&pdmac->ddma);
>> +
>>       amba_set_drvdata(adev, NULL);
>>
>>       list_for_each_entry_safe(pch, _p, &pdmac->ddma.channels,
>
> Ok with this one :)
>
> Tried applying but didn't work out. You would need to regenerate this
> one.
>

I will regenerate this along with other patches and resend.

With Regards,
Inder

> Thanks
> --
> Vinod Koul
> Intel Corp.
>
diff mbox

Patch

diff --git a/drivers/dma/pl330.c b/drivers/dma/pl330.c
index 4b7a34d..e7dc040 100644
--- a/drivers/dma/pl330.c
+++ b/drivers/dma/pl330.c
@@ -3017,6 +3017,8 @@  static int __devexit pl330_remove(struct amba_device *adev)
 			return -EBUSY;
 	}
 
+	dma_async_device_unregister(&pdmac->ddma);
+
 	amba_set_drvdata(adev, NULL);
 
 	list_for_each_entry_safe(pch, _p, &pdmac->ddma.channels,