Re: [PATCH 1/2] ASoC: TWL4030: Fix for capture mixer strings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Aug 13, 2009 at 03:59:33PM +0300, Peter Ujfalusi wrote:

>  static const struct snd_kcontrol_new twl4030_dapm_analoglmic_controls[] = {
> -	SOC_DAPM_SINGLE("Main mic", TWL4030_REG_ANAMICL, 0, 1, 0),
> -	SOC_DAPM_SINGLE("Headset mic", TWL4030_REG_ANAMICL, 1, 1, 0),
> -	SOC_DAPM_SINGLE("AUXL", TWL4030_REG_ANAMICL, 2, 1, 0),
> -	SOC_DAPM_SINGLE("Carkit mic", TWL4030_REG_ANAMICL, 3, 1, 0),
> +	SOC_DAPM_SINGLE("Main mic Capture Route",
> +			TWL4030_REG_ANAMICL, 0, 1, 0),
> +	SOC_DAPM_SINGLE("Headset mic Capture Route",
> +			TWL4030_REG_ANAMICL, 1, 1, 0),
> +	SOC_DAPM_SINGLE("AUXL Capture Route",
> +			TWL4030_REG_ANAMICL, 2, 1, 0),
> +	SOC_DAPM_SINGLE("Carkit mic Capture Route",
> +			TWL4030_REG_ANAMICL, 3, 1, 0),

These should really be "foo Switch" - a switch is an on/off control.  If
there's a volume control associated with the path then alsamixer will
show the pair as a combined volume/mute control if they have the same
name but with Volume and Switch as the last word.

I'd also spell "mic" "Mic".
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux