On Thu, May 28, 2020 at 11:32:30AM +0530, Akshu Agrawal wrote:

> @@ -307,6 +295,7 @@ static const struct snd_kcontrol_new acp3x_mc_controls[] 
> = {
>       SOC_DAPM_PIN_SWITCH("Headphone Jack"),
>       SOC_DAPM_PIN_SWITCH("Spk"),
>       SOC_DAPM_PIN_SWITCH("Headset Mic"),
> +     SOC_SINGLE_BOOL_EXT("DMIC Switch", 0, dmic_get, dmic_set),

This now looks like a mute for the DMIC which is not accurate.  I really
do think this would be better as a mux, rather than labelling it with
positions on the case you could follow what other drivers do and label
it with the connections on the device.

Attachment: signature.asc
Description: PGP signature

Reply via email to