[alsa-devel] [PATCH] ASoC: qdsp6: q6asm-dai: clean up a return
Srinivas Kandagatla
srinivas.kandagatla at linaro.org
Mon Sep 10 12:04:40 CEST 2018
On 10/09/18 09:40, Dan Carpenter wrote:
> Smatch complains that if both "psubstream" and "csubstream" are NULL
> then "ret" is uninitialized. That probably can't happen, but it's
> cleaner to just return zero anyway so let's do that.
>
> Signed-off-by: Dan Carpenter <dan.carpenter at oracle.com>
Acked-by: Srinivas Kandagatla <srinivas.kandagatla at linaro.org>
>
> diff --git a/sound/soc/qcom/qdsp6/q6asm-dai.c b/sound/soc/qcom/qdsp6/q6asm-dai.c
> index 9db9a2944ef2..c75fab38905d 100644
> --- a/sound/soc/qcom/qdsp6/q6asm-dai.c
> +++ b/sound/soc/qcom/qdsp6/q6asm-dai.c
> @@ -493,7 +493,7 @@ static int q6asm_dai_pcm_new(struct snd_soc_pcm_runtime *rtd)
> }
> }
>
> - return ret;
> + return 0;
> }
>
> static void q6asm_dai_pcm_free(struct snd_pcm *pcm)
>
More information about the Alsa-devel
mailing list