Re: mmotm 2020-03-11-21-11 uploaded (sound/soc/codecs/wcd9335.c)

[Srinivas Kandagatla]

Adding+ Masahiro Yamada for more inputs w.r.t kconfig.


Kconfig side we have:

config SND_SOC_ALL_CODECS
        tristate "Build all ASoC CODEC drivers"
        imply SND_SOC_WCD9335

config SND_SOC_WCD9335
        tristate "WCD9335 Codec"
        depends on SLIMBUS
	...

The implied symbol SND_SOC_WCD9335 should be set based on direct 
dependency, However in this case, direct dependency SLIMBUS=m where as 
SND_SOC_WCD9335=y. I would have expected to be SND_SOC_WCD9335=m in this 
case.

Is this a valid possible case or a bug in Kconfig?


Thanks,
srini

On 12/03/2020 15:03, Randy Dunlap wrote:
> On 3/11/20 9:12 PM, Andrew Morton wrote:
> > The mm-of-the-moment snapshot 2020-03-11-21-11 has been uploaded to
> >
> >     http://www.ozlabs.org/~akpm/mmotm/
> >
> > mmotm-readme.txt says
> >
> > README for mm-of-the-moment:
> >
> > http://www.ozlabs.org/~akpm/mmotm/
> >
> > This is a snapshot of my -mm patch queue.  Uploaded at random hopefully
> > more than once a week.
> >
> > You will need quilt to apply these patches to the latest Linus release (5.x
> > or 5.x-rcY).  The series file is in broken-out.tar.gz and is duplicated in
> > http://ozlabs.org/~akpm/mmotm/series
> >
> > The file broken-out.tar.gz contains two datestamp files: .DATE and
> > .DATE-yyyy-mm-dd-hh-mm-ss.  Both contain the string yyyy-mm-dd-hh-mm-ss,
> > followed by the base kernel version against which this patch series is to
> > be applied.
> >
> > This tree is partially included in linux-next.  To see which patches are
> > included in linux-next, consult the `series' file.  Only the patches
> > within the #NEXT_PATCHES_START/#NEXT_PATCHES_END markers are included in
> > linux-next.
> >
> >
> > A full copy of the full kernel tree with the linux-next and mmotm patches
> > already applied is available through git within an hour of the mmotm
> > release.  Individual mmotm releases are tagged.  The master branch always
> > points to the latest release, so it's constantly rebasing.
> >
> > 	https://github.com/hnaz/linux-mm
> >
> > The directory http://www.ozlabs.org/~akpm/mmots/ (mm-of-the-second)
> > contains daily snapshots of the -mm tree.  It is updated more frequently
> > than mmotm, and is untested.
> >
> > A git copy of this tree is also available at
> >
> > 	https://github.com/hnaz/linux-mm
>
>
> on x86_64:
>
> ld: sound/soc/codecs/wcd9335.o: in function `wcd9335_trigger':
> wcd9335.c:(.text+0x451): undefined reference to `slim_stream_prepare'
> ld: wcd9335.c:(.text+0x465): undefined reference to `slim_stream_enable'
> ld: wcd9335.c:(.text+0x48f): undefined reference to `slim_stream_unprepare'
> ld: wcd9335.c:(.text+0x4a3): undefined reference to `slim_stream_disable'
> ld: sound/soc/codecs/wcd9335.o: in function `wcd9335_slim_status':
> wcd9335.c:(.text+0x23df): undefined reference to `of_slim_get_device'
> ld: wcd9335.c:(.text+0x2414): undefined reference to `slim_get_logical_addr'
> ld: wcd9335.c:(.text+0x2427): undefined reference to `__regmap_init_slimbus'
> ld: wcd9335.c:(.text+0x245f): undefined reference to `__regmap_init_slimbus'
> ld: sound/soc/codecs/wcd9335.o: in function `wcd9335_hw_params':
> wcd9335.c:(.text+0x3e05): undefined reference to `slim_stream_allocate'
> ld: sound/soc/codecs/wcd9335.o: in function `wcd9335_slim_driver_init':
> wcd9335.c:(.init.text+0x15): undefined reference to `__slim_driver_register'
> ld: sound/soc/codecs/wcd9335.o: in function `wcd9335_slim_driver_exit':
> wcd9335.c:(.exit.text+0x11): undefined reference to `slim_driver_unregister'
>
>
> Full randconfig file is attached.