[PATCH] ASoC: wm_adsp: Avoid attempt to free buffers that might still be in use

[PATCH] ASoC: wm_adsp: Avoid attempt to free buffers that might still be in use

[Charles Keepax]

We should not free any buffers associated with writing out coefficients
to the DSP until all the async writes have completed. This patch updates
the out of memory path when allocating a new buffer to include a call to
regmap_async_complete.

Reported-by: JS Park <aitdark.park@samsung.com>
Signed-off-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
---
 sound/soc/codecs/wm_adsp.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/sound/soc/codecs/wm_adsp.c b/sound/soc/codecs/wm_adsp.c
index 0a08ef5e..08d8ec3 100644
--- a/sound/soc/codecs/wm_adsp.c
+++ b/sound/soc/codecs/wm_adsp.c
@@ -1343,6 +1343,7 @@ static int wm_adsp_load_coeff(struct wm_adsp *dsp)
 			  file, blocks, pos - firmware->size);
 
 out_fw:
+	regmap_async_complete(regmap);
 	release_firmware(firmware);
 	wm_adsp_buf_free(&buf_list);
 out:
-- 
1.7.2.5

Re: [PATCH] ASoC: wm_adsp: Avoid attempt to free buffers that might still be in use

[Mark Brown]

[-- Attachment #1.1: Type: text/plain, Size: 331 bytes --]

On Mon, Nov 17, 2014 at 10:48:21AM +0000, Charles Keepax wrote:
> We should not free any buffers associated with writing out coefficients
> to the DSP until all the async writes have completed. This patch updates
> the out of memory path when allocating a new buffer to include a call to
> regmap_async_complete.

Applied, thanks.

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]