Re: [PATCH v2 2/2] ASoC: fsl_ssi: add 20-bit sample format for AC'97 and use it for capture

[kbuild test robot]

Hi Maciej,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on sound/for-next]
[also build test ERROR on v4.14 next-20171124]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Maciej-S-Szmigiero/ALSA-pcm-add-SNDRV_PCM_FORMAT_-S-U-20_4/20171125-082039
base:   https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next
config: x86_64-randconfig-x012-201747 (attached as .config)
compiler: gcc-6 (Debian 6.4.0-9) 6.4.0 20171026
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All errors (new ones prefixed by >>):

>> sound/soc//fsl/fsl_ssi.c:1237:37: error: 'SNDRV_PCM_FMTBIT_S20' undeclared here (not in a function)
      .formats = SNDRV_PCM_FMTBIT_S16 | SNDRV_PCM_FMTBIT_S20,
                                        ^~~~~~~~~~~~~~~~~~~~

vim +/SNDRV_PCM_FMTBIT_S20 +1237 sound/soc//fsl/fsl_ssi.c

  1228	
  1229	static struct snd_soc_dai_driver fsl_ssi_ac97_dai = {
  1230		.bus_control = true,
  1231		.probe = fsl_ssi_dai_probe,
  1232		.playback = {
  1233			.stream_name = "AC97 Playback",
  1234			.channels_min = 2,
  1235			.channels_max = 2,
  1236			.rates = SNDRV_PCM_RATE_8000_48000,
> 1237			.formats = SNDRV_PCM_FMTBIT_S16 | SNDRV_PCM_FMTBIT_S20,
  1238		},
  1239		.capture = {
  1240			.stream_name = "AC97 Capture",
  1241			.channels_min = 2,
  1242			.channels_max = 2,
  1243			.rates = SNDRV_PCM_RATE_48000,
  1244			/* 16-bit capture is broken (errata ERR003778) */
  1245			.formats = SNDRV_PCM_FMTBIT_S20,
  1246		},
  1247		.ops = &fsl_ssi_dai_ops,
  1248	};
  1249	
  1250	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip