diff options
| author | Shengjiu Wang <shengjiu.wang@freescale.com> | 2015-12-22 16:57:06 +0800 |
|---|---|---|
| committer | Nitin Garg <nitin.garg@nxp.com> | 2016-01-14 11:02:46 -0600 |
| commit | 729e59f8870c1094820793a4b027c1bff76ddeee (patch) | |
| tree | 334624f63c5f75e76e8d073a26d185dc33d10e63 /sound | |
| parent | 6537c6bda2d72ccb1712a5846ad4d3a2ec63bb64 (diff) | |
MLK-12043: ASoC: fsl: imx-cs42888: Add return value check
Report by coverity (CID 18428). The return value need be checked
for snd_soc_dai_set_sysclk().
Signed-off-by: Shengjiu Wang <shengjiu.wang@freescale.com>
Diffstat (limited to 'sound')
| -rw-r--r-- | sound/soc/fsl/imx-cs42888.c | 16 |
1 files changed, 13 insertions, 3 deletions
diff --git a/sound/soc/fsl/imx-cs42888.c b/sound/soc/fsl/imx-cs42888.c index 681788659cff..dea5a9a7e8f9 100644 --- a/sound/soc/fsl/imx-cs42888.c +++ b/sound/soc/fsl/imx-cs42888.c @@ -55,12 +55,22 @@ static int imx_cs42888_surround_hw_params(struct snd_pcm_substream *substream, dai_format = SND_SOC_DAIFMT_LEFT_J | SND_SOC_DAIFMT_NB_NF | SND_SOC_DAIFMT_CBS_CFS; if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK) - snd_soc_dai_set_sysclk(cpu_dai, ESAI_HCKT_EXTAL, + ret = snd_soc_dai_set_sysclk(cpu_dai, ESAI_HCKT_EXTAL, priv->mclk_freq, SND_SOC_CLOCK_OUT); else - snd_soc_dai_set_sysclk(cpu_dai, ESAI_HCKR_EXTAL, + ret = snd_soc_dai_set_sysclk(cpu_dai, ESAI_HCKR_EXTAL, priv->mclk_freq, SND_SOC_CLOCK_OUT); - snd_soc_dai_set_sysclk(codec_dai, 0, priv->mclk_freq, SND_SOC_CLOCK_IN); + if (ret) { + dev_err(dev, "failed to set cpu sysclk: %d\n", ret); + return ret; + } + + ret = snd_soc_dai_set_sysclk(codec_dai, 0, + priv->mclk_freq, SND_SOC_CLOCK_IN); + if (ret) { + dev_err(dev, "failed to set codec sysclk: %d\n", ret); + return ret; + } /* set cpu DAI configuration */ ret = snd_soc_dai_set_fmt(cpu_dai, dai_format); |