* [PATCH RESEND] ARM: pxa: ssp: remove kfree for memory allocated with devm_kzalloc
@ 2013-09-24 3:35 Wei Yongjun
2013-09-24 8:54 ` Daniel Mack
0 siblings, 1 reply; 3+ messages in thread
From: Wei Yongjun @ 2013-09-24 3:35 UTC (permalink / raw)
To: linux-arm-kernel
From: Wei Yongjun <yongjun_wei@trendmicro•com.cn>
It's not necessary to free memory allocated with devm_kzalloc
and using kfree leads to a double free.
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro•com.cn>
---
Correct Mark's mail address
---
arch/arm/plat-pxa/ssp.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/plat-pxa/ssp.c b/arch/arm/plat-pxa/ssp.c
index 3ea0290..528143e 100644
--- a/arch/arm/plat-pxa/ssp.c
+++ b/arch/arm/plat-pxa/ssp.c
@@ -250,7 +250,6 @@ static int pxa_ssp_remove(struct platform_device *pdev)
list_del(&ssp->node);
mutex_unlock(&ssp_lock);
- kfree(ssp);
return 0;
}
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH RESEND] ARM: pxa: ssp: remove kfree for memory allocated with devm_kzalloc
2013-09-24 3:35 [PATCH RESEND] ARM: pxa: ssp: remove kfree for memory allocated with devm_kzalloc Wei Yongjun
@ 2013-09-24 8:54 ` Daniel Mack
2013-09-27 2:15 ` Chao Xie
0 siblings, 1 reply; 3+ messages in thread
From: Daniel Mack @ 2013-09-24 8:54 UTC (permalink / raw)
To: linux-arm-kernel
On 24.09.2013 05:35, Wei Yongjun wrote:
> From: Wei Yongjun <yongjun_wei@trendmicro•com.cn>
>
> It's not necessary to free memory allocated with devm_kzalloc
> and using kfree leads to a double free.
>
> Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro•com.cn>
Acked-by: Daniel Mack <zonque@gmail•com>
Chao Xie wanted to work on getting rid of the "middle layer" the pxa ssp
driver has. If that's going to happen for 3.13, it will again affect
both the ASoC part and code in arch/arm. Therefore, I would again prefer
if ssp patches went through Mark's ASoC tree. Chao, are you still
planning to work on this?
Daniel
> ---
> arch/arm/plat-pxa/ssp.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm/plat-pxa/ssp.c b/arch/arm/plat-pxa/ssp.c
> index 3ea0290..528143e 100644
> --- a/arch/arm/plat-pxa/ssp.c
> +++ b/arch/arm/plat-pxa/ssp.c
> @@ -250,7 +250,6 @@ static int pxa_ssp_remove(struct platform_device *pdev)
> list_del(&ssp->node);
> mutex_unlock(&ssp_lock);
>
> - kfree(ssp);
> return 0;
> }
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH RESEND] ARM: pxa: ssp: remove kfree for memory allocated with devm_kzalloc
2013-09-24 8:54 ` Daniel Mack
@ 2013-09-27 2:15 ` Chao Xie
0 siblings, 0 replies; 3+ messages in thread
From: Chao Xie @ 2013-09-27 2:15 UTC (permalink / raw)
To: linux-arm-kernel
> -----Original Message-----
> From: Daniel Mack [mailto:zonque at gmail.com]
> Sent: Tuesday, September 24, 2013 4:55 PM
> To: Wei Yongjun
> Cc: linux at arm.linux.org.uk; grant.likely at linaro.org; rob.herring at calxeda.com; broonie at sirena.org.uk; haojian.zhuang at gmail.com; olof at lixom.net; gregkh at linuxfoundation.org; yongjun_wei at trendmicro.com.cn; linux-arm-kernel at lists.infradead.org; Chao Xie
> Subject: Re: [PATCH RESEND] ARM: pxa: ssp: remove kfree for memory allocated with devm_kzalloc
>
> On 24.09.2013 05:35, Wei Yongjun wrote:
> From: Wei Yongjun <yongjun_wei@trendmicro•com.cn>
>
> It's not necessary to free memory allocated with devm_kzalloc
> and using kfree leads to a double free.
>
> Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro•com.cn>
>
> Acked-by: Daniel Mack <zonque@gmail•com>
>
> Chao Xie wanted to work on getting rid of the "middle layer" the pxa ssp
> driver has. If that's going to happen for 3.13, it will again affect
> both the ASoC part and code in arch/arm. Therefore, I would again prefer
> if ssp patches went through Mark's ASoC tree. Chao, are you still
> planning to work on this?
>
For last month I had to work at other project.
I will continue to work at it at early Oct.
>
>
> Daniel
>
>
> ---
> arch/arm/plat-pxa/ssp.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm/plat-pxa/ssp.c b/arch/arm/plat-pxa/ssp.c
> index 3ea0290..528143e 100644
> --- a/arch/arm/plat-pxa/ssp.c
> +++ b/arch/arm/plat-pxa/ssp.c
> @@ -250,7 +250,6 @@ static int pxa_ssp_remove(struct platform_device *pdev)
> list_del(&ssp->node);
> mutex_unlock(&ssp_lock);
>
> - kfree(ssp);
> return 0;
> }
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-09-27 2:15 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-09-24 3:35 [PATCH RESEND] ARM: pxa: ssp: remove kfree for memory allocated with devm_kzalloc Wei Yongjun
2013-09-24 8:54 ` Daniel Mack
2013-09-27 2:15 ` Chao Xie
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox