On 10/22/17 9:45 PM, Takashi Iwai wrote:
On Sun, 22 Oct 2017 15:00:38 +0200,
Pierre-Louis Bossart wrote:
On 10/22/17 3:25 PM, Pierre-Louis Bossart wrote:
Not sure what happened here, the code was posted on an internal
mailing list which is supposed to be checked by the 0-day
infrastructure precisely to avoid this sort of issues before going
upstream...
Anyways, there is obviously a missing dependency on X86, this code
can't possibly work on extensa or s390.
I am really confused here, errors already happen with 4.13 (before my
patches) with the two suggested configs, so it's either that 01-day
added a new set of tests for extensa and s390 or didn't run tests
correctly before?
I guess it happens because now you changed a few Kconfig items to be
user-selectable by adding the prompt. Formerly, they were only
selected by the top-level drivers, and these top-level drivers have
the proper dependency on x86. But the lower layers don't have the
dependency, and now you allow to select them freely, hence such a
thing may happen.
If you take 4.13 and use the config/command line provided below it's
also broken... we'll fix this but I am not clear on why this is showing now.
Takashi
On 10/21/17 4:38 PM, kbuild test robot wrote:
tree:
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git
topic/intel
head: f6a118a800e35af2c63f90cbcc23093f4b53b3a2
commit: f6a118a800e35af2c63f90cbcc23093f4b53b3a2 [38/38] ASoC:
Intel: clarify Kconfig dependencies
config: xtensa-allyesconfig (attached as .config)
compiler: xtensa-linux-gcc (GCC) 4.9.0
reproduce:
wget
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross
-O ~/bin/make.cross
chmod +x ~/bin/make.cross
git checkout f6a118a800e35af2c63f90cbcc23093f4b53b3a2
# save the attached .config to linux build tree
make.cross ARCH=xtensa
All errors (new ones prefixed by >>):
sound/soc/intel/common/sst-firmware.c:1278:20: error: expected
declaration specifiers or '...' before string constant
MODULE_DESCRIPTION("Intel SST Firmware Loader");
^
sound/soc/intel/common/sst-firmware.c:1279:16: error: expected
declaration specifiers or '...' before string constant
MODULE_LICENSE("GPL v2");
^
--
sound/soc/intel/haswell/sst-haswell-pcm.c: In function
'create_adsp_page_table':
sound/soc/intel/haswell/sst-haswell-pcm.c:446:2: error: implicit
declaration of function 'snd_sgbuf_aligned_pages'
[-Werror=implicit-function-declaration]
pages = snd_sgbuf_aligned_pages(size);
^
cc1: some warnings being treated as errors
vim +1278 sound/soc/intel/common/sst-firmware.c
a395bdd6 Takashi Iwai 2016-07-11 1277
a395bdd6 Takashi Iwai 2016-07-11 @1278 MODULE_DESCRIPTION("Intel
SST Firmware Loader");
:::::: The code at line 1278 was first introduced by commit
:::::: a395bdd6b24b692adbce0df6510ec9f2af57573e ASoC: intel: Fix
sst-dsp dependency on dw stuff
:::::: TO: Takashi Iwai <tiwai@xxxxxxx>
:::::: CC: Mark Brown <broonie@xxxxxxxxxx>
---
0-DAY kernel test infrastructure Open Source
Technology Center
https://lists.01.org/pipermail/kbuild-all ; Intel
Corporation
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
