linux-kernel - Re: [alsa-devel] [PATCH 06/10] ASoC: img: Add driver for parallel output controller

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <201509302334.zqf2LD5F%fengguang.wu@intel.com>
Date:	Wed, 30 Sep 2015 23:48:00 +0800
From:	kbuild test robot <lkp@...el.com>
To:	Damien Horsley <Damien.Horsley@...tec.com>
Cc:	kbuild-all@...org, alsa-devel@...a-project.org,
	Mark Rutland <mark.rutland@....com>,
	devicetree@...r.kernel.org, Pawel Moll <pawel.moll@....com>,
	Ian Campbell <ijc+devicetree@...lion.org.uk>,
	linux-kernel@...r.kernel.org, Mark Brown <broonie@...nel.org>,
	Takashi Iwai <tiwai@...e.com>,
	Liam Girdwood <lgirdwood@...il.com>,
	Rob Herring <robh+dt@...nel.org>,
	Kumar Gala <galak@...eaurora.org>,
	"Damien.Horsley" <Damien.Horsley@...tec.com>
Subject: Re: [alsa-devel] [PATCH 06/10] ASoC: img: Add driver for parallel
 output controller

Hi Damien.Horsley,

[auto build test results on v4.3-rc3 -- if it's inappropriate base, please ignore]

reproduce:
  # apt-get install sparse
  make ARCH=x86_64 allmodconfig
  make C=1 CF=-D__CHECK_ENDIAN__


sparse warnings: (new ones prefixed by >>)

>> sound/soc/img/img-parallel-out.c:198:16: sparse: incorrect type in assignment (different base types)
   sound/soc/img/img-parallel-out.c:198:16:    expected unsigned int [unsigned] format
   sound/soc/img/img-parallel-out.c:198:16:    got restricted snd_pcm_format_t

vim +198 sound/soc/img/img-parallel-out.c

   182		}
   183	
   184		spin_unlock_irqrestore(&prl->lock, flags);
   185	
   186		return ret;
   187	}
   188	
   189	static int img_prl_out_hw_params(struct snd_pcm_substream *substream,
   190		struct snd_pcm_hw_params *params, struct snd_soc_dai *dai)
   191	{
   192		struct img_prl_out *prl = snd_soc_dai_get_drvdata(dai);
   193		unsigned int rate, format, channels;
   194		u32 reg, reg_set = 0;
   195		unsigned long flags;
   196	
   197		rate = params_rate(params);
 > 198		format = params_format(params);
   199		channels = params_channels(params);
   200	
   201		switch (params_format(params)) {
   202		case SNDRV_PCM_FORMAT_S32_LE:
   203			reg_set |= IMG_PRL_OUT_CTL_PACKH_MASK;
   204			break;
   205		case SNDRV_PCM_FORMAT_S24_LE:
   206			break;

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/