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. ��.n��������+%������w��{.n����z�{��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f