[alsa-devel] [PATCH] ASoC: wm8741: Fix (and refactor) volume controls

Mark Brown broonie at kernel.org
Sat Jan 16 17:49:39 CET 2016


On Sat, Jan 16, 2016 at 01:25:29PM +0100, Sergej Sawazki wrote:

> +static unsigned int _wm8741_volume_get(struct snd_soc_codec *codec,
> +		struct soc_mixer_control *mc, unsigned int lsb_reg)

All the readback stuff looks entirely generic and should be using
generic functions.  If they don't exist add them to the core.

> +static int wm8741_volume_set(struct snd_kcontrol *kcontrol,
> +		struct snd_ctl_elem_value *ucontrol)
> +{

Similarly for the set stuff, there's nothing visibly device specific
here.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20160116/106064de/attachment-0001.sig>


More information about the Alsa-devel mailing list