RE: [PATCH 2/2] ASoC: codecs: Add da7218 codec driver

[Opensource [Adam Thomson]]

On November 6, 2015 11:22, 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?

Input filters 1L/R and 2L/R, which are fed from either Mic1(ADC1) or DMic1L/R
and Mic2(ADC2) or DMic2L/R.