[alsa-devel] [PATCH 5/5] ASoC: sh: fsi: add FSI-WM8978 support

[Kuninori Morimoto]

Hi Mark

> > > +config SND_FSI_WM8978
> > > +	tristate "FSI-WM8978 sound support"
> > > +	depends on SND_SOC_SH4_FSI
> > > +	select SND_SOC_WM8978
> > > +        select SND_FSI_CODEC
> > > +	help
> > > +	  This option enables generic sound support for the
> > > +	  FSI - WM8978 unit
> > 
> > Rather than having an explicit Kconfig option like this it might be more
> > sensible to have the boards select their CODEC driver.  We should only
> > build the CODEC driver if the user enables the subsystem IIRC so this
> > shouldn't be a big cost.  You could then just have a single Kconfig for
> > the simple driver.
> 
> OK. I understand.
> I guess this opinion is not only for this "FSI-WM8978" but also for existing cards.

Hmm...
If board selects their CODEC driver, the driver will be forcefully included to kernel.
And some of these are depend on extra config like I2C.
I'm afraid about this kind of "forcefully select".

Of course we can create selectable Kconfig for each boards.
but it is just complex..

Best regards
---
Kuninori Morimoto