Re: [PATCH v15 3/6] ASoC: es9356-sdca: Add ES9356 SDCA driver

[Nathan Chancellor <nathan@kernel.org>]

On Tue, May 19, 2026 at 02:42:08PM +0200, Pierre-Louis Bossart wrote:
> On 5/19/26 00:46, Nathan Chancellor wrote:
> > Maybe something like this is needed?
> > 
> > diff --git a/sound/soc/codecs/Kconfig b/sound/soc/codecs/Kconfig
> > index 5fdd0334c355..b9e8df5ddfd5 100644
> > --- a/sound/soc/codecs/Kconfig
> > +++ b/sound/soc/codecs/Kconfig
> > @@ -1302,7 +1302,9 @@ config SND_SOC_ES8389
> >  
> >  config SND_SOC_ES9356
> >          tristate "Everest Semi ES9356 CODEC SDW"
> > +	depends on ACPI
> >          depends on SOUNDWIRE
> > +	select SND_SOC_SDCA
> >          select REGMAP_SOUNDWIRE
> >          select REGMAP_SOUNDWIRE_MBQ
> 
> Maybe this instead?
> 
> depends on SOUNDWIRE
> depends on SND_SOC_SDCA

Yeah, I realized shortly after sending that suggestion through testing
that it would error during the Kconfig stage due to a recursive
dependency issue. However, CONFIG_SND_SOC_SDCA is not a user selectable
symbol and I only see it selected through

  CONFIG_SND_SOC_ACPI_INTEL_MATCH -> CONFIG_SND_SOC_ACPI_INTEL_SDCA_QUIRKS

so is it expected that users have to enable a driver that selects
CONFIG_SND_SOC_ACPI_INTEL_MATCH to get CONFIG_SND_SOC_SDCA?

-- 
Cheers,
Nathan