Re: [PATCHv2 3/6] ASoC: codec: cpcap: new codec

From: Sebastian Reichel
Date: Mon Jul 17 2017 - 10:18:18 EST


Hi,

On Mon, Jul 17, 2017 at 02:29:29AM -0700, Tony Lindgren wrote:
> * Sebastian Reichel <sebastian.reichel@xxxxxxxxxxxxxxx> [170712 08:19]:
> > diff --git a/sound/soc/codecs/Kconfig b/sound/soc/codecs/Kconfig
> > index 6c78b0b49b81..1615e5acc8fd 100644
> > --- a/sound/soc/codecs/Kconfig
> > +++ b/sound/soc/codecs/Kconfig
> > @@ -401,6 +401,11 @@ config SND_SOC_ALC5632
> > config SND_SOC_BT_SCO
> > tristate "Dummy BT SCO codec driver"
> >
> > +config SND_SOC_CPCAP
> > + tristate "Motorola CPCAP codec"
> > + depends on MFD_CPCAP
> > + default MFD_CPCAP
> > +
>
> Should it also have depends on SND_AUDIO_GRAPH_CARD?

The codec does not depend on audio graph card. You can
also use it with (deprecated) simple card or a custom
written one. In case of a custom written one it would
make sense to add a dependency from the custom written
one to SND_SOC_CPCAP. So I think we should not add such
a dependency.

-- Sebastian

Attachment: signature.asc
Description: PGP signature