[alsa-devel] [PATCH] [RFC] ASoC: Use imply for SND_SOC_ALL_CODECS
Currently SND_SOC_ALL_CODECS selects the config symbols for all codec drivers. As "select" bypasses dependencies, lots of "select" statements need explicit dependencies, which are hard to get right, and hard to maintain[*].
Fix this by using "imply" instead, which is a weak version of "select", and which obeys dependencies of target symbols.
Add dependencies to invisible symbols that are currently selected only if their dependencies are fulfilled.
[*] See e.g. commit 13426feaf46c48fc ("ASoC: wcd934x: Add missing COMMON_CLK dependency to SND_SOC_ALL_CODECS").
Signed-off-by: Geert Uytterhoeven geert@linux-m68k.org --- Does this sound like a good solution? Anything I'm missing w.r.t. the "imply" semantics? FIXME Nothing but SND_SOC_ALL_CODECS selects SND_SOC_WM8400!
Tested with m68k/allmodconfig and m68k/allyesconfig only. Hence some other invisible symbols may still have missing dependencies. --- sound/soc/codecs/Kconfig | 520 ++++++++++++++++++++------------------- 1 file changed, 263 insertions(+), 257 deletions(-)
diff --git a/sound/soc/codecs/Kconfig b/sound/soc/codecs/Kconfig index 7e90f5d830971309..7a14b1c416b55e46 100644 --- a/sound/soc/codecs/Kconfig +++ b/sound/soc/codecs/Kconfig @@ -14,262 +14,262 @@ menu "CODEC drivers" config SND_SOC_ALL_CODECS tristate "Build all ASoC CODEC drivers" depends on COMPILE_TEST - select SND_SOC_88PM860X if MFD_88PM860X - select SND_SOC_L3 - select SND_SOC_AB8500_CODEC if ABX500_CORE - select SND_SOC_AC97_CODEC - select SND_SOC_AD1836 if SPI_MASTER - select SND_SOC_AD193X_SPI if SPI_MASTER - select SND_SOC_AD193X_I2C if I2C - select SND_SOC_AD1980 if SND_SOC_AC97_BUS - select SND_SOC_AD73311 - select SND_SOC_ADAU1373 if I2C - select SND_SOC_ADAU1761_I2C if I2C - select SND_SOC_ADAU1761_SPI if SPI - select SND_SOC_ADAU1781_I2C if I2C - select SND_SOC_ADAU1781_SPI if SPI - select SND_SOC_ADAV801 if SPI_MASTER - select SND_SOC_ADAV803 if I2C - select SND_SOC_ADAU1977_SPI if SPI_MASTER - select SND_SOC_ADAU1977_I2C if I2C - select SND_SOC_ADAU1701 if I2C - select SND_SOC_ADAU7002 - select SND_SOC_ADAU7118_I2C if I2C - select SND_SOC_ADAU7118_HW - select SND_SOC_ADS117X - select SND_SOC_AK4104 if SPI_MASTER - select SND_SOC_AK4118 if I2C - select SND_SOC_AK4458 if I2C - select SND_SOC_AK4535 if I2C - select SND_SOC_AK4554 - select SND_SOC_AK4613 if I2C - select SND_SOC_AK4641 if I2C - select SND_SOC_AK4642 if I2C - select SND_SOC_AK4671 if I2C - select SND_SOC_AK5386 - select SND_SOC_AK5558 if I2C - select SND_SOC_ALC5623 if I2C - select SND_SOC_ALC5632 if I2C - select SND_SOC_BT_SCO - select SND_SOC_BD28623 - select SND_SOC_CQ0093VC - select SND_SOC_CROS_EC_CODEC if CROS_EC - select SND_SOC_CS35L32 if I2C - select SND_SOC_CS35L33 if I2C - select SND_SOC_CS35L34 if I2C - select SND_SOC_CS35L35 if I2C - select SND_SOC_CS35L36 if I2C - select SND_SOC_CS42L42 if I2C - select SND_SOC_CS42L51_I2C if I2C - select SND_SOC_CS42L52 if I2C && INPUT - select SND_SOC_CS42L56 if I2C && INPUT - select SND_SOC_CS42L73 if I2C - select SND_SOC_CS4265 if I2C - select SND_SOC_CS4270 if I2C - select SND_SOC_CS4271_I2C if I2C - select SND_SOC_CS4271_SPI if SPI_MASTER - select SND_SOC_CS42XX8_I2C if I2C - select SND_SOC_CS43130 if I2C - select SND_SOC_CS4341 if SND_SOC_I2C_AND_SPI - select SND_SOC_CS4349 if I2C - select SND_SOC_CS47L15 if MFD_CS47L15 - select SND_SOC_CS47L24 if MFD_CS47L24 - select SND_SOC_CS47L35 if MFD_CS47L35 - select SND_SOC_CS47L85 if MFD_CS47L85 - select SND_SOC_CS47L90 if MFD_CS47L90 - select SND_SOC_CS47L92 if MFD_CS47L92 - select SND_SOC_CS53L30 if I2C - select SND_SOC_CX20442 if TTY - select SND_SOC_CX2072X if I2C - select SND_SOC_DA7210 if SND_SOC_I2C_AND_SPI - select SND_SOC_DA7213 if I2C - select SND_SOC_DA7218 if I2C - select SND_SOC_DA7219 if I2C - select SND_SOC_DA732X if I2C - select SND_SOC_DA9055 if I2C - select SND_SOC_DMIC if GPIOLIB - select SND_SOC_ES8316 if I2C - select SND_SOC_ES8328_SPI if SPI_MASTER - select SND_SOC_ES8328_I2C if I2C - select SND_SOC_ES7134 - select SND_SOC_ES7241 - select SND_SOC_GTM601 - select SND_SOC_HDAC_HDMI - select SND_SOC_HDAC_HDA - select SND_SOC_ICS43432 - select SND_SOC_INNO_RK3036 - select SND_SOC_ISABELLE if I2C - select SND_SOC_JZ4740_CODEC - select SND_SOC_JZ4725B_CODEC - select SND_SOC_JZ4770_CODEC - select SND_SOC_LM4857 if I2C - select SND_SOC_LM49453 if I2C - select SND_SOC_LOCHNAGAR_SC if MFD_LOCHNAGAR - select SND_SOC_MAX98088 if I2C - select SND_SOC_MAX98090 if I2C - select SND_SOC_MAX98095 if I2C - select SND_SOC_MAX98357A if GPIOLIB - select SND_SOC_MAX98371 if I2C - select SND_SOC_MAX98504 if I2C - select SND_SOC_MAX9867 if I2C - select SND_SOC_MAX98925 if I2C - select SND_SOC_MAX98926 if I2C - select SND_SOC_MAX98927 if I2C - select SND_SOC_MAX98373 if I2C - select SND_SOC_MAX9850 if I2C - select SND_SOC_MAX9860 if I2C - select SND_SOC_MAX9759 - select SND_SOC_MAX9768 if I2C - select SND_SOC_MAX9877 if I2C - select SND_SOC_MC13783 if MFD_MC13XXX - select SND_SOC_ML26124 if I2C - select SND_SOC_MT6351 if MTK_PMIC_WRAP - select SND_SOC_MT6358 if MTK_PMIC_WRAP - select SND_SOC_MT6660 if I2C - select SND_SOC_NAU8540 if I2C - select SND_SOC_NAU8810 if I2C - select SND_SOC_NAU8822 if I2C - select SND_SOC_NAU8824 if I2C - select SND_SOC_NAU8825 if I2C - select SND_SOC_HDMI_CODEC - select SND_SOC_PCM1681 if I2C - select SND_SOC_PCM1789_I2C if I2C - select SND_SOC_PCM179X_I2C if I2C - select SND_SOC_PCM179X_SPI if SPI_MASTER - select SND_SOC_PCM186X_I2C if I2C - select SND_SOC_PCM186X_SPI if SPI_MASTER - select SND_SOC_PCM3008 - select SND_SOC_PCM3060_I2C if I2C - select SND_SOC_PCM3060_SPI if SPI_MASTER - select SND_SOC_PCM3168A_I2C if I2C - select SND_SOC_PCM3168A_SPI if SPI_MASTER - select SND_SOC_PCM5102A - select SND_SOC_PCM512x_I2C if I2C - select SND_SOC_PCM512x_SPI if SPI_MASTER - select SND_SOC_RK3328 - select SND_SOC_RT274 if I2C - select SND_SOC_RT286 if I2C - select SND_SOC_RT298 if I2C - select SND_SOC_RT1011 if I2C - select SND_SOC_RT1015 if I2C - select SND_SOC_RT1305 if I2C - select SND_SOC_RT1308 if I2C - select SND_SOC_RT5514 if I2C - select SND_SOC_RT5616 if I2C - select SND_SOC_RT5631 if I2C - select SND_SOC_RT5640 if I2C - select SND_SOC_RT5645 if I2C - select SND_SOC_RT5651 if I2C - select SND_SOC_RT5659 if I2C - select SND_SOC_RT5660 if I2C - select SND_SOC_RT5663 if I2C - select SND_SOC_RT5665 if I2C - select SND_SOC_RT5668 if I2C - select SND_SOC_RT5670 if I2C - select SND_SOC_RT5677 if I2C && SPI_MASTER - select SND_SOC_RT5682 if I2C - select SND_SOC_RT700_SDW if SOUNDWIRE - select SND_SOC_RT711_SDW if SOUNDWIRE - select SND_SOC_RT715_SDW if SOUNDWIRE - select SND_SOC_RT1308_SDW if SOUNDWIRE - select SND_SOC_SGTL5000 if I2C - select SND_SOC_SI476X if MFD_SI476X_CORE - select SND_SOC_SIMPLE_AMPLIFIER - select SND_SOC_SIRF_AUDIO_CODEC - select SND_SOC_SPDIF - select SND_SOC_SSM2305 - select SND_SOC_SSM2518 if I2C - select SND_SOC_SSM2602_SPI if SPI_MASTER - select SND_SOC_SSM2602_I2C if I2C - select SND_SOC_SSM4567 if I2C - select SND_SOC_STA32X if I2C - select SND_SOC_STA350 if I2C - select SND_SOC_STA529 if I2C - select SND_SOC_STAC9766 if SND_SOC_AC97_BUS - select SND_SOC_STI_SAS - select SND_SOC_TAS2552 if I2C - select SND_SOC_TAS2562 if I2C - select SND_SOC_TAS2770 if I2C - select SND_SOC_TAS5086 if I2C - select SND_SOC_TAS571X if I2C - select SND_SOC_TAS5720 if I2C - select SND_SOC_TAS6424 if I2C - select SND_SOC_TDA7419 if I2C - select SND_SOC_TFA9879 if I2C - select SND_SOC_TLV320AIC23_I2C if I2C - select SND_SOC_TLV320AIC23_SPI if SPI_MASTER - select SND_SOC_TLV320AIC26 if SPI_MASTER - select SND_SOC_TLV320AIC31XX if I2C - select SND_SOC_TLV320AIC32X4_I2C if I2C && COMMON_CLK - select SND_SOC_TLV320AIC32X4_SPI if SPI_MASTER && COMMON_CLK - select SND_SOC_TLV320AIC3X if I2C - select SND_SOC_TPA6130A2 if I2C - select SND_SOC_TLV320DAC33 if I2C - select SND_SOC_TSCS42XX if I2C - select SND_SOC_TSCS454 if I2C - select SND_SOC_TS3A227E if I2C - select SND_SOC_TWL4030 if TWL4030_CORE - select SND_SOC_TWL6040 if TWL6040_CORE - select SND_SOC_UDA1334 if GPIOLIB - select SND_SOC_UDA134X - select SND_SOC_UDA1380 if I2C - select SND_SOC_WCD9335 if SLIMBUS - select SND_SOC_WCD934X if MFD_WCD934X && COMMON_CLK - select SND_SOC_WL1273 if MFD_WL1273_CORE - select SND_SOC_WM0010 if SPI_MASTER - select SND_SOC_WM1250_EV1 if I2C - select SND_SOC_WM2000 if I2C - select SND_SOC_WM2200 if I2C - select SND_SOC_WM5100 if I2C - select SND_SOC_WM5102 if MFD_WM5102 - select SND_SOC_WM5110 if MFD_WM5110 - select SND_SOC_WM8350 if MFD_WM8350 - select SND_SOC_WM8400 if MFD_WM8400 - select SND_SOC_WM8510 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8523 if I2C - select SND_SOC_WM8524 if GPIOLIB - select SND_SOC_WM8580 if I2C - select SND_SOC_WM8711 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8727 - select SND_SOC_WM8728 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8731 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8737 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8741 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8750 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8753 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8770 if SPI_MASTER - select SND_SOC_WM8776 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8782 - select SND_SOC_WM8804_I2C if I2C - select SND_SOC_WM8804_SPI if SPI_MASTER - select SND_SOC_WM8900 if I2C - select SND_SOC_WM8903 if I2C - select SND_SOC_WM8904 if I2C - select SND_SOC_WM8940 if I2C - select SND_SOC_WM8955 if I2C - select SND_SOC_WM8960 if I2C - select SND_SOC_WM8961 if I2C - select SND_SOC_WM8962 if I2C && INPUT - select SND_SOC_WM8971 if I2C - select SND_SOC_WM8974 if I2C - select SND_SOC_WM8978 if I2C - select SND_SOC_WM8983 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8985 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8988 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8990 if I2C - select SND_SOC_WM8991 if I2C - select SND_SOC_WM8993 if I2C - select SND_SOC_WM8994 if MFD_WM8994 - select SND_SOC_WM8995 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8996 if I2C - select SND_SOC_WM8997 if MFD_WM8997 - select SND_SOC_WM8998 if MFD_WM8998 - select SND_SOC_WM9081 if I2C - select SND_SOC_WM9090 if I2C - select SND_SOC_WM9705 if (SND_SOC_AC97_BUS || SND_SOC_AC97_BUS_NEW) - select SND_SOC_WM9712 if (SND_SOC_AC97_BUS || SND_SOC_AC97_BUS_NEW) - select SND_SOC_WM9713 if (SND_SOC_AC97_BUS || SND_SOC_AC97_BUS_NEW) - select SND_SOC_WSA881X if SOUNDWIRE + imply SND_SOC_88PM860X + imply SND_SOC_L3 + imply SND_SOC_AB8500_CODEC + imply SND_SOC_AC97_CODEC + imply SND_SOC_AD1836 + imply SND_SOC_AD193X_SPI + imply SND_SOC_AD193X_I2C + imply SND_SOC_AD1980 + imply SND_SOC_AD73311 + imply SND_SOC_ADAU1373 + imply SND_SOC_ADAU1761_I2C + imply SND_SOC_ADAU1761_SPI + imply SND_SOC_ADAU1781_I2C + imply SND_SOC_ADAU1781_SPI + imply SND_SOC_ADAV801 + imply SND_SOC_ADAV803 + imply SND_SOC_ADAU1977_SPI + imply SND_SOC_ADAU1977_I2C + imply SND_SOC_ADAU1701 + imply SND_SOC_ADAU7002 + imply SND_SOC_ADAU7118_I2C + imply SND_SOC_ADAU7118_HW + imply SND_SOC_ADS117X + imply SND_SOC_AK4104 + imply SND_SOC_AK4118 + imply SND_SOC_AK4458 + imply SND_SOC_AK4535 + imply SND_SOC_AK4554 + imply SND_SOC_AK4613 + imply SND_SOC_AK4641 + imply SND_SOC_AK4642 + imply SND_SOC_AK4671 + imply SND_SOC_AK5386 + imply SND_SOC_AK5558 + imply SND_SOC_ALC5623 + imply SND_SOC_ALC5632 + imply SND_SOC_BT_SCO + imply SND_SOC_BD28623 + imply SND_SOC_CQ0093VC + imply SND_SOC_CROS_EC_CODEC + imply SND_SOC_CS35L32 + imply SND_SOC_CS35L33 + imply SND_SOC_CS35L34 + imply SND_SOC_CS35L35 + imply SND_SOC_CS35L36 + imply SND_SOC_CS42L42 + imply SND_SOC_CS42L51_I2C + imply SND_SOC_CS42L52 + imply SND_SOC_CS42L56 + imply SND_SOC_CS42L73 + imply SND_SOC_CS4265 + imply SND_SOC_CS4270 + imply SND_SOC_CS4271_I2C + imply SND_SOC_CS4271_SPI + imply SND_SOC_CS42XX8_I2C + imply SND_SOC_CS43130 + imply SND_SOC_CS4341 + imply SND_SOC_CS4349 + imply SND_SOC_CS47L15 + imply SND_SOC_CS47L24 + imply SND_SOC_CS47L35 + imply SND_SOC_CS47L85 + imply SND_SOC_CS47L90 + imply SND_SOC_CS47L92 + imply SND_SOC_CS53L30 + imply SND_SOC_CX20442 + imply SND_SOC_CX2072X + imply SND_SOC_DA7210 + imply SND_SOC_DA7213 + imply SND_SOC_DA7218 + imply SND_SOC_DA7219 + imply SND_SOC_DA732X + imply SND_SOC_DA9055 + imply SND_SOC_DMIC + imply SND_SOC_ES8316 + imply SND_SOC_ES8328_SPI + imply SND_SOC_ES8328_I2C + imply SND_SOC_ES7134 + imply SND_SOC_ES7241 + imply SND_SOC_GTM601 + imply SND_SOC_HDAC_HDMI + imply SND_SOC_HDAC_HDA + imply SND_SOC_ICS43432 + imply SND_SOC_INNO_RK3036 + imply SND_SOC_ISABELLE + imply SND_SOC_JZ4740_CODEC + imply SND_SOC_JZ4725B_CODEC + imply SND_SOC_JZ4770_CODEC + imply SND_SOC_LM4857 + imply SND_SOC_LM49453 + imply SND_SOC_LOCHNAGAR_SC + imply SND_SOC_MAX98088 + imply SND_SOC_MAX98090 + imply SND_SOC_MAX98095 + imply SND_SOC_MAX98357A + imply SND_SOC_MAX98371 + imply SND_SOC_MAX98504 + imply SND_SOC_MAX9867 + imply SND_SOC_MAX98925 + imply SND_SOC_MAX98926 + imply SND_SOC_MAX98927 + imply SND_SOC_MAX98373 + imply SND_SOC_MAX9850 + imply SND_SOC_MAX9860 + imply SND_SOC_MAX9759 + imply SND_SOC_MAX9768 + imply SND_SOC_MAX9877 + imply SND_SOC_MC13783 + imply SND_SOC_ML26124 + imply SND_SOC_MT6351 + imply SND_SOC_MT6358 + imply SND_SOC_MT6660 + imply SND_SOC_NAU8540 + imply SND_SOC_NAU8810 + imply SND_SOC_NAU8822 + imply SND_SOC_NAU8824 + imply SND_SOC_NAU8825 + imply SND_SOC_HDMI_CODEC + imply SND_SOC_PCM1681 + imply SND_SOC_PCM1789_I2C + imply SND_SOC_PCM179X_I2C + imply SND_SOC_PCM179X_SPI + imply SND_SOC_PCM186X_I2C + imply SND_SOC_PCM186X_SPI + imply SND_SOC_PCM3008 + imply SND_SOC_PCM3060_I2C + imply SND_SOC_PCM3060_SPI + imply SND_SOC_PCM3168A_I2C + imply SND_SOC_PCM3168A_SPI + imply SND_SOC_PCM5102A + imply SND_SOC_PCM512x_I2C + imply SND_SOC_PCM512x_SPI + imply SND_SOC_RK3328 + imply SND_SOC_RT274 + imply SND_SOC_RT286 + imply SND_SOC_RT298 + imply SND_SOC_RT1011 + imply SND_SOC_RT1015 + imply SND_SOC_RT1305 + imply SND_SOC_RT1308 + imply SND_SOC_RT5514 + imply SND_SOC_RT5616 + imply SND_SOC_RT5631 + imply SND_SOC_RT5640 + imply SND_SOC_RT5645 + imply SND_SOC_RT5651 + imply SND_SOC_RT5659 + imply SND_SOC_RT5660 + imply SND_SOC_RT5663 + imply SND_SOC_RT5665 + imply SND_SOC_RT5668 + imply SND_SOC_RT5670 + imply SND_SOC_RT5677 + imply SND_SOC_RT5682 + imply SND_SOC_RT700_SDW + imply SND_SOC_RT711_SDW + imply SND_SOC_RT715_SDW + imply SND_SOC_RT1308_SDW + imply SND_SOC_SGTL5000 + imply SND_SOC_SI476X + imply SND_SOC_SIMPLE_AMPLIFIER + imply SND_SOC_SIRF_AUDIO_CODEC + imply SND_SOC_SPDIF + imply SND_SOC_SSM2305 + imply SND_SOC_SSM2518 + imply SND_SOC_SSM2602_SPI + imply SND_SOC_SSM2602_I2C + imply SND_SOC_SSM4567 + imply SND_SOC_STA32X + imply SND_SOC_STA350 + imply SND_SOC_STA529 + imply SND_SOC_STAC9766 + imply SND_SOC_STI_SAS + imply SND_SOC_TAS2552 + imply SND_SOC_TAS2562 + imply SND_SOC_TAS2770 + imply SND_SOC_TAS5086 + imply SND_SOC_TAS571X + imply SND_SOC_TAS5720 + imply SND_SOC_TAS6424 + imply SND_SOC_TDA7419 + imply SND_SOC_TFA9879 + imply SND_SOC_TLV320AIC23_I2C + imply SND_SOC_TLV320AIC23_SPI + imply SND_SOC_TLV320AIC26 + imply SND_SOC_TLV320AIC31XX + imply SND_SOC_TLV320AIC32X4_I2C + imply SND_SOC_TLV320AIC32X4_SPI + imply SND_SOC_TLV320AIC3X + imply SND_SOC_TPA6130A2 + imply SND_SOC_TLV320DAC33 + imply SND_SOC_TSCS42XX + imply SND_SOC_TSCS454 + imply SND_SOC_TS3A227E + imply SND_SOC_TWL4030 + imply SND_SOC_TWL6040 + imply SND_SOC_UDA1334 + imply SND_SOC_UDA134X + imply SND_SOC_UDA1380 + imply SND_SOC_WCD9335 + imply SND_SOC_WCD934X + imply SND_SOC_WL1273 + imply SND_SOC_WM0010 + imply SND_SOC_WM1250_EV1 + imply SND_SOC_WM2000 + imply SND_SOC_WM2200 + imply SND_SOC_WM5100 + imply SND_SOC_WM5102 + imply SND_SOC_WM5110 + imply SND_SOC_WM8350 + imply SND_SOC_WM8400 + imply SND_SOC_WM8510 + imply SND_SOC_WM8523 + imply SND_SOC_WM8524 + imply SND_SOC_WM8580 + imply SND_SOC_WM8711 + imply SND_SOC_WM8727 + imply SND_SOC_WM8728 + imply SND_SOC_WM8731 + imply SND_SOC_WM8737 + imply SND_SOC_WM8741 + imply SND_SOC_WM8750 + imply SND_SOC_WM8753 + imply SND_SOC_WM8770 + imply SND_SOC_WM8776 + imply SND_SOC_WM8782 + imply SND_SOC_WM8804_I2C + imply SND_SOC_WM8804_SPI + imply SND_SOC_WM8900 + imply SND_SOC_WM8903 + imply SND_SOC_WM8904 + imply SND_SOC_WM8940 + imply SND_SOC_WM8955 + imply SND_SOC_WM8960 + imply SND_SOC_WM8961 + imply SND_SOC_WM8962 + imply SND_SOC_WM8971 + imply SND_SOC_WM8974 + imply SND_SOC_WM8978 + imply SND_SOC_WM8983 + imply SND_SOC_WM8985 + imply SND_SOC_WM8988 + imply SND_SOC_WM8990 + imply SND_SOC_WM8991 + imply SND_SOC_WM8993 + imply SND_SOC_WM8994 + imply SND_SOC_WM8995 + imply SND_SOC_WM8996 + imply SND_SOC_WM8997 + imply SND_SOC_WM8998 + imply SND_SOC_WM9081 + imply SND_SOC_WM9090 + imply SND_SOC_WM9705 + imply SND_SOC_WM9712 + imply SND_SOC_WM9713 + imply SND_SOC_WSA881X help Normally ASoC codec drivers are only built if a machine driver which uses them is also built since they are only usable with a machine @@ -283,6 +283,7 @@ config SND_SOC_ALL_CODECS
config SND_SOC_88PM860X tristate + depends on MFD_88PM860X
config SND_SOC_ARIZONA tristate @@ -1301,11 +1302,13 @@ config SND_SOC_TSCS454 Add support for Tempo Semiconductor's TSCS454 audio CODEC.
config SND_SOC_TWL4030 - select MFD_TWL4030_AUDIO tristate + depends on TWL4030_CORE + select MFD_TWL4030_AUDIO
config SND_SOC_TWL6040 tristate + depends on TWL6040_CORE
config SND_SOC_UDA1334 tristate "NXP UDA1334 DAC" @@ -1366,9 +1369,12 @@ config SND_SOC_WM5110
config SND_SOC_WM8350 tristate + depends on MFD_WM8350
config SND_SOC_WM8400 tristate + # FIXME nothing selects SND_SOC_WM8400?? + depends on MFD_WM8400
config SND_SOC_WM8510 tristate "Wolfson Microelectronics WM8510 CODEC"
On Fri, Feb 07, 2020 at 10:13:51AM +0100, Geert Uytterhoeven wrote:
Currently SND_SOC_ALL_CODECS selects the config symbols for all codec drivers. As "select" bypasses dependencies, lots of "select" statements need explicit dependencies, which are hard to get right, and hard to maintain[*].
Fix this by using "imply" instead, which is a weak version of "select", and which obeys dependencies of target symbols.
This is a *really* badly named feature :/
The patch
ASoC: Use imply for SND_SOC_ALL_CODECS
has been applied to the asoc tree at
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-5.7
All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying to this mail.
Thanks, Mark
From ea00d95200d02ece71f5814d41b14f2eb16d598b Mon Sep 17 00:00:00 2001
From: Geert Uytterhoeven geert@linux-m68k.org Date: Fri, 7 Feb 2020 10:13:51 +0100 Subject: [PATCH] ASoC: Use imply for SND_SOC_ALL_CODECS
Currently SND_SOC_ALL_CODECS selects the config symbols for all codec drivers. As "select" bypasses dependencies, lots of "select" statements need explicit dependencies, which are hard to get right, and hard to maintain[*].
Fix this by using "imply" instead, which is a weak version of "select", and which obeys dependencies of target symbols.
Add dependencies to invisible symbols that are currently selected only if their dependencies are fulfilled.
[*] See e.g. commit 13426feaf46c48fc ("ASoC: wcd934x: Add missing COMMON_CLK dependency to SND_SOC_ALL_CODECS").
Signed-off-by: Geert Uytterhoeven geert@linux-m68k.org Link: https://lore.kernel.org/r/20200207091351.18133-1-geert@linux-m68k.org Signed-off-by: Mark Brown broonie@kernel.org --- sound/soc/codecs/Kconfig | 520 ++++++++++++++++++++------------------- 1 file changed, 263 insertions(+), 257 deletions(-)
diff --git a/sound/soc/codecs/Kconfig b/sound/soc/codecs/Kconfig index 7e90f5d83097..7a14b1c416b5 100644 --- a/sound/soc/codecs/Kconfig +++ b/sound/soc/codecs/Kconfig @@ -14,262 +14,262 @@ menu "CODEC drivers" config SND_SOC_ALL_CODECS tristate "Build all ASoC CODEC drivers" depends on COMPILE_TEST - select SND_SOC_88PM860X if MFD_88PM860X - select SND_SOC_L3 - select SND_SOC_AB8500_CODEC if ABX500_CORE - select SND_SOC_AC97_CODEC - select SND_SOC_AD1836 if SPI_MASTER - select SND_SOC_AD193X_SPI if SPI_MASTER - select SND_SOC_AD193X_I2C if I2C - select SND_SOC_AD1980 if SND_SOC_AC97_BUS - select SND_SOC_AD73311 - select SND_SOC_ADAU1373 if I2C - select SND_SOC_ADAU1761_I2C if I2C - select SND_SOC_ADAU1761_SPI if SPI - select SND_SOC_ADAU1781_I2C if I2C - select SND_SOC_ADAU1781_SPI if SPI - select SND_SOC_ADAV801 if SPI_MASTER - select SND_SOC_ADAV803 if I2C - select SND_SOC_ADAU1977_SPI if SPI_MASTER - select SND_SOC_ADAU1977_I2C if I2C - select SND_SOC_ADAU1701 if I2C - select SND_SOC_ADAU7002 - select SND_SOC_ADAU7118_I2C if I2C - select SND_SOC_ADAU7118_HW - select SND_SOC_ADS117X - select SND_SOC_AK4104 if SPI_MASTER - select SND_SOC_AK4118 if I2C - select SND_SOC_AK4458 if I2C - select SND_SOC_AK4535 if I2C - select SND_SOC_AK4554 - select SND_SOC_AK4613 if I2C - select SND_SOC_AK4641 if I2C - select SND_SOC_AK4642 if I2C - select SND_SOC_AK4671 if I2C - select SND_SOC_AK5386 - select SND_SOC_AK5558 if I2C - select SND_SOC_ALC5623 if I2C - select SND_SOC_ALC5632 if I2C - select SND_SOC_BT_SCO - select SND_SOC_BD28623 - select SND_SOC_CQ0093VC - select SND_SOC_CROS_EC_CODEC if CROS_EC - select SND_SOC_CS35L32 if I2C - select SND_SOC_CS35L33 if I2C - select SND_SOC_CS35L34 if I2C - select SND_SOC_CS35L35 if I2C - select SND_SOC_CS35L36 if I2C - select SND_SOC_CS42L42 if I2C - select SND_SOC_CS42L51_I2C if I2C - select SND_SOC_CS42L52 if I2C && INPUT - select SND_SOC_CS42L56 if I2C && INPUT - select SND_SOC_CS42L73 if I2C - select SND_SOC_CS4265 if I2C - select SND_SOC_CS4270 if I2C - select SND_SOC_CS4271_I2C if I2C - select SND_SOC_CS4271_SPI if SPI_MASTER - select SND_SOC_CS42XX8_I2C if I2C - select SND_SOC_CS43130 if I2C - select SND_SOC_CS4341 if SND_SOC_I2C_AND_SPI - select SND_SOC_CS4349 if I2C - select SND_SOC_CS47L15 if MFD_CS47L15 - select SND_SOC_CS47L24 if MFD_CS47L24 - select SND_SOC_CS47L35 if MFD_CS47L35 - select SND_SOC_CS47L85 if MFD_CS47L85 - select SND_SOC_CS47L90 if MFD_CS47L90 - select SND_SOC_CS47L92 if MFD_CS47L92 - select SND_SOC_CS53L30 if I2C - select SND_SOC_CX20442 if TTY - select SND_SOC_CX2072X if I2C - select SND_SOC_DA7210 if SND_SOC_I2C_AND_SPI - select SND_SOC_DA7213 if I2C - select SND_SOC_DA7218 if I2C - select SND_SOC_DA7219 if I2C - select SND_SOC_DA732X if I2C - select SND_SOC_DA9055 if I2C - select SND_SOC_DMIC if GPIOLIB - select SND_SOC_ES8316 if I2C - select SND_SOC_ES8328_SPI if SPI_MASTER - select SND_SOC_ES8328_I2C if I2C - select SND_SOC_ES7134 - select SND_SOC_ES7241 - select SND_SOC_GTM601 - select SND_SOC_HDAC_HDMI - select SND_SOC_HDAC_HDA - select SND_SOC_ICS43432 - select SND_SOC_INNO_RK3036 - select SND_SOC_ISABELLE if I2C - select SND_SOC_JZ4740_CODEC - select SND_SOC_JZ4725B_CODEC - select SND_SOC_JZ4770_CODEC - select SND_SOC_LM4857 if I2C - select SND_SOC_LM49453 if I2C - select SND_SOC_LOCHNAGAR_SC if MFD_LOCHNAGAR - select SND_SOC_MAX98088 if I2C - select SND_SOC_MAX98090 if I2C - select SND_SOC_MAX98095 if I2C - select SND_SOC_MAX98357A if GPIOLIB - select SND_SOC_MAX98371 if I2C - select SND_SOC_MAX98504 if I2C - select SND_SOC_MAX9867 if I2C - select SND_SOC_MAX98925 if I2C - select SND_SOC_MAX98926 if I2C - select SND_SOC_MAX98927 if I2C - select SND_SOC_MAX98373 if I2C - select SND_SOC_MAX9850 if I2C - select SND_SOC_MAX9860 if I2C - select SND_SOC_MAX9759 - select SND_SOC_MAX9768 if I2C - select SND_SOC_MAX9877 if I2C - select SND_SOC_MC13783 if MFD_MC13XXX - select SND_SOC_ML26124 if I2C - select SND_SOC_MT6351 if MTK_PMIC_WRAP - select SND_SOC_MT6358 if MTK_PMIC_WRAP - select SND_SOC_MT6660 if I2C - select SND_SOC_NAU8540 if I2C - select SND_SOC_NAU8810 if I2C - select SND_SOC_NAU8822 if I2C - select SND_SOC_NAU8824 if I2C - select SND_SOC_NAU8825 if I2C - select SND_SOC_HDMI_CODEC - select SND_SOC_PCM1681 if I2C - select SND_SOC_PCM1789_I2C if I2C - select SND_SOC_PCM179X_I2C if I2C - select SND_SOC_PCM179X_SPI if SPI_MASTER - select SND_SOC_PCM186X_I2C if I2C - select SND_SOC_PCM186X_SPI if SPI_MASTER - select SND_SOC_PCM3008 - select SND_SOC_PCM3060_I2C if I2C - select SND_SOC_PCM3060_SPI if SPI_MASTER - select SND_SOC_PCM3168A_I2C if I2C - select SND_SOC_PCM3168A_SPI if SPI_MASTER - select SND_SOC_PCM5102A - select SND_SOC_PCM512x_I2C if I2C - select SND_SOC_PCM512x_SPI if SPI_MASTER - select SND_SOC_RK3328 - select SND_SOC_RT274 if I2C - select SND_SOC_RT286 if I2C - select SND_SOC_RT298 if I2C - select SND_SOC_RT1011 if I2C - select SND_SOC_RT1015 if I2C - select SND_SOC_RT1305 if I2C - select SND_SOC_RT1308 if I2C - select SND_SOC_RT5514 if I2C - select SND_SOC_RT5616 if I2C - select SND_SOC_RT5631 if I2C - select SND_SOC_RT5640 if I2C - select SND_SOC_RT5645 if I2C - select SND_SOC_RT5651 if I2C - select SND_SOC_RT5659 if I2C - select SND_SOC_RT5660 if I2C - select SND_SOC_RT5663 if I2C - select SND_SOC_RT5665 if I2C - select SND_SOC_RT5668 if I2C - select SND_SOC_RT5670 if I2C - select SND_SOC_RT5677 if I2C && SPI_MASTER - select SND_SOC_RT5682 if I2C - select SND_SOC_RT700_SDW if SOUNDWIRE - select SND_SOC_RT711_SDW if SOUNDWIRE - select SND_SOC_RT715_SDW if SOUNDWIRE - select SND_SOC_RT1308_SDW if SOUNDWIRE - select SND_SOC_SGTL5000 if I2C - select SND_SOC_SI476X if MFD_SI476X_CORE - select SND_SOC_SIMPLE_AMPLIFIER - select SND_SOC_SIRF_AUDIO_CODEC - select SND_SOC_SPDIF - select SND_SOC_SSM2305 - select SND_SOC_SSM2518 if I2C - select SND_SOC_SSM2602_SPI if SPI_MASTER - select SND_SOC_SSM2602_I2C if I2C - select SND_SOC_SSM4567 if I2C - select SND_SOC_STA32X if I2C - select SND_SOC_STA350 if I2C - select SND_SOC_STA529 if I2C - select SND_SOC_STAC9766 if SND_SOC_AC97_BUS - select SND_SOC_STI_SAS - select SND_SOC_TAS2552 if I2C - select SND_SOC_TAS2562 if I2C - select SND_SOC_TAS2770 if I2C - select SND_SOC_TAS5086 if I2C - select SND_SOC_TAS571X if I2C - select SND_SOC_TAS5720 if I2C - select SND_SOC_TAS6424 if I2C - select SND_SOC_TDA7419 if I2C - select SND_SOC_TFA9879 if I2C - select SND_SOC_TLV320AIC23_I2C if I2C - select SND_SOC_TLV320AIC23_SPI if SPI_MASTER - select SND_SOC_TLV320AIC26 if SPI_MASTER - select SND_SOC_TLV320AIC31XX if I2C - select SND_SOC_TLV320AIC32X4_I2C if I2C && COMMON_CLK - select SND_SOC_TLV320AIC32X4_SPI if SPI_MASTER && COMMON_CLK - select SND_SOC_TLV320AIC3X if I2C - select SND_SOC_TPA6130A2 if I2C - select SND_SOC_TLV320DAC33 if I2C - select SND_SOC_TSCS42XX if I2C - select SND_SOC_TSCS454 if I2C - select SND_SOC_TS3A227E if I2C - select SND_SOC_TWL4030 if TWL4030_CORE - select SND_SOC_TWL6040 if TWL6040_CORE - select SND_SOC_UDA1334 if GPIOLIB - select SND_SOC_UDA134X - select SND_SOC_UDA1380 if I2C - select SND_SOC_WCD9335 if SLIMBUS - select SND_SOC_WCD934X if MFD_WCD934X && COMMON_CLK - select SND_SOC_WL1273 if MFD_WL1273_CORE - select SND_SOC_WM0010 if SPI_MASTER - select SND_SOC_WM1250_EV1 if I2C - select SND_SOC_WM2000 if I2C - select SND_SOC_WM2200 if I2C - select SND_SOC_WM5100 if I2C - select SND_SOC_WM5102 if MFD_WM5102 - select SND_SOC_WM5110 if MFD_WM5110 - select SND_SOC_WM8350 if MFD_WM8350 - select SND_SOC_WM8400 if MFD_WM8400 - select SND_SOC_WM8510 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8523 if I2C - select SND_SOC_WM8524 if GPIOLIB - select SND_SOC_WM8580 if I2C - select SND_SOC_WM8711 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8727 - select SND_SOC_WM8728 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8731 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8737 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8741 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8750 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8753 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8770 if SPI_MASTER - select SND_SOC_WM8776 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8782 - select SND_SOC_WM8804_I2C if I2C - select SND_SOC_WM8804_SPI if SPI_MASTER - select SND_SOC_WM8900 if I2C - select SND_SOC_WM8903 if I2C - select SND_SOC_WM8904 if I2C - select SND_SOC_WM8940 if I2C - select SND_SOC_WM8955 if I2C - select SND_SOC_WM8960 if I2C - select SND_SOC_WM8961 if I2C - select SND_SOC_WM8962 if I2C && INPUT - select SND_SOC_WM8971 if I2C - select SND_SOC_WM8974 if I2C - select SND_SOC_WM8978 if I2C - select SND_SOC_WM8983 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8985 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8988 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8990 if I2C - select SND_SOC_WM8991 if I2C - select SND_SOC_WM8993 if I2C - select SND_SOC_WM8994 if MFD_WM8994 - select SND_SOC_WM8995 if SND_SOC_I2C_AND_SPI - select SND_SOC_WM8996 if I2C - select SND_SOC_WM8997 if MFD_WM8997 - select SND_SOC_WM8998 if MFD_WM8998 - select SND_SOC_WM9081 if I2C - select SND_SOC_WM9090 if I2C - select SND_SOC_WM9705 if (SND_SOC_AC97_BUS || SND_SOC_AC97_BUS_NEW) - select SND_SOC_WM9712 if (SND_SOC_AC97_BUS || SND_SOC_AC97_BUS_NEW) - select SND_SOC_WM9713 if (SND_SOC_AC97_BUS || SND_SOC_AC97_BUS_NEW) - select SND_SOC_WSA881X if SOUNDWIRE + imply SND_SOC_88PM860X + imply SND_SOC_L3 + imply SND_SOC_AB8500_CODEC + imply SND_SOC_AC97_CODEC + imply SND_SOC_AD1836 + imply SND_SOC_AD193X_SPI + imply SND_SOC_AD193X_I2C + imply SND_SOC_AD1980 + imply SND_SOC_AD73311 + imply SND_SOC_ADAU1373 + imply SND_SOC_ADAU1761_I2C + imply SND_SOC_ADAU1761_SPI + imply SND_SOC_ADAU1781_I2C + imply SND_SOC_ADAU1781_SPI + imply SND_SOC_ADAV801 + imply SND_SOC_ADAV803 + imply SND_SOC_ADAU1977_SPI + imply SND_SOC_ADAU1977_I2C + imply SND_SOC_ADAU1701 + imply SND_SOC_ADAU7002 + imply SND_SOC_ADAU7118_I2C + imply SND_SOC_ADAU7118_HW + imply SND_SOC_ADS117X + imply SND_SOC_AK4104 + imply SND_SOC_AK4118 + imply SND_SOC_AK4458 + imply SND_SOC_AK4535 + imply SND_SOC_AK4554 + imply SND_SOC_AK4613 + imply SND_SOC_AK4641 + imply SND_SOC_AK4642 + imply SND_SOC_AK4671 + imply SND_SOC_AK5386 + imply SND_SOC_AK5558 + imply SND_SOC_ALC5623 + imply SND_SOC_ALC5632 + imply SND_SOC_BT_SCO + imply SND_SOC_BD28623 + imply SND_SOC_CQ0093VC + imply SND_SOC_CROS_EC_CODEC + imply SND_SOC_CS35L32 + imply SND_SOC_CS35L33 + imply SND_SOC_CS35L34 + imply SND_SOC_CS35L35 + imply SND_SOC_CS35L36 + imply SND_SOC_CS42L42 + imply SND_SOC_CS42L51_I2C + imply SND_SOC_CS42L52 + imply SND_SOC_CS42L56 + imply SND_SOC_CS42L73 + imply SND_SOC_CS4265 + imply SND_SOC_CS4270 + imply SND_SOC_CS4271_I2C + imply SND_SOC_CS4271_SPI + imply SND_SOC_CS42XX8_I2C + imply SND_SOC_CS43130 + imply SND_SOC_CS4341 + imply SND_SOC_CS4349 + imply SND_SOC_CS47L15 + imply SND_SOC_CS47L24 + imply SND_SOC_CS47L35 + imply SND_SOC_CS47L85 + imply SND_SOC_CS47L90 + imply SND_SOC_CS47L92 + imply SND_SOC_CS53L30 + imply SND_SOC_CX20442 + imply SND_SOC_CX2072X + imply SND_SOC_DA7210 + imply SND_SOC_DA7213 + imply SND_SOC_DA7218 + imply SND_SOC_DA7219 + imply SND_SOC_DA732X + imply SND_SOC_DA9055 + imply SND_SOC_DMIC + imply SND_SOC_ES8316 + imply SND_SOC_ES8328_SPI + imply SND_SOC_ES8328_I2C + imply SND_SOC_ES7134 + imply SND_SOC_ES7241 + imply SND_SOC_GTM601 + imply SND_SOC_HDAC_HDMI + imply SND_SOC_HDAC_HDA + imply SND_SOC_ICS43432 + imply SND_SOC_INNO_RK3036 + imply SND_SOC_ISABELLE + imply SND_SOC_JZ4740_CODEC + imply SND_SOC_JZ4725B_CODEC + imply SND_SOC_JZ4770_CODEC + imply SND_SOC_LM4857 + imply SND_SOC_LM49453 + imply SND_SOC_LOCHNAGAR_SC + imply SND_SOC_MAX98088 + imply SND_SOC_MAX98090 + imply SND_SOC_MAX98095 + imply SND_SOC_MAX98357A + imply SND_SOC_MAX98371 + imply SND_SOC_MAX98504 + imply SND_SOC_MAX9867 + imply SND_SOC_MAX98925 + imply SND_SOC_MAX98926 + imply SND_SOC_MAX98927 + imply SND_SOC_MAX98373 + imply SND_SOC_MAX9850 + imply SND_SOC_MAX9860 + imply SND_SOC_MAX9759 + imply SND_SOC_MAX9768 + imply SND_SOC_MAX9877 + imply SND_SOC_MC13783 + imply SND_SOC_ML26124 + imply SND_SOC_MT6351 + imply SND_SOC_MT6358 + imply SND_SOC_MT6660 + imply SND_SOC_NAU8540 + imply SND_SOC_NAU8810 + imply SND_SOC_NAU8822 + imply SND_SOC_NAU8824 + imply SND_SOC_NAU8825 + imply SND_SOC_HDMI_CODEC + imply SND_SOC_PCM1681 + imply SND_SOC_PCM1789_I2C + imply SND_SOC_PCM179X_I2C + imply SND_SOC_PCM179X_SPI + imply SND_SOC_PCM186X_I2C + imply SND_SOC_PCM186X_SPI + imply SND_SOC_PCM3008 + imply SND_SOC_PCM3060_I2C + imply SND_SOC_PCM3060_SPI + imply SND_SOC_PCM3168A_I2C + imply SND_SOC_PCM3168A_SPI + imply SND_SOC_PCM5102A + imply SND_SOC_PCM512x_I2C + imply SND_SOC_PCM512x_SPI + imply SND_SOC_RK3328 + imply SND_SOC_RT274 + imply SND_SOC_RT286 + imply SND_SOC_RT298 + imply SND_SOC_RT1011 + imply SND_SOC_RT1015 + imply SND_SOC_RT1305 + imply SND_SOC_RT1308 + imply SND_SOC_RT5514 + imply SND_SOC_RT5616 + imply SND_SOC_RT5631 + imply SND_SOC_RT5640 + imply SND_SOC_RT5645 + imply SND_SOC_RT5651 + imply SND_SOC_RT5659 + imply SND_SOC_RT5660 + imply SND_SOC_RT5663 + imply SND_SOC_RT5665 + imply SND_SOC_RT5668 + imply SND_SOC_RT5670 + imply SND_SOC_RT5677 + imply SND_SOC_RT5682 + imply SND_SOC_RT700_SDW + imply SND_SOC_RT711_SDW + imply SND_SOC_RT715_SDW + imply SND_SOC_RT1308_SDW + imply SND_SOC_SGTL5000 + imply SND_SOC_SI476X + imply SND_SOC_SIMPLE_AMPLIFIER + imply SND_SOC_SIRF_AUDIO_CODEC + imply SND_SOC_SPDIF + imply SND_SOC_SSM2305 + imply SND_SOC_SSM2518 + imply SND_SOC_SSM2602_SPI + imply SND_SOC_SSM2602_I2C + imply SND_SOC_SSM4567 + imply SND_SOC_STA32X + imply SND_SOC_STA350 + imply SND_SOC_STA529 + imply SND_SOC_STAC9766 + imply SND_SOC_STI_SAS + imply SND_SOC_TAS2552 + imply SND_SOC_TAS2562 + imply SND_SOC_TAS2770 + imply SND_SOC_TAS5086 + imply SND_SOC_TAS571X + imply SND_SOC_TAS5720 + imply SND_SOC_TAS6424 + imply SND_SOC_TDA7419 + imply SND_SOC_TFA9879 + imply SND_SOC_TLV320AIC23_I2C + imply SND_SOC_TLV320AIC23_SPI + imply SND_SOC_TLV320AIC26 + imply SND_SOC_TLV320AIC31XX + imply SND_SOC_TLV320AIC32X4_I2C + imply SND_SOC_TLV320AIC32X4_SPI + imply SND_SOC_TLV320AIC3X + imply SND_SOC_TPA6130A2 + imply SND_SOC_TLV320DAC33 + imply SND_SOC_TSCS42XX + imply SND_SOC_TSCS454 + imply SND_SOC_TS3A227E + imply SND_SOC_TWL4030 + imply SND_SOC_TWL6040 + imply SND_SOC_UDA1334 + imply SND_SOC_UDA134X + imply SND_SOC_UDA1380 + imply SND_SOC_WCD9335 + imply SND_SOC_WCD934X + imply SND_SOC_WL1273 + imply SND_SOC_WM0010 + imply SND_SOC_WM1250_EV1 + imply SND_SOC_WM2000 + imply SND_SOC_WM2200 + imply SND_SOC_WM5100 + imply SND_SOC_WM5102 + imply SND_SOC_WM5110 + imply SND_SOC_WM8350 + imply SND_SOC_WM8400 + imply SND_SOC_WM8510 + imply SND_SOC_WM8523 + imply SND_SOC_WM8524 + imply SND_SOC_WM8580 + imply SND_SOC_WM8711 + imply SND_SOC_WM8727 + imply SND_SOC_WM8728 + imply SND_SOC_WM8731 + imply SND_SOC_WM8737 + imply SND_SOC_WM8741 + imply SND_SOC_WM8750 + imply SND_SOC_WM8753 + imply SND_SOC_WM8770 + imply SND_SOC_WM8776 + imply SND_SOC_WM8782 + imply SND_SOC_WM8804_I2C + imply SND_SOC_WM8804_SPI + imply SND_SOC_WM8900 + imply SND_SOC_WM8903 + imply SND_SOC_WM8904 + imply SND_SOC_WM8940 + imply SND_SOC_WM8955 + imply SND_SOC_WM8960 + imply SND_SOC_WM8961 + imply SND_SOC_WM8962 + imply SND_SOC_WM8971 + imply SND_SOC_WM8974 + imply SND_SOC_WM8978 + imply SND_SOC_WM8983 + imply SND_SOC_WM8985 + imply SND_SOC_WM8988 + imply SND_SOC_WM8990 + imply SND_SOC_WM8991 + imply SND_SOC_WM8993 + imply SND_SOC_WM8994 + imply SND_SOC_WM8995 + imply SND_SOC_WM8996 + imply SND_SOC_WM8997 + imply SND_SOC_WM8998 + imply SND_SOC_WM9081 + imply SND_SOC_WM9090 + imply SND_SOC_WM9705 + imply SND_SOC_WM9712 + imply SND_SOC_WM9713 + imply SND_SOC_WSA881X help Normally ASoC codec drivers are only built if a machine driver which uses them is also built since they are only usable with a machine @@ -283,6 +283,7 @@ config SND_SOC_ALL_CODECS
config SND_SOC_88PM860X tristate + depends on MFD_88PM860X
config SND_SOC_ARIZONA tristate @@ -1301,11 +1302,13 @@ config SND_SOC_TSCS454 Add support for Tempo Semiconductor's TSCS454 audio CODEC.
config SND_SOC_TWL4030 - select MFD_TWL4030_AUDIO tristate + depends on TWL4030_CORE + select MFD_TWL4030_AUDIO
config SND_SOC_TWL6040 tristate + depends on TWL6040_CORE
config SND_SOC_UDA1334 tristate "NXP UDA1334 DAC" @@ -1366,9 +1369,12 @@ config SND_SOC_WM5110
config SND_SOC_WM8350 tristate + depends on MFD_WM8350
config SND_SOC_WM8400 tristate + # FIXME nothing selects SND_SOC_WM8400?? + depends on MFD_WM8400
config SND_SOC_WM8510 tristate "Wolfson Microelectronics WM8510 CODEC"
On Fri, Feb 7, 2020 at 6:13 PM Geert Uytterhoeven geert@linux-m68k.org wrote:
Currently SND_SOC_ALL_CODECS selects the config symbols for all codec drivers. As "select" bypasses dependencies, lots of "select" statements need explicit dependencies, which are hard to get right, and hard to maintain[*].
Fix this by using "imply" instead, which is a weak version of "select", and which obeys dependencies of target symbols.
Add dependencies to invisible symbols that are currently selected only if their dependencies are fulfilled.
[*] See e.g. commit 13426feaf46c48fc ("ASoC: wcd934x: Add missing COMMON_CLK dependency to SND_SOC_ALL_CODECS").
Why does SND_SOC_ALL_CODECS exist in the first place? Is this compile-test or run-test?
If it is the former, allyesconfig / allmodconfig covers all of them.
Masahiro Yamada
Signed-off-by: Geert Uytterhoeven geert@linux-m68k.org
Does this sound like a good solution? Anything I'm missing w.r.t. the "imply" semantics? FIXME Nothing but SND_SOC_ALL_CODECS selects SND_SOC_WM8400!
Tested with m68k/allmodconfig and m68k/allyesconfig only. Hence some other invisible symbols may still have missing dependencies.
sound/soc/codecs/Kconfig | 520 ++++++++++++++++++++------------------- 1 file changed, 263 insertions(+), 257 deletions(-)
diff --git a/sound/soc/codecs/Kconfig b/sound/soc/codecs/Kconfig index 7e90f5d830971309..7a14b1c416b55e46 100644 --- a/sound/soc/codecs/Kconfig +++ b/sound/soc/codecs/Kconfig @@ -14,262 +14,262 @@ menu "CODEC drivers" config SND_SOC_ALL_CODECS tristate "Build all ASoC CODEC drivers" depends on COMPILE_TEST
select SND_SOC_88PM860X if MFD_88PM860Xselect SND_SOC_L3select SND_SOC_AB8500_CODEC if ABX500_COREselect SND_SOC_AC97_CODECselect SND_SOC_AD1836 if SPI_MASTERselect SND_SOC_AD193X_SPI if SPI_MASTERselect SND_SOC_AD193X_I2C if I2Cselect SND_SOC_AD1980 if SND_SOC_AC97_BUSselect SND_SOC_AD73311select SND_SOC_ADAU1373 if I2Cselect SND_SOC_ADAU1761_I2C if I2Cselect SND_SOC_ADAU1761_SPI if SPIselect SND_SOC_ADAU1781_I2C if I2Cselect SND_SOC_ADAU1781_SPI if SPIselect SND_SOC_ADAV801 if SPI_MASTERselect SND_SOC_ADAV803 if I2Cselect SND_SOC_ADAU1977_SPI if SPI_MASTERselect SND_SOC_ADAU1977_I2C if I2Cselect SND_SOC_ADAU1701 if I2Cselect SND_SOC_ADAU7002select SND_SOC_ADAU7118_I2C if I2Cselect SND_SOC_ADAU7118_HWselect SND_SOC_ADS117Xselect SND_SOC_AK4104 if SPI_MASTERselect SND_SOC_AK4118 if I2Cselect SND_SOC_AK4458 if I2Cselect SND_SOC_AK4535 if I2Cselect SND_SOC_AK4554select SND_SOC_AK4613 if I2Cselect SND_SOC_AK4641 if I2Cselect SND_SOC_AK4642 if I2Cselect SND_SOC_AK4671 if I2Cselect SND_SOC_AK5386select SND_SOC_AK5558 if I2Cselect SND_SOC_ALC5623 if I2Cselect SND_SOC_ALC5632 if I2Cselect SND_SOC_BT_SCOselect SND_SOC_BD28623select SND_SOC_CQ0093VCselect SND_SOC_CROS_EC_CODEC if CROS_ECselect SND_SOC_CS35L32 if I2Cselect SND_SOC_CS35L33 if I2Cselect SND_SOC_CS35L34 if I2Cselect SND_SOC_CS35L35 if I2Cselect SND_SOC_CS35L36 if I2Cselect SND_SOC_CS42L42 if I2Cselect SND_SOC_CS42L51_I2C if I2Cselect SND_SOC_CS42L52 if I2C && INPUTselect SND_SOC_CS42L56 if I2C && INPUTselect SND_SOC_CS42L73 if I2Cselect SND_SOC_CS4265 if I2Cselect SND_SOC_CS4270 if I2Cselect SND_SOC_CS4271_I2C if I2Cselect SND_SOC_CS4271_SPI if SPI_MASTERselect SND_SOC_CS42XX8_I2C if I2Cselect SND_SOC_CS43130 if I2Cselect SND_SOC_CS4341 if SND_SOC_I2C_AND_SPIselect SND_SOC_CS4349 if I2Cselect SND_SOC_CS47L15 if MFD_CS47L15select SND_SOC_CS47L24 if MFD_CS47L24select SND_SOC_CS47L35 if MFD_CS47L35select SND_SOC_CS47L85 if MFD_CS47L85select SND_SOC_CS47L90 if MFD_CS47L90select SND_SOC_CS47L92 if MFD_CS47L92select SND_SOC_CS53L30 if I2Cselect SND_SOC_CX20442 if TTYselect SND_SOC_CX2072X if I2Cselect SND_SOC_DA7210 if SND_SOC_I2C_AND_SPIselect SND_SOC_DA7213 if I2Cselect SND_SOC_DA7218 if I2Cselect SND_SOC_DA7219 if I2Cselect SND_SOC_DA732X if I2Cselect SND_SOC_DA9055 if I2Cselect SND_SOC_DMIC if GPIOLIBselect SND_SOC_ES8316 if I2Cselect SND_SOC_ES8328_SPI if SPI_MASTERselect SND_SOC_ES8328_I2C if I2Cselect SND_SOC_ES7134select SND_SOC_ES7241select SND_SOC_GTM601select SND_SOC_HDAC_HDMIselect SND_SOC_HDAC_HDAselect SND_SOC_ICS43432select SND_SOC_INNO_RK3036select SND_SOC_ISABELLE if I2Cselect SND_SOC_JZ4740_CODECselect SND_SOC_JZ4725B_CODECselect SND_SOC_JZ4770_CODECselect SND_SOC_LM4857 if I2Cselect SND_SOC_LM49453 if I2Cselect SND_SOC_LOCHNAGAR_SC if MFD_LOCHNAGARselect SND_SOC_MAX98088 if I2Cselect SND_SOC_MAX98090 if I2Cselect SND_SOC_MAX98095 if I2Cselect SND_SOC_MAX98357A if GPIOLIBselect SND_SOC_MAX98371 if I2Cselect SND_SOC_MAX98504 if I2Cselect SND_SOC_MAX9867 if I2Cselect SND_SOC_MAX98925 if I2Cselect SND_SOC_MAX98926 if I2Cselect SND_SOC_MAX98927 if I2Cselect SND_SOC_MAX98373 if I2Cselect SND_SOC_MAX9850 if I2Cselect SND_SOC_MAX9860 if I2Cselect SND_SOC_MAX9759select SND_SOC_MAX9768 if I2Cselect SND_SOC_MAX9877 if I2Cselect SND_SOC_MC13783 if MFD_MC13XXXselect SND_SOC_ML26124 if I2Cselect SND_SOC_MT6351 if MTK_PMIC_WRAPselect SND_SOC_MT6358 if MTK_PMIC_WRAPselect SND_SOC_MT6660 if I2Cselect SND_SOC_NAU8540 if I2Cselect SND_SOC_NAU8810 if I2Cselect SND_SOC_NAU8822 if I2Cselect SND_SOC_NAU8824 if I2Cselect SND_SOC_NAU8825 if I2Cselect SND_SOC_HDMI_CODECselect SND_SOC_PCM1681 if I2Cselect SND_SOC_PCM1789_I2C if I2Cselect SND_SOC_PCM179X_I2C if I2Cselect SND_SOC_PCM179X_SPI if SPI_MASTERselect SND_SOC_PCM186X_I2C if I2Cselect SND_SOC_PCM186X_SPI if SPI_MASTERselect SND_SOC_PCM3008select SND_SOC_PCM3060_I2C if I2Cselect SND_SOC_PCM3060_SPI if SPI_MASTERselect SND_SOC_PCM3168A_I2C if I2Cselect SND_SOC_PCM3168A_SPI if SPI_MASTERselect SND_SOC_PCM5102Aselect SND_SOC_PCM512x_I2C if I2Cselect SND_SOC_PCM512x_SPI if SPI_MASTERselect SND_SOC_RK3328select SND_SOC_RT274 if I2Cselect SND_SOC_RT286 if I2Cselect SND_SOC_RT298 if I2Cselect SND_SOC_RT1011 if I2Cselect SND_SOC_RT1015 if I2Cselect SND_SOC_RT1305 if I2Cselect SND_SOC_RT1308 if I2Cselect SND_SOC_RT5514 if I2Cselect SND_SOC_RT5616 if I2Cselect SND_SOC_RT5631 if I2Cselect SND_SOC_RT5640 if I2Cselect SND_SOC_RT5645 if I2Cselect SND_SOC_RT5651 if I2Cselect SND_SOC_RT5659 if I2Cselect SND_SOC_RT5660 if I2Cselect SND_SOC_RT5663 if I2Cselect SND_SOC_RT5665 if I2Cselect SND_SOC_RT5668 if I2Cselect SND_SOC_RT5670 if I2Cselect SND_SOC_RT5677 if I2C && SPI_MASTERselect SND_SOC_RT5682 if I2Cselect SND_SOC_RT700_SDW if SOUNDWIREselect SND_SOC_RT711_SDW if SOUNDWIREselect SND_SOC_RT715_SDW if SOUNDWIREselect SND_SOC_RT1308_SDW if SOUNDWIREselect SND_SOC_SGTL5000 if I2Cselect SND_SOC_SI476X if MFD_SI476X_COREselect SND_SOC_SIMPLE_AMPLIFIERselect SND_SOC_SIRF_AUDIO_CODECselect SND_SOC_SPDIFselect SND_SOC_SSM2305select SND_SOC_SSM2518 if I2Cselect SND_SOC_SSM2602_SPI if SPI_MASTERselect SND_SOC_SSM2602_I2C if I2Cselect SND_SOC_SSM4567 if I2Cselect SND_SOC_STA32X if I2Cselect SND_SOC_STA350 if I2Cselect SND_SOC_STA529 if I2Cselect SND_SOC_STAC9766 if SND_SOC_AC97_BUSselect SND_SOC_STI_SASselect SND_SOC_TAS2552 if I2Cselect SND_SOC_TAS2562 if I2Cselect SND_SOC_TAS2770 if I2Cselect SND_SOC_TAS5086 if I2Cselect SND_SOC_TAS571X if I2Cselect SND_SOC_TAS5720 if I2Cselect SND_SOC_TAS6424 if I2Cselect SND_SOC_TDA7419 if I2Cselect SND_SOC_TFA9879 if I2Cselect SND_SOC_TLV320AIC23_I2C if I2Cselect SND_SOC_TLV320AIC23_SPI if SPI_MASTERselect SND_SOC_TLV320AIC26 if SPI_MASTERselect SND_SOC_TLV320AIC31XX if I2Cselect SND_SOC_TLV320AIC32X4_I2C if I2C && COMMON_CLKselect SND_SOC_TLV320AIC32X4_SPI if SPI_MASTER && COMMON_CLKselect SND_SOC_TLV320AIC3X if I2Cselect SND_SOC_TPA6130A2 if I2Cselect SND_SOC_TLV320DAC33 if I2Cselect SND_SOC_TSCS42XX if I2Cselect SND_SOC_TSCS454 if I2Cselect SND_SOC_TS3A227E if I2Cselect SND_SOC_TWL4030 if TWL4030_COREselect SND_SOC_TWL6040 if TWL6040_COREselect SND_SOC_UDA1334 if GPIOLIBselect SND_SOC_UDA134Xselect SND_SOC_UDA1380 if I2Cselect SND_SOC_WCD9335 if SLIMBUSselect SND_SOC_WCD934X if MFD_WCD934X && COMMON_CLKselect SND_SOC_WL1273 if MFD_WL1273_COREselect SND_SOC_WM0010 if SPI_MASTERselect SND_SOC_WM1250_EV1 if I2Cselect SND_SOC_WM2000 if I2Cselect SND_SOC_WM2200 if I2Cselect SND_SOC_WM5100 if I2Cselect SND_SOC_WM5102 if MFD_WM5102select SND_SOC_WM5110 if MFD_WM5110select SND_SOC_WM8350 if MFD_WM8350select SND_SOC_WM8400 if MFD_WM8400select SND_SOC_WM8510 if SND_SOC_I2C_AND_SPIselect SND_SOC_WM8523 if I2Cselect SND_SOC_WM8524 if GPIOLIBselect SND_SOC_WM8580 if I2Cselect SND_SOC_WM8711 if SND_SOC_I2C_AND_SPIselect SND_SOC_WM8727select SND_SOC_WM8728 if SND_SOC_I2C_AND_SPIselect SND_SOC_WM8731 if SND_SOC_I2C_AND_SPIselect SND_SOC_WM8737 if SND_SOC_I2C_AND_SPIselect SND_SOC_WM8741 if SND_SOC_I2C_AND_SPIselect SND_SOC_WM8750 if SND_SOC_I2C_AND_SPIselect SND_SOC_WM8753 if SND_SOC_I2C_AND_SPIselect SND_SOC_WM8770 if SPI_MASTERselect SND_SOC_WM8776 if SND_SOC_I2C_AND_SPIselect SND_SOC_WM8782select SND_SOC_WM8804_I2C if I2Cselect SND_SOC_WM8804_SPI if SPI_MASTERselect SND_SOC_WM8900 if I2Cselect SND_SOC_WM8903 if I2Cselect SND_SOC_WM8904 if I2Cselect SND_SOC_WM8940 if I2Cselect SND_SOC_WM8955 if I2Cselect SND_SOC_WM8960 if I2Cselect SND_SOC_WM8961 if I2Cselect SND_SOC_WM8962 if I2C && INPUTselect SND_SOC_WM8971 if I2Cselect SND_SOC_WM8974 if I2Cselect SND_SOC_WM8978 if I2Cselect SND_SOC_WM8983 if SND_SOC_I2C_AND_SPIselect SND_SOC_WM8985 if SND_SOC_I2C_AND_SPIselect SND_SOC_WM8988 if SND_SOC_I2C_AND_SPIselect SND_SOC_WM8990 if I2Cselect SND_SOC_WM8991 if I2Cselect SND_SOC_WM8993 if I2Cselect SND_SOC_WM8994 if MFD_WM8994select SND_SOC_WM8995 if SND_SOC_I2C_AND_SPIselect SND_SOC_WM8996 if I2Cselect SND_SOC_WM8997 if MFD_WM8997select SND_SOC_WM8998 if MFD_WM8998select SND_SOC_WM9081 if I2Cselect SND_SOC_WM9090 if I2Cselect SND_SOC_WM9705 if (SND_SOC_AC97_BUS || SND_SOC_AC97_BUS_NEW)select SND_SOC_WM9712 if (SND_SOC_AC97_BUS || SND_SOC_AC97_BUS_NEW)select SND_SOC_WM9713 if (SND_SOC_AC97_BUS || SND_SOC_AC97_BUS_NEW)select SND_SOC_WSA881X if SOUNDWIRE
imply SND_SOC_88PM860Ximply SND_SOC_L3imply SND_SOC_AB8500_CODECimply SND_SOC_AC97_CODECimply SND_SOC_AD1836imply SND_SOC_AD193X_SPIimply SND_SOC_AD193X_I2Cimply SND_SOC_AD1980imply SND_SOC_AD73311imply SND_SOC_ADAU1373imply SND_SOC_ADAU1761_I2Cimply SND_SOC_ADAU1761_SPIimply SND_SOC_ADAU1781_I2Cimply SND_SOC_ADAU1781_SPIimply SND_SOC_ADAV801imply SND_SOC_ADAV803imply SND_SOC_ADAU1977_SPIimply SND_SOC_ADAU1977_I2Cimply SND_SOC_ADAU1701imply SND_SOC_ADAU7002imply SND_SOC_ADAU7118_I2Cimply SND_SOC_ADAU7118_HWimply SND_SOC_ADS117Ximply SND_SOC_AK4104imply SND_SOC_AK4118imply SND_SOC_AK4458imply SND_SOC_AK4535imply SND_SOC_AK4554imply SND_SOC_AK4613imply SND_SOC_AK4641imply SND_SOC_AK4642imply SND_SOC_AK4671imply SND_SOC_AK5386imply SND_SOC_AK5558imply SND_SOC_ALC5623imply SND_SOC_ALC5632imply SND_SOC_BT_SCOimply SND_SOC_BD28623imply SND_SOC_CQ0093VCimply SND_SOC_CROS_EC_CODECimply SND_SOC_CS35L32imply SND_SOC_CS35L33imply SND_SOC_CS35L34imply SND_SOC_CS35L35imply SND_SOC_CS35L36imply SND_SOC_CS42L42imply SND_SOC_CS42L51_I2Cimply SND_SOC_CS42L52imply SND_SOC_CS42L56imply SND_SOC_CS42L73imply SND_SOC_CS4265imply SND_SOC_CS4270imply SND_SOC_CS4271_I2Cimply SND_SOC_CS4271_SPIimply SND_SOC_CS42XX8_I2Cimply SND_SOC_CS43130imply SND_SOC_CS4341imply SND_SOC_CS4349imply SND_SOC_CS47L15imply SND_SOC_CS47L24imply SND_SOC_CS47L35imply SND_SOC_CS47L85imply SND_SOC_CS47L90imply SND_SOC_CS47L92imply SND_SOC_CS53L30imply SND_SOC_CX20442imply SND_SOC_CX2072Ximply SND_SOC_DA7210imply SND_SOC_DA7213imply SND_SOC_DA7218imply SND_SOC_DA7219imply SND_SOC_DA732Ximply SND_SOC_DA9055imply SND_SOC_DMICimply SND_SOC_ES8316imply SND_SOC_ES8328_SPIimply SND_SOC_ES8328_I2Cimply SND_SOC_ES7134imply SND_SOC_ES7241imply SND_SOC_GTM601imply SND_SOC_HDAC_HDMIimply SND_SOC_HDAC_HDAimply SND_SOC_ICS43432imply SND_SOC_INNO_RK3036imply SND_SOC_ISABELLEimply SND_SOC_JZ4740_CODECimply SND_SOC_JZ4725B_CODECimply SND_SOC_JZ4770_CODECimply SND_SOC_LM4857imply SND_SOC_LM49453imply SND_SOC_LOCHNAGAR_SCimply SND_SOC_MAX98088imply SND_SOC_MAX98090imply SND_SOC_MAX98095imply SND_SOC_MAX98357Aimply SND_SOC_MAX98371imply SND_SOC_MAX98504imply SND_SOC_MAX9867imply SND_SOC_MAX98925imply SND_SOC_MAX98926imply SND_SOC_MAX98927imply SND_SOC_MAX98373imply SND_SOC_MAX9850imply SND_SOC_MAX9860imply SND_SOC_MAX9759imply SND_SOC_MAX9768imply SND_SOC_MAX9877imply SND_SOC_MC13783imply SND_SOC_ML26124imply SND_SOC_MT6351imply SND_SOC_MT6358imply SND_SOC_MT6660imply SND_SOC_NAU8540imply SND_SOC_NAU8810imply SND_SOC_NAU8822imply SND_SOC_NAU8824imply SND_SOC_NAU8825imply SND_SOC_HDMI_CODECimply SND_SOC_PCM1681imply SND_SOC_PCM1789_I2Cimply SND_SOC_PCM179X_I2Cimply SND_SOC_PCM179X_SPIimply SND_SOC_PCM186X_I2Cimply SND_SOC_PCM186X_SPIimply SND_SOC_PCM3008imply SND_SOC_PCM3060_I2Cimply SND_SOC_PCM3060_SPIimply SND_SOC_PCM3168A_I2Cimply SND_SOC_PCM3168A_SPIimply SND_SOC_PCM5102Aimply SND_SOC_PCM512x_I2Cimply SND_SOC_PCM512x_SPIimply SND_SOC_RK3328imply SND_SOC_RT274imply SND_SOC_RT286imply SND_SOC_RT298imply SND_SOC_RT1011imply SND_SOC_RT1015imply SND_SOC_RT1305imply SND_SOC_RT1308imply SND_SOC_RT5514imply SND_SOC_RT5616imply SND_SOC_RT5631imply SND_SOC_RT5640imply SND_SOC_RT5645imply SND_SOC_RT5651imply SND_SOC_RT5659imply SND_SOC_RT5660imply SND_SOC_RT5663imply SND_SOC_RT5665imply SND_SOC_RT5668imply SND_SOC_RT5670imply SND_SOC_RT5677imply SND_SOC_RT5682imply SND_SOC_RT700_SDWimply SND_SOC_RT711_SDWimply SND_SOC_RT715_SDWimply SND_SOC_RT1308_SDWimply SND_SOC_SGTL5000imply SND_SOC_SI476Ximply SND_SOC_SIMPLE_AMPLIFIERimply SND_SOC_SIRF_AUDIO_CODECimply SND_SOC_SPDIFimply SND_SOC_SSM2305imply SND_SOC_SSM2518imply SND_SOC_SSM2602_SPIimply SND_SOC_SSM2602_I2Cimply SND_SOC_SSM4567imply SND_SOC_STA32Ximply SND_SOC_STA350imply SND_SOC_STA529imply SND_SOC_STAC9766imply SND_SOC_STI_SASimply SND_SOC_TAS2552imply SND_SOC_TAS2562imply SND_SOC_TAS2770imply SND_SOC_TAS5086imply SND_SOC_TAS571Ximply SND_SOC_TAS5720imply SND_SOC_TAS6424imply SND_SOC_TDA7419imply SND_SOC_TFA9879imply SND_SOC_TLV320AIC23_I2Cimply SND_SOC_TLV320AIC23_SPIimply SND_SOC_TLV320AIC26imply SND_SOC_TLV320AIC31XXimply SND_SOC_TLV320AIC32X4_I2Cimply SND_SOC_TLV320AIC32X4_SPIimply SND_SOC_TLV320AIC3Ximply SND_SOC_TPA6130A2imply SND_SOC_TLV320DAC33imply SND_SOC_TSCS42XXimply SND_SOC_TSCS454imply SND_SOC_TS3A227Eimply SND_SOC_TWL4030imply SND_SOC_TWL6040imply SND_SOC_UDA1334imply SND_SOC_UDA134Ximply SND_SOC_UDA1380imply SND_SOC_WCD9335imply SND_SOC_WCD934Ximply SND_SOC_WL1273imply SND_SOC_WM0010imply SND_SOC_WM1250_EV1imply SND_SOC_WM2000imply SND_SOC_WM2200imply SND_SOC_WM5100imply SND_SOC_WM5102imply SND_SOC_WM5110imply SND_SOC_WM8350imply SND_SOC_WM8400imply SND_SOC_WM8510imply SND_SOC_WM8523imply SND_SOC_WM8524imply SND_SOC_WM8580imply SND_SOC_WM8711imply SND_SOC_WM8727imply SND_SOC_WM8728imply SND_SOC_WM8731imply SND_SOC_WM8737imply SND_SOC_WM8741imply SND_SOC_WM8750imply SND_SOC_WM8753imply SND_SOC_WM8770imply SND_SOC_WM8776imply SND_SOC_WM8782imply SND_SOC_WM8804_I2Cimply SND_SOC_WM8804_SPIimply SND_SOC_WM8900imply SND_SOC_WM8903imply SND_SOC_WM8904imply SND_SOC_WM8940imply SND_SOC_WM8955imply SND_SOC_WM8960imply SND_SOC_WM8961imply SND_SOC_WM8962imply SND_SOC_WM8971imply SND_SOC_WM8974imply SND_SOC_WM8978imply SND_SOC_WM8983imply SND_SOC_WM8985imply SND_SOC_WM8988imply SND_SOC_WM8990imply SND_SOC_WM8991imply SND_SOC_WM8993imply SND_SOC_WM8994imply SND_SOC_WM8995imply SND_SOC_WM8996imply SND_SOC_WM8997imply SND_SOC_WM8998imply SND_SOC_WM9081imply SND_SOC_WM9090imply SND_SOC_WM9705imply SND_SOC_WM9712imply SND_SOC_WM9713imply SND_SOC_WSA881X help Normally ASoC codec drivers are only built if a machine driver which uses them is also built since they are only usable with a machine@@ -283,6 +283,7 @@ config SND_SOC_ALL_CODECS
config SND_SOC_88PM860X tristate
depends on MFD_88PM860Xconfig SND_SOC_ARIZONA tristate @@ -1301,11 +1302,13 @@ config SND_SOC_TSCS454 Add support for Tempo Semiconductor's TSCS454 audio CODEC.
config SND_SOC_TWL4030
select MFD_TWL4030_AUDIO tristate
depends on TWL4030_COREselect MFD_TWL4030_AUDIOconfig SND_SOC_TWL6040 tristate
depends on TWL6040_COREconfig SND_SOC_UDA1334 tristate "NXP UDA1334 DAC" @@ -1366,9 +1369,12 @@ config SND_SOC_WM5110
config SND_SOC_WM8350 tristate
depends on MFD_WM8350config SND_SOC_WM8400 tristate
# FIXME nothing selects SND_SOC_WM8400??depends on MFD_WM8400config SND_SOC_WM8510 tristate "Wolfson Microelectronics WM8510 CODEC" -- 2.17.1
On Wed, Feb 12, 2020 at 01:09:30AM +0900, Masahiro Yamada wrote:
Why does SND_SOC_ALL_CODECS exist in the first place? Is this compile-test or run-test?
It's for compile coverage.
If it is the former, allyesconfig / allmodconfig covers all of them.
No it doesn't. Those only enable user selectable options, many CODEC drivers don't have user selectable options as they are selected by machine drivers some of which either have more platform specific dependencies that the CODEC drivers or aren't upstream.
On Wed, Feb 12, 2020 at 1:19 AM Mark Brown broonie@kernel.org wrote:
On Wed, Feb 12, 2020 at 01:09:30AM +0900, Masahiro Yamada wrote:
Why does SND_SOC_ALL_CODECS exist in the first place? Is this compile-test or run-test?
It's for compile coverage.
If it is the former, allyesconfig / allmodconfig covers all of them.
No it doesn't. Those only enable user selectable options, many CODEC drivers don't have user selectable options as they are selected by machine drivers some of which either have more platform specific dependencies that the CODEC drivers or aren't upstream.
Maybe, can you add a prompt which is only displayed if COMPILE_TEST is enabled?
For example,
config SND_SOC_L3 tristate "some prompt" if COMPILE_TEST
In normal use-cases, this CONFIG option is hidden since we expect it to be selected by some machine options.
If COMPILE_TEST is enabled, we can toggle as you like.
If I run $ git grep "if COMPILE_TEST"
I see many usages of this tip.
On Wed, Feb 12, 2020 at 01:38:16AM +0900, Masahiro Yamada wrote:
On Wed, Feb 12, 2020 at 1:19 AM Mark Brown broonie@kernel.org wrote:
No it doesn't. Those only enable user selectable options, many CODEC drivers don't have user selectable options as they are selected by machine drivers some of which either have more platform specific dependencies that the CODEC drivers or aren't upstream.
Maybe, can you add a prompt which is only displayed if COMPILE_TEST is enabled?
For example,
config SND_SOC_L3 tristate "some prompt" if COMPILE_TEST
That's effectively what this is doing, it's just making it a single option that enables everything en masse rather than lots of per driver options and isn't as esoteric so is easier to understand (almost all the existing examples are Renesas specific).
participants (3)
-
Geert Uytterhoeven -
Mark Brown -
Masahiro Yamada