From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jarkko Nikula Date: Mon, 20 Aug 2012 06:09:35 +0000 Subject: Re: [PATCH 4/9] sound/soc/omap/am3517evm.c: fix error return code Message-Id: <5031D49F.7050301@bitmer.com> List-Id: References: <1345359780-18647-1-git-send-email-Julia.Lawall@lip6.fr> <1345359780-18647-10-git-send-email-Julia.Lawall@lip6.fr> In-Reply-To: <1345359780-18647-10-git-send-email-Julia.Lawall@lip6.fr> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Julia Lawall Cc: Peter Ujfalusi , kernel-janitors@vger.kernel.org, Liam Girdwood , Mark Brown , Jaroslav Kysela , Takashi Iwai , linux-omap@vger.kernel.org, alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org On 08/19/2012 10:03 AM, Julia Lawall wrote: > From: Julia Lawall > > It was forgotten to initialize ret to the result of calling > snd_soc_dai_set_sysclk, unlike at the other calls in the same function. > > A simplified version of the semantic match that finds this problem is as > follows: (http://coccinelle.lip6.fr/) > > // > ( > if@p1 (\(ret < 0\|ret != 0\)) > { ... return ret; } > | > ret@p1 = 0 > ) > ... when != ret = e1 > when != &ret > *if(...) > { > ... when != ret = e2 > when forall > return ret; > } > > // > > Signed-off-by: Julia Lawall > > --- > sound/soc/omap/am3517evm.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/sound/soc/omap/am3517evm.c b/sound/soc/omap/am3517evm.c > index 009533a..df65f98 100644 > --- a/sound/soc/omap/am3517evm.c > +++ b/sound/soc/omap/am3517evm.c > @@ -59,7 +59,7 @@ static int am3517evm_hw_params(struct snd_pcm_substream *substream, > return ret; > } > > - snd_soc_dai_set_sysclk(cpu_dai, OMAP_MCBSP_FSR_SRC_FSX, 0, > + ret = snd_soc_dai_set_sysclk(cpu_dai, OMAP_MCBSP_FSR_SRC_FSX, 0, > SND_SOC_CLOCK_IN); > if (ret < 0) { > printk(KERN_ERR "can't set CPU system clock OMAP_MCBSP_FSR_SRC_FSX\n"); > Acked-by: Jarkko Nikula