diff options
| author |   Takashi Iwai <tiwai@suse.de> | 2015-04-13 14:23:29 +0200 |
|---|---|---|
| committer |   Mark Brown <broonie@kernel.org> | 2015-04-15 17:04:15 +0100 |
| commit | 96f05be37f4ece1dffba92d4ae079a486a4cf6df (patch) | |
| tree | 96a01feb916b411df2cc43600dec625f907b0b7f /sound | |
| parent | Merge remote-tracking branches 'asoc/topic/wm2200', 'asoc/topic/wm5100', 'asoc/topic/wm8731', 'asoc/topic/wm8804' and 'asoc/topic/wm8996' into asoc-next (diff) | |
| download | linux-dev-96f05be37f4ece1dffba92d4ae079a486a4cf6df.tar.xz linux-dev-96f05be37f4ece1dffba92d4ae079a486a4cf6df.zip | |
ASoC: qcom: Return an error for invalid PCM trigger command
Fix a compile warning
sound/soc/qcom/lpass-cpu.c: In function ‘lpass_cpu_daiops_trigger’:
sound/soc/qcom/lpass-cpu.c:224:2: warning: ‘ret’ may be used uninitialized in this function [-Wmaybe-uninitialized]
return ret;
^
Although switch () lists the most of existing cases, it's still better
to cover the rest as an error properly.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Acked-by: Kenneth Westfield <kwestfie@codeaurora.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound')
| -rw-r--r-- | sound/soc/qcom/lpass-cpu.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/soc/qcom/lpass-cpu.c b/sound/soc/qcom/lpass-cpu.c index 6698d058de29..dc790abaa331 100644 --- a/sound/soc/qcom/lpass-cpu.c +++ b/sound/soc/qcom/lpass-cpu.c @@ -194,7 +194,7 @@ static int lpass_cpu_daiops_trigger(struct snd_pcm_substream *substream, int cmd, struct snd_soc_dai *dai) { struct lpass_data *drvdata = snd_soc_dai_get_drvdata(dai); - int ret; + int ret = -EINVAL; switch (cmd) { case SNDRV_PCM_TRIGGER_START: |