* [asoc:for-5.7 21/76] wcd9335.c:undefined reference to `slim_stream_prepare'
@ 2020-02-17 21:54 kbuild test robot
2020-02-18 14:13 ` "imply" and modular target dependencies (was: Re: [asoc:for-5.7 21/76] wcd9335.c:undefined reference to `slim_stream_prepare') Geert Uytterhoeven
0 siblings, 1 reply; 3+ messages in thread
From: kbuild test robot @ 2020-02-17 21:54 UTC (permalink / raw)
To: Geert Uytterhoeven; +Cc: alsa-devel, Mark Brown, kbuild-all
[-- Attachment #1: Type: text/plain, Size: 4244 bytes --]
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>
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'
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'
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'
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'
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 46483 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* "imply" and modular target dependencies (was: Re: [asoc:for-5.7 21/76] wcd9335.c:undefined reference to `slim_stream_prepare')
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
2020-02-18 17:31 ` Masahiro Yamada
0 siblings, 1 reply; 3+ messages in thread
From: Geert Uytterhoeven @ 2020-02-18 14:13 UTC (permalink / raw)
To: kbuild test robot
Cc: ALSA Development Mailing List, Mark Brown, kbuild-all,
Masahiro Yamada, Arnd Bergmann
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
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: "imply" and modular target dependencies (was: Re: [asoc:for-5.7 21/76] wcd9335.c:undefined reference to `slim_stream_prepare')
2020-02-18 14:13 ` "imply" and modular target dependencies (was: Re: [asoc:for-5.7 21/76] wcd9335.c:undefined reference to `slim_stream_prepare') Geert Uytterhoeven
@ 2020-02-18 17:31 ` Masahiro Yamada
0 siblings, 0 replies; 3+ messages in thread
From: Masahiro Yamada @ 2020-02-18 17:31 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: ALSA Development Mailing List, Mark Brown, kbuild-all,
kbuild test robot, Arnd Bergmann
[-- Attachment #1: Type: text/plain, Size: 6274 bytes --]
Hi Geert,
On Tue, Feb 18, 2020 at 11:13 PM Geert Uytterhoeven
<geert@linux-m68k.org> wrote:
>
> 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?
I quickly checked the Kconfig code.
Surprisingly or not, 'imply' does not
obey 'depends on'.
I do not know if it is a bug, or intended behavior
but at least the current code works so.
(I am guessing this is a bug...)
I checked Documentation/kbuild/kconfig-language.rst
Unfortunately, this case is not documented.
=== === ============= ==============
FOO BAR BAZ's default choice for BAZ
=== === ============= ==============
n y n N/m/y
m y m M/y/n
y y y Y/n
y n * N
y m <undocumented> <undocumented>
=== === ============= ==============
I attached a patch, which changes the 'imply' to
follow the restriction of 'depends on',
but I need to take a closer look at this.
> Or am I missing something?
> Thanks!
--
Best Regards
Masahiro Yamada
[-- Attachment #2: 0001-kconfig-change-imply-to-respect-depends-on.patch --]
[-- Type: text/x-patch, Size: 1238 bytes --]
From 4ec542e201900c0dd1e06a49ca307f0fe9d2c4ca Mon Sep 17 00:00:00 2001
From: Masahiro Yamada <masahiroy@kernel.org>
Date: Wed, 19 Feb 2020 02:14:48 +0900
Subject: [PATCH] kconfig: change 'imply' to respect 'depends on'
Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
---
scripts/kconfig/symbol.c | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/scripts/kconfig/symbol.c b/scripts/kconfig/symbol.c
index 8d38b700b314..45003d7dfb0a 100644
--- a/scripts/kconfig/symbol.c
+++ b/scripts/kconfig/symbol.c
@@ -393,7 +393,9 @@ void sym_calc_value(struct symbol *sym)
}
if (sym->implied.tri != no) {
sym->flags |= SYMBOL_WRITE;
- newval.tri = EXPR_OR(newval.tri, sym->implied.tri);
+ newval.tri = EXPR_OR(newval.tri,
+ EXPR_AND(sym->dir_dep.tri,
+ sym->implied.tri));
}
}
calc_newval:
@@ -401,8 +403,7 @@ void sym_calc_value(struct symbol *sym)
sym_warn_unmet_dep(sym);
newval.tri = EXPR_OR(newval.tri, sym->rev_dep.tri);
}
- if (newval.tri == mod &&
- (sym_get_type(sym) == S_BOOLEAN || sym->implied.tri == yes))
+ if (newval.tri == mod && sym_get_type(sym) == S_BOOLEAN)
newval.tri = yes;
break;
case S_STRING:
--
2.17.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-02-18 17:33 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
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 ` "imply" and modular target dependencies (was: Re: [asoc:for-5.7 21/76] wcd9335.c:undefined reference to `slim_stream_prepare') Geert Uytterhoeven
2020-02-18 17:31 ` Masahiro Yamada
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).