Hi Oswald,
kernel test robot noticed the following build errors:
[auto build test ERROR on tiwai-sound/for-next] [cannot apply to tiwai-sound/for-linus linus/master v6.3-rc7 next-20230421] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Oswald-Buddenhagen/ALSA-emu10... base: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next patch link: https://lore.kernel.org/r/20230422161021.1143903-8-oswald.buddenhagen%40gmx.... patch subject: [PATCH 7/7] ALSA: emu10k1: use more existing defines instead of open-coded numbers config: i386-allyesconfig (https://download.01.org/0day-ci/archive/20230423/202304230230.Hzwubvl5-lkp@i...) compiler: gcc-11 (Debian 11.3.0-8) 11.3.0 reproduce (this is a W=1 build): # https://github.com/intel-lab-lkp/linux/commit/e9c8b11aa4d8ccd105aba0217ff72e... git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Oswald-Buddenhagen/ALSA-emu10k1-comment-updates/20230423-002113 git checkout e9c8b11aa4d8ccd105aba0217ff72e87bad545bf # save the config file mkdir build_dir && cp config build_dir/.config make W=1 O=build_dir ARCH=i386 olddefconfig make W=1 O=build_dir ARCH=i386 SHELL=/bin/bash
If you fix the issue, kindly add following tag where applicable | Reported-by: kernel test robot lkp@intel.com | Link: https://lore.kernel.org/oe-kbuild-all/202304230230.Hzwubvl5-lkp@intel.com/
All errors (new ones prefixed by >>):
sound/pci/emu10k1/p16v.c: In function 'snd_p16v_pcm_prepare_capture':
sound/pci/emu10k1/p16v.c:313:38: error: 'A_I2S_CAPTURE_RATE' undeclared (first use in this function); did you mean 'A_I2S_CAPTURE_RATE_MASK'?
313 | snd_emu10k1_ptr_write(emu, A_I2S_CAPTURE_RATE, channel, A_I2S_CAPTURE_44100); | ^~~~~~~~~~~~~~~~~~ | A_I2S_CAPTURE_RATE_MASK sound/pci/emu10k1/p16v.c:313:38: note: each undeclared identifier is reported only once for each function it appears in
vim +313 sound/pci/emu10k1/p16v.c
295 296 /* prepare capture callback */ 297 static int snd_p16v_pcm_prepare_capture(struct snd_pcm_substream *substream) 298 { 299 struct snd_emu10k1 *emu = snd_pcm_substream_chip(substream); 300 struct snd_pcm_runtime *runtime = substream->runtime; 301 int channel = substream->pcm->device - emu->p16v_device_offset; 302 303 /* 304 dev_dbg(emu->card->dev, "prepare capture:channel_number=%d, rate=%d, " 305 "format=0x%x, channels=%d, buffer_size=%ld, period_size=%ld, " 306 "frames_to_bytes=%d\n", 307 channel, runtime->rate, runtime->format, runtime->channels, 308 runtime->buffer_size, runtime->period_size, 309 frames_to_bytes(runtime, 1)); 310 */ 311 switch (runtime->rate) { 312 case 44100:
313 snd_emu10k1_ptr_write(emu, A_I2S_CAPTURE_RATE, channel, A_I2S_CAPTURE_44100);
314 break; 315 case 96000: 316 snd_emu10k1_ptr_write(emu, A_I2S_CAPTURE_RATE, channel, A_I2S_CAPTURE_96000); 317 break; 318 case 192000: 319 snd_emu10k1_ptr_write(emu, A_I2S_CAPTURE_RATE, channel, A_I2S_CAPTURE_192000); 320 break; 321 case 48000: 322 default: 323 snd_emu10k1_ptr_write(emu, A_I2S_CAPTURE_RATE, channel, A_I2S_CAPTURE_48000); 324 break; 325 } 326 /* FIXME: Check emu->buffer.size before actually writing to it. */ 327 snd_emu10k1_ptr20_write(emu, CAPTURE_FIFO_POINTER, channel, 0); 328 snd_emu10k1_ptr20_write(emu, CAPTURE_DMA_ADDR, channel, runtime->dma_addr); 329 snd_emu10k1_ptr20_write(emu, CAPTURE_BUFFER_SIZE, channel, frames_to_bytes(runtime, runtime->buffer_size) << 16); // buffer size in bytes 330 snd_emu10k1_ptr20_write(emu, CAPTURE_POINTER, channel, 0); 331 //snd_emu10k1_ptr20_write(emu, CAPTURE_SOURCE, 0x0, 0x333300e4); /* Select MIC or Line in */ 332 //snd_emu10k1_ptr20_write(emu, EXTENDED_INT_MASK, 0, snd_emu10k1_ptr20_read(emu, EXTENDED_INT_MASK, 0) | (0x110000<<channel)); 333 334 return 0; 335 } 336