summaryrefslogtreecommitdiffstats
path: root/sound
diff options
context:
space:
mode:
authorNicolin Chen <Guangyu.Chen@freescale.com>2013-12-20 16:41:00 +0800
committerMark Brown <broonie@linaro.org>2013-12-21 14:47:10 +0000
commit1fb2d9d7465bcbb519c582fa4a3bd04ff4fce2d2 (patch)
tree418826f3cf2f2929ea3969c446ca336a805d3df3 /sound
parentdd9f40602e96353c210805a99abd9af6abd28473 (diff)
downloadop-kernel-dev-1fb2d9d7465bcbb519c582fa4a3bd04ff4fce2d2.zip
op-kernel-dev-1fb2d9d7465bcbb519c582fa4a3bd04ff4fce2d2.tar.gz
ASoC: fsl_sai: Keep symmetry for clk_enable() and clk_disable()
There are two functions haven't clk_disable_unprepare() if having error. Thus fix them. Signed-off-by: Nicolin Chen <Guangyu.Chen@freescale.com> Reviewed-by: Xiubo Li <Li.Xiubo@freescale.com> Signed-off-by: Mark Brown <broonie@linaro.org>
Diffstat (limited to 'sound')
-rw-r--r--sound/soc/fsl/fsl_sai.c14
1 files changed, 8 insertions, 6 deletions
diff --git a/sound/soc/fsl/fsl_sai.c b/sound/soc/fsl/fsl_sai.c
index b8cdbf8..69a375f 100644
--- a/sound/soc/fsl/fsl_sai.c
+++ b/sound/soc/fsl/fsl_sai.c
@@ -111,7 +111,7 @@ static int fsl_sai_set_dai_sysclk(struct snd_soc_dai *cpu_dai,
dev_err(cpu_dai->dev,
"Cannot set SAI's transmitter sysclk: %d\n",
ret);
- return ret;
+ goto err_clk;
}
ret = fsl_sai_set_dai_sysclk_tr(cpu_dai, clk_id, freq,
@@ -120,12 +120,13 @@ static int fsl_sai_set_dai_sysclk(struct snd_soc_dai *cpu_dai,
dev_err(cpu_dai->dev,
"Cannot set SAI's receiver sysclk: %d\n",
ret);
- return ret;
+ goto err_clk;
}
+err_clk:
clk_disable_unprepare(sai->clk);
- return 0;
+ return ret;
}
static int fsl_sai_set_dai_fmt_tr(struct snd_soc_dai *cpu_dai,
@@ -222,7 +223,7 @@ static int fsl_sai_set_dai_fmt(struct snd_soc_dai *cpu_dai, unsigned int fmt)
dev_err(cpu_dai->dev,
"Cannot set SAI's transmitter format: %d\n",
ret);
- return ret;
+ goto err_clk;
}
ret = fsl_sai_set_dai_fmt_tr(cpu_dai, fmt, FSL_FMT_RECEIVER);
@@ -230,12 +231,13 @@ static int fsl_sai_set_dai_fmt(struct snd_soc_dai *cpu_dai, unsigned int fmt)
dev_err(cpu_dai->dev,
"Cannot set SAI's receiver format: %d\n",
ret);
- return ret;
+ goto err_clk;
}
+err_clk:
clk_disable_unprepare(sai->clk);
- return 0;
+ return ret;
}
static int fsl_sai_hw_params(struct snd_pcm_substream *substream,
OpenPOWER on IntegriCloud