On Mon, Jun 26, 2017 at 09:48:59AM +0800, Chen-Yu Tsai wrote:
On Sat, Jun 24, 2017 at 2:24 PM, Danny Milosavljevic
+static const struct snd_kcontrol_new sun4i_codec_mixer_controls[] = {
SOC_DAPM_DOUBLE("DAC Playback Switch", SUN4I_CODEC_DAC_ACTL,SUN4I_CODEC_DAC_ACTL_LDACLMIXS, SUN4I_CODEC_DAC_ACTL_RDACRMIXS, 1, 0),
This changes the mixer control names exposed to userspace. While I think consolidating mixer controls is nice, giving the user a less cluttered view, I'm not sure about the changing a possible userspace ABI.
Maxime, ALSA and ASoC people, any comments about this?
It's better not to change things but on the other hand it really depends if there's any users who will notice - if nobody notices it doesn't matter.
Can we *please* get some review of the rest of this series? It's enormous, we're on version *13* and the first review on this version here is on patch 5. If the first four patches are fine then please says so, if they're not then let's have some review comments so things can move towards merging them.