<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, May 22, 2023 at 6:57 AM Randy Dunlap <<a href="mailto:rdunlap@infradead.org">rdunlap@infradead.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Both SND_MPC52xx_SOC_PCM030 and SND_MPC52xx_SOC_EFIKA select<br>
SND_SOC_MPC5200_AC97. The latter symbol depends on PPC_BESTCOMM,<br>
so the 2 former symbols should also depend on PPC_BESTCOMM since<br>
"select" does not follow any dependency chains.<br>
<br>
This prevents a kconfig warning and build errors:<br>
<br>
WARNING: unmet direct dependencies detected for SND_SOC_MPC5200_AC97<br>
  Depends on [n]: SOUND [=y] && !UML && SND [=m] && SND_SOC [=m] && SND_POWERPC_SOC [=m] && PPC_MPC52xx [=y] && PPC_BESTCOMM [=n]<br>
  Selected by [m]:<br>
  - SND_MPC52xx_SOC_PCM030 [=m] && SOUND [=y] && !UML && SND [=m] && SND_SOC [=m] && SND_POWERPC_SOC [=m] && PPC_MPC5200_SIMPLE [=y]<br>
  - SND_MPC52xx_SOC_EFIKA [=m] && SOUND [=y] && !UML && SND [=m] && SND_SOC [=m] && SND_POWERPC_SOC [=m] && PPC_EFIKA [=y]<br>
<br>
ERROR: modpost: "mpc5200_audio_dma_destroy" [sound/soc/fsl/mpc5200_psc_ac97.ko] undefined!<br>
ERROR: modpost: "mpc5200_audio_dma_create" [sound/soc/fsl/mpc5200_psc_ac97.ko] undefined!<br>
<br>
Fixes: 40d9ec14e7e1 ("ASoC: remove BROKEN from Efika and pcm030 fabric drivers")<br>
Signed-off-by: Randy Dunlap <<a href="mailto:rdunlap@infradead.org" target="_blank">rdunlap@infradead.org</a>><br>
Cc: Grant Likely <<a href="mailto:grant.likely@secretlab.ca" target="_blank">grant.likely@secretlab.ca</a>><br>
Cc: Mark Brown <<a href="mailto:broonie@kernel.org" target="_blank">broonie@kernel.org</a>><br>
Cc: Liam Girdwood <<a href="mailto:lgirdwood@gmail.com" target="_blank">lgirdwood@gmail.com</a>><br>
Cc: Shengjiu Wang <<a href="mailto:shengjiu.wang@gmail.com" target="_blank">shengjiu.wang@gmail.com</a>><br>
Cc: Xiubo Li <<a href="mailto:Xiubo.Lee@gmail.com" target="_blank">Xiubo.Lee@gmail.com</a>><br>
Cc: <a href="mailto:alsa-devel@alsa-project.org" target="_blank">alsa-devel@alsa-project.org</a><br>
Cc: <a href="mailto:linuxppc-dev@lists.ozlabs.org" target="_blank">linuxppc-dev@lists.ozlabs.org</a><br>
Cc: Jaroslav Kysela <<a href="mailto:perex@perex.cz" target="_blank">perex@perex.cz</a>><br>
Cc: Takashi Iwai <<a href="mailto:tiwai@suse.com" target="_blank">tiwai@suse.com</a>><br></blockquote><div><br></div><div>Acked-by: Shengjiu Wang <<a href="mailto:shengjiu.wang@gmail.com">shengjiu.wang@gmail.com</a>></div><div><br></div><div>Best regards</div><div>Wang shengjiu </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
---<br>
v2: use correct email address for Mark Brown.<br>
<br>
 sound/soc/fsl/Kconfig |    4 ++--<br>
 1 file changed, 2 insertions(+), 2 deletions(-)<br>
<br>
diff -- a/sound/soc/fsl/Kconfig b/sound/soc/fsl/Kconfig<br>
--- a/sound/soc/fsl/Kconfig<br>
+++ b/sound/soc/fsl/Kconfig<br>
@@ -243,7 +243,7 @@ config SND_SOC_MPC5200_AC97<br>
<br>
 config SND_MPC52xx_SOC_PCM030<br>
        tristate "SoC AC97 Audio support for Phytec pcm030 and WM9712"<br>
-       depends on PPC_MPC5200_SIMPLE<br>
+       depends on PPC_MPC5200_SIMPLE && PPC_BESTCOMM<br>
        select SND_SOC_MPC5200_AC97<br>
        select SND_SOC_WM9712<br>
        help<br>
@@ -252,7 +252,7 @@ config SND_MPC52xx_SOC_PCM030<br>
<br>
 config SND_MPC52xx_SOC_EFIKA<br>
        tristate "SoC AC97 Audio support for bbplan Efika and STAC9766"<br>
-       depends on PPC_EFIKA<br>
+       depends on PPC_EFIKA && PPC_BESTCOMM<br>
        select SND_SOC_MPC5200_AC97<br>
        select SND_SOC_STAC9766<br>
        help<br>
</blockquote></div></div>