[PATCH v2] ASoC: omap-pcm: Free dma buffers in case of error.

[PATCH v2] ASoC: omap-pcm: Free dma buffers in case of error.

[Oleg Matcovschi]

Change-Id: I4ba9de0de4681332539246ccc5e11a7a8fb32e79
Signed-off-by: Oleg Matcovschi <oleg.matcovschi@ti.com>
---
v1:
 initial revision
v2:
 resending patch including maintainers

 sound/soc/omap/omap-pcm.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/sound/soc/omap/omap-pcm.c b/sound/soc/omap/omap-pcm.c
index a2a464f..380a5bd 100644
--- a/sound/soc/omap/omap-pcm.c
+++ b/sound/soc/omap/omap-pcm.c
@@ -414,6 +414,10 @@ static int omap_pcm_new(struct snd_soc_pcm_runtime *rtd)
 	}
 
 out:
+	/* free preallocated buffers in case of error */
+	if (ret)
+		omap_pcm_free_dma_buffers(pcm);
+
 	return ret;
 }
 
-- 
1.7.4.1

Re: [PATCH v2] ASoC: omap-pcm: Free dma buffers in case of error.

[Jarkko Nikula]

On 04/25/2012 05:02 AM, Oleg Matcovschi wrote:
> Change-Id: I4ba9de0de4681332539246ccc5e11a7a8fb32e79
> Signed-off-by: Oleg Matcovschi <oleg.matcovschi@ti.com>
> ---
> v1:
>  initial revision
> v2:
>  resending patch including maintainers
> 
>  sound/soc/omap/omap-pcm.c |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
> 
Acked-by: Jarkko Nikula <jarkko.nikula@bitmer.com>

Re: [PATCH v2] ASoC: omap-pcm: Free dma buffers in case of error.

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 366 bytes --]

On Tue, Apr 24, 2012 at 07:02:02PM -0700, Oleg Matcovschi wrote:
> Change-Id: I4ba9de0de4681332539246ccc5e11a7a8fb32e79
> Signed-off-by: Oleg Matcovschi <oleg.matcovschi@ti.com>

Don't include Change-Ids in upstream submissions, they only make sense
within whatever private development tree you are using.

Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: [PATCH v2] ASoC: omap-pcm: Free dma buffers in case of error.

[Peter Ujfalusi]

On 04/25/2012 05:02 AM, Oleg Matcovschi wrote:
> Change-Id: I4ba9de0de4681332539246ccc5e11a7a8fb32e79
> Signed-off-by: Oleg Matcovschi <oleg.matcovschi@ti.com>
> ---
> v1:
>  initial revision
> v2:
>  resending patch including maintainers
> 
>  sound/soc/omap/omap-pcm.c |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)

Acked-by: Peter Ujfalusi <peter.ujfalusi@ti.com>

Re: [PATCH v2] ASoC: omap-pcm: Free dma buffers in case of error.

[Liam Girdwood]

On Tue, 2012-04-24 at 19:02 -0700, Oleg Matcovschi wrote:
> Change-Id: I4ba9de0de4681332539246ccc5e11a7a8fb32e79
> Signed-off-by: Oleg Matcovschi <oleg.matcovschi@ti.com>
> ---
> v1:
>  initial revision
> v2:
>  resending patch including maintainers
> 
>  sound/soc/omap/omap-pcm.c |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
> 

Now applied for 3.4 with change ID removed.

Thanks

Liam