From: Geert Uytterhoeven <geert@linux-m68k.org>
To: kbuild test robot <lkp@intel.com>
Cc: ALSA Development Mailing List <alsa-devel@alsa-project.org>,
Mark Brown <broonie@kernel.org>,
kbuild-all@lists.01.org, Masahiro Yamada <masahiroy@kernel.org>,
Arnd Bergmann <arnd@arndb.de>
Subject: "imply" and modular target dependencies (was: Re: [asoc:for-5.7 21/76] wcd9335.c:undefined reference to `slim_stream_prepare')
Date: Tue, 18 Feb 2020 15:13:33 +0100 [thread overview]
Message-ID: <CAMuHMdW8SvDgQJyenTtEm4Xn2Ma6PK9pfwKR2_gn60t2AqNWXg@mail.gmail.com> (raw)
In-Reply-To: <202002180512.4RU8Rxge%lkp@intel.com>
On Mon, Feb 17, 2020 at 10:55 PM kbuild test robot <lkp@intel.com> wrote:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-5.7
> head: 3cd23f021e2e5f3350125abcb39f12430df87d06
> commit: ea00d95200d02ece71f5814d41b14f2eb16d598b [21/76] ASoC: Use imply for SND_SOC_ALL_CODECS
> config: x86_64-randconfig-s2-20200217 (attached as .config)
> compiler: gcc-7 (Debian 7.5.0-5) 7.5.0
> reproduce:
> git checkout ea00d95200d02ece71f5814d41b14f2eb16d598b
> # save the attached .config to linux build tree
> make ARCH=x86_64
>
> If you fix the issue, kindly add following tag
> Reported-by: kbuild test robot <lkp@intel.com>
Thanks for your report!
> All errors (new ones prefixed by >>):
>
> ld: sound/soc/codecs/ab8500-codec.o: in function `ab8500_codec_read_reg':
> ab8500-codec.c:(.text+0x479): undefined reference to `abx500_get_register_interruptible'
> ld: sound/soc/codecs/ab8500-codec.o: in function `ab8500_codec_probe':
> ab8500-codec.c:(.text+0xdf1): undefined reference to `abx500_get_register_interruptible'
> ld: ab8500-codec.c:(.text+0xe18): undefined reference to `abx500_set_register_interruptible'
> ld: sound/soc/codecs/ab8500-codec.o: in function `ab8500_codec_write_reg':
> ab8500-codec.c:(.text+0x4b1): undefined reference to `abx500_set_register_interruptible'
The above are fixed in next-20200218.
> ld: sound/soc/codecs/mc13783.o: in function `mc13783_probe':
> mc13783.c:(.text+0x24c): undefined reference to `mc13xxx_reg_write'
> ld: mc13783.c:(.text+0x25e): undefined reference to `mc13xxx_reg_write'
> ld: mc13783.c:(.text+0x270): undefined reference to `mc13xxx_reg_write'
> ld: mc13783.c:(.text+0x282): undefined reference to `mc13xxx_reg_write'
> ld: mc13783.c:(.text+0x294): undefined reference to `mc13xxx_reg_write'
> ld: sound/soc/codecs/mc13783.o:mc13783.c:(.text+0x2a6): more undefined references to `mc13xxx_reg_write' follow
> ld: sound/soc/codecs/mc13783.o: in function `mc13783_probe':
> mc13783.c:(.text+0x2c5): undefined reference to `mc13xxx_reg_rmw'
> ld: mc13783.c:(.text+0x2e4): undefined reference to `mc13xxx_reg_rmw'
> ld: sound/soc/codecs/mc13783.o: in function `mc13783_remove':
> mc13783.c:(.text+0x212): undefined reference to `mc13xxx_reg_rmw'
CONFIG_SND_SOC_MC13783=y
CONFIG_MFD_MC13XXX=m
> ld: sound/soc/codecs/wcd9335.o: in function `wcd9335_trigger':
> >> wcd9335.c:(.text+0x323): undefined reference to `slim_stream_prepare'
> >> ld: wcd9335.c:(.text+0x32f): undefined reference to `slim_stream_enable'
> >> ld: wcd9335.c:(.text+0x345): undefined reference to `slim_stream_unprepare'
> >> ld: wcd9335.c:(.text+0x351): undefined reference to `slim_stream_disable'
> ld: sound/soc/codecs/wcd9335.o: in function `wcd9335_slim_status':
> >> wcd9335.c:(.text+0x1970): undefined reference to `of_slim_get_device'
> >> ld: wcd9335.c:(.text+0x1995): undefined reference to `slim_get_logical_addr'
> >> ld: wcd9335.c:(.text+0x19b2): undefined reference to `__regmap_init_slimbus'
> ld: wcd9335.c:(.text+0x19f3): undefined reference to `__regmap_init_slimbus'
> ld: sound/soc/codecs/wcd9335.o: in function `wcd9335_hw_params':
> >> wcd9335.c:(.text+0x348f): undefined reference to `slim_stream_allocate'
> ld: sound/soc/codecs/wcd9335.o: in function `wcd9335_slim_driver_init':
> >> wcd9335.c:(.init.text+0x3c): undefined reference to `__slim_driver_register'
> ld: sound/soc/codecs/wcd9335.o: in function `wcd9335_slim_driver_exit':
> >> wcd9335.c:(.exit.text+0x8): undefined reference to `slim_driver_unregister'
CONFIG_SND_SOC_WCD9335=y
CONFIG_SLIMBUS=m
> ld: sound/soc/codecs/wcd934x.o: in function `wcd934x_trigger':
> >> wcd934x.c:(.text+0x30d): undefined reference to `slim_stream_prepare'
> >> ld: wcd934x.c:(.text+0x319): undefined reference to `slim_stream_enable'
> >> ld: wcd934x.c:(.text+0x32f): undefined reference to `slim_stream_unprepare'
> >> ld: wcd934x.c:(.text+0x33b): undefined reference to `slim_stream_disable'
> ld: sound/soc/codecs/wcd934x.o: in function `wcd934x_hw_params':
> >> wcd934x.c:(.text+0x2620): undefined reference to `slim_stream_allocate'
> ld: sound/soc/codecs/wcd934x.o: in function `wcd934x_codec_probe':
> >> wcd934x.c:(.text+0x27ee): undefined reference to `of_slim_get_device'
> >> ld: wcd934x.c:(.text+0x2818): undefined reference to `slim_get_logical_addr'
> >> ld: wcd934x.c:(.text+0x2836): undefined reference to `__regmap_init_slimbus'
CONFIG_SND_SOC_WCD934X=y
CONFIG_SLIMBUS=m
All CONFIG_SND_SOC_* symbols above are tristate and have dependencies
on the modular symbols.
However, the "imply" statements in SND_SOC_ALL_CODECS force all of them
builtin, despite their dependencies being modular.
So while "imply" is meant to be a softer form of "select", obeying target
dependencies, it does not seem to obey modular tristate dependencies?
Or am I missing something?
Thanks!
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
next prev parent reply other threads:[~2020-02-18 14:14 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-17 21:54 [asoc:for-5.7 21/76] wcd9335.c:undefined reference to `slim_stream_prepare' kbuild test robot
2020-02-18 14:13 ` Geert Uytterhoeven [this message]
2020-02-18 17:31 ` "imply" and modular target dependencies (was: Re: [asoc:for-5.7 21/76] wcd9335.c:undefined reference to `slim_stream_prepare') Masahiro Yamada
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAMuHMdW8SvDgQJyenTtEm4Xn2Ma6PK9pfwKR2_gn60t2AqNWXg@mail.gmail.com \
--to=geert@linux-m68k.org \
--cc=alsa-devel@alsa-project.org \
--cc=arnd@arndb.de \
--cc=broonie@kernel.org \
--cc=kbuild-all@lists.01.org \
--cc=lkp@intel.com \
--cc=masahiroy@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).