Re: [PATCH 2/2] ASoC: cs4270: add Master Playback Switch

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

 



On Fri, Apr 24, 2009 at 9:37 AM, Daniel Mack <daniel@xxxxxxxx> wrote:
> This adds a new control named 'Master Playback Switch' for cs4270
> codecs. It is implemented using the new SOC_DOUBLE_EXT macro to catch
> the put function and store the information about manually set mute
> controls from userspace. When a manual mute is set, we don't want the
> soc core to un-mute the outputs.
>
> Renamed cs4270_mute() to cs4270_dai_mute() to avoid confusion.
>
> Signed-off-by: Daniel Mack <daniel@xxxxxxxx>
> Cc: Mark Brown <broonie@xxxxxxxxxxxxx>
> Cc: Timur Tabi <timur@xxxxxxxxxxxxx>

Acked-by: Timur Tabi <timur@xxxxxxxxxxxxx>

However, ...

> @@ -486,7 +518,9 @@ static const struct snd_kcontrol_new cs4270_snd_controls[] = {
>        SOC_SINGLE("Zero Cross Switch", CS4270_TRANS, 5, 1, 0),
>        SOC_SINGLE("Popguard Switch", CS4270_MODE, 0, 1, 1),
>        SOC_SINGLE("Auto-Mute Switch", CS4270_MUTE, 5, 1, 0),
> -       SOC_DOUBLE("Master Capture Switch", CS4270_MUTE, 3, 4, 1, 1)
> +       SOC_DOUBLE("Master Capture Switch", CS4270_MUTE, 3, 4, 1, 1),
> +       SOC_DOUBLE_EXT("Master Playback Switch", CS4270_MUTE, 0, 1, 1, 1,
> +               snd_soc_get_volsw, cs4270_soc_put_mute)
>  };

Mark, could you add a comma at the end of "cs4270_soc_put_mute)" so
that any future patches won't need to delete any lines?

-- 
Timur Tabi
Linux kernel developer at Freescale
_______________________________________________
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