Re: [PATCH] ASoC: fix unmet dependencies on PPC_BESTCOMM and SND_SOC_AC97_BUS
Julian Braha <[email protected]>
| Newsgroups | org.kernel.vger.linux-sound,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <[email protected]> |
Hi Rosen, On 8/2/26 22:14, Rosen Penev wrote: > On Sun, Aug 2, 2026 at 7:49 AM Julian Braha <[email protected]> wrote: >> >> SND_MPC52xx_SOC_PCM030 and SND_MPC52xx_SOC_EFIKA both select >> SND_SOC_MPC5200_AC97 without ensuring that its dependency on PPC_BESTCOMM is >> met. This causes unmet dependencies such as: >> >> WARNING: unmet direct dependencies detected for SND_SOC_MPC5200_AC97 >> Depends on [n]: SOUND [=m] && SND [=m] && SND_SOC [=m] && SND_POWERPC_SOC [=m] && PPC_BESTCOMM [=n] >> Selected by [m]: >> - SND_MPC52xx_SOC_PCM030 [=m] && SOUND [=m] && SND [=m] && SND_SOC [=m] && SND_POWERPC_SOC [=m] && PPC_MPC5200_SIMPLE [=y] >> >> SND_MPC52xx_SOC_EFIKA also selects SND_SOC_STAC9766 without ensuring its >> dependency on SND_SOC_AC97_BUS is met. >> >> Let's resolve all 3 unmet dependencies by adding the dependencies on >> PPC_BESTCOMM where needed, and selecting SND_SOC_AC97_BUS as other, similar >> options in this subsystem do. >> >> These unmet dependency bugs were detected by kconfirm, a static analysis tool >> for Kconfig. >> >> Fixes: a9262c4fd404 ("ASoC: Support for AC97 on Phytec pmc030 base board.") >> Fixes: 6ffee43ecf8b ("ASoC: Fabric bindings for STAC9766 on the Efika") >> Signed-off-by: Julian Braha <[email protected]> >> --- >> sound/soc/fsl/Kconfig | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) >> >> diff --git a/sound/soc/fsl/Kconfig b/sound/soc/fsl/Kconfig >> index 8ae59c094878..a5d7cf534391 100644 >> --- a/sound/soc/fsl/Kconfig >> +++ b/sound/soc/fsl/Kconfig >> @@ -243,7 +243,7 @@ config SND_SOC_MPC5200_AC97 >> >> config SND_MPC52xx_SOC_PCM030 >> tristate "SoC AC97 Audio support for Phytec pcm030 and WM9712" >> - depends on PPC_MPC5200_SIMPLE >> + depends on PPC_BESTCOMM >> select SND_SOC_MPC5200_AC97 >> select SND_SOC_WM9712 >> help >> @@ -253,6 +253,8 @@ config SND_MPC52xx_SOC_PCM030 >> config SND_MPC52xx_SOC_EFIKA >> tristate "SoC AC97 Audio support for bbplan Efika and STAC9766" >> depends on PPC_EFIKA >> + depends on PPC_BESTCOMM > does this really depend on PPC_BESTCOMM? Yes, BestComm is the co-processor found on Freescale MPC5200. - Julian Braha