On Thu, Jul 09, 2015 at 11:21:03AM +0300, Dan Carpenter wrote: > We need "i" to be signed or it leads to a forever loop in the error > handling. > > Fixes: fa3be9208dcb ('ASoC: fsl: fsl_spdif: Check for clk_prepare_enable() error') > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Acked-by: Nicolin Chen <nicoleotsuka@xxxxxxxxx> Thank you > diff --git a/sound/soc/fsl/fsl_spdif.c b/sound/soc/fsl/fsl_spdif.c > index 3ee4539..7a92f30 100644 > --- a/sound/soc/fsl/fsl_spdif.c > +++ b/sound/soc/fsl/fsl_spdif.c > @@ -454,7 +454,8 @@ static int fsl_spdif_startup(struct snd_pcm_substream *substream, > struct fsl_spdif_priv *spdif_priv = snd_soc_dai_get_drvdata(rtd->cpu_dai); > struct platform_device *pdev = spdif_priv->pdev; > struct regmap *regmap = spdif_priv->regmap; > - u32 scr, mask, i; > + u32 scr, mask; > + int i; > int ret; > > /* Reset module and interrupts only for first initialization */ -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html