Re: [PATCH] ASoC: Add max98926 codec driver

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

 



On Thu, Jan 14, 2016 at 02:03:33PM -0800, anish kumar wrote:

> +static const struct snd_kcontrol_new max98926_dai_controls[] = {
> +	SOC_DAPM_SINGLE("Left", MAX98926_GAIN,
> +		MAX98926_DAC_IN_SEL_SHIFT, 0, 0),
> +	SOC_DAPM_SINGLE("Right", MAX98926_GAIN,
> +		MAX98926_DAC_IN_SEL_SHIFT, 1, 0),
> +	SOC_DAPM_SINGLE("LeftRight", MAX98926_GAIN,
> +		MAX98926_DAC_IN_SEL_SHIFT, 2, 0),

These are on/off switches too and so should end in Switch like all the
other on/off controls.  Other than this the driver looks good so I've
applied it, please send a followup patch correcting this.

> +	SOC_DAPM_SINGLE("(Left+Right)/2 Switch", MAX98926_GAIN,
> +		MAX98926_DAC_IN_SEL_SHIFT, 3, 0),
> +};

Attachment: signature.asc
Description: PGP signature

_______________________________________________
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