Re: [PATCH 2/2] ASoC: codecs: Add da7218 codec driver
From: Mark Brown
Date: Fri Nov 06 2015 - 06:22:44 EST
On Fri, Nov 06, 2015 at 11:11:38AM +0000, Opensource [Adam Thomson] wrote:
> On November 5, 2015 15:28, Mark Brown wrote:
> > > +static int da7218_mic_lvl_det_sw_put(struct snd_kcontrol *kcontrol,
> > > + struct snd_ctl_elem_value *ucontrol)
> > > +{
> > Why is this a user visible control?
> I can envisage in a system you may want to choose which capture channels can
> trigger level detection (if any), and this may change depending on the use-case
> at the time, so having it as a control makes sense to me.
What is a "capture channel" here?
Attachment:
signature.asc
Description: PGP signature