diff options
author | Roel Kluin <roel.kluin@gmail.com> | 2009-12-07 19:34:22 -0500 |
---|---|---|
committer | Mark Brown <broonie@opensource.wolfsonmicro.com> | 2009-12-08 07:46:11 -0500 |
commit | 370066e2b13bafa8e742673f658e617b6ed143a4 (patch) | |
tree | 9ca2c3ec2ac4d3ed294f5c2005e2e05dfa7806ce /sound/soc/imx | |
parent | 43f0de8d0298e624e6c3bf2185b6003a59b331bd (diff) |
ASoC: Wrong variable returned on error
The wrong variable was returned in the case of an error
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Acked-by: Liam Girdwood <lrg@slimlogic.co.uk>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Diffstat (limited to 'sound/soc/imx')
-rw-r--r-- | sound/soc/imx/mx1_mx2-pcm.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sound/soc/imx/mx1_mx2-pcm.c b/sound/soc/imx/mx1_mx2-pcm.c index b83866529397..bffffcd5ff34 100644 --- a/sound/soc/imx/mx1_mx2-pcm.c +++ b/sound/soc/imx/mx1_mx2-pcm.c | |||
@@ -322,12 +322,12 @@ static int mx1_mx2_pcm_open(struct snd_pcm_substream *substream) | |||
322 | 322 | ||
323 | pr_debug("%s: Requesting dma channel (%s)\n", __func__, | 323 | pr_debug("%s: Requesting dma channel (%s)\n", __func__, |
324 | prtd->dma_params->name); | 324 | prtd->dma_params->name); |
325 | prtd->dma_ch = imx_dma_request_by_prio(prtd->dma_params->name, | 325 | ret = imx_dma_request_by_prio(prtd->dma_params->name, DMA_PRIO_HIGH); |
326 | DMA_PRIO_HIGH); | 326 | if (ret < 0) { |
327 | if (prtd->dma_ch < 0) { | ||
328 | printk(KERN_ERR "Error %d requesting dma channel\n", ret); | 327 | printk(KERN_ERR "Error %d requesting dma channel\n", ret); |
329 | return ret; | 328 | return ret; |
330 | } | 329 | } |
330 | prtd->dma_ch = ret; | ||
331 | imx_dma_config_burstlen(prtd->dma_ch, | 331 | imx_dma_config_burstlen(prtd->dma_ch, |
332 | prtd->dma_params->watermark_level); | 332 | prtd->dma_params->watermark_level); |
333 | 333 | ||