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 | |
| 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>
| -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 | ||
