Re: mmotm 2022-03-06-20-33 uploaded (sound/soc/intel/boards/)

[Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx>]

On 3/7/22 15:12, Randy Dunlap wrote:
> On 3/6/22 20:34, Andrew Morton wrote:
> > The mm-of-the-moment snapshot 2022-03-06-20-33 has been uploaded to
> >
> >     https://www.ozlabs.org/~akpm/mmotm/
> >
> > mmotm-readme.txt says
> >
> > README for mm-of-the-moment:
> >
> > https://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
> > https://ozlabs.org/~akpm/mmotm/series
>
> on x86_64:
>
> ERROR: modpost: "sof_dai_get_bclk" [sound/soc/intel/boards/snd-soc-intel-sof-cirrus-common.ko] undefined!
> ERROR: modpost: "sof_dai_get_mclk" [sound/soc/intel/boards/snd-soc-intel-sof-realtek-common.ko] undefined!
>
>
> Full randconfig file is attached.

Thanks Randy for the report. Indeed there's a problem with the 
SND_SOC_INTEL_SOF_SSP_AMP_MACH option. It should be conditional on at 
least one Intel/SOF platform being selected, as all the other platforms 
are already.

The following diff makes the problem go away:

diff --git a/sound/soc/intel/boards/Kconfig b/sound/soc/intel/boards/Kconfig

index 6884ddf9edad..81e012c164b0 100644

--- a/sound/soc/intel/boards/Kconfig

+++ b/sound/soc/intel/boards/Kconfig

@@ -615,6 +615,8 @@ config SND_SOC_INTEL_SOF_DA7219_MAX98373_MACH



 endif ## SND_SOC_SOF_JASPERLAKE



+if SND_SOC_SOF_HDA_LINK

+

 config SND_SOC_INTEL_SOF_SSP_AMP_MACH

        tristate "SOF with amplifiers in I2S Mode"

        depends on I2C && ACPI

@@ -631,6 +633,7 @@ config SND_SOC_INTEL_SOF_SSP_AMP_MACH

           with RT1308/CS35L41 I2S audio codec.

           Say Y if you have such a device.

           If unsure select "N".

+endif ## SND_SOC_SOF_HDA_LINK



 if SND_SOC_SOF_ELKHARTLAKE