From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751127AbdCNEbu convert rfc822-to-8bit (ORCPT ); Tue, 14 Mar 2017 00:31:50 -0400 Received: from smtp.csie.ntu.edu.tw ([140.112.30.61]:59828 "EHLO smtp.csie.ntu.edu.tw" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750924AbdCNEbs (ORCPT ); Tue, 14 Mar 2017 00:31:48 -0400 MIME-Version: 1.0 In-Reply-To: References: <20170210094133.11382-3-mylene.josserand@free-electrons.com> From: Chen-Yu Tsai Date: Tue, 14 Mar 2017 12:31:22 +0800 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: Applied "ASoC: sun8i-codec: Update mixer to use SOC_DAPM_DOUBLE" to the asoc tree To: Mark Brown Cc: =?UTF-8?Q?Myl=C3=A8ne_Josserand?= , Chen-Yu Tsai , Liam Girdwood , Rob Herring , Mark Rutland , Maxime Ripard , Jaroslav Kysela , Takashi Iwai , Thomas Petazzoni , devicetree , Linux-ALSA , linux-kernel , Alexandre Belloni , linux-arm-kernel Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Mark, On Wed, Mar 8, 2017 at 9:22 PM, Mark Brown wrote: > The patch > > ASoC: sun8i-codec: Update mixer to use SOC_DAPM_DOUBLE > > has been applied to the asoc tree at > > git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git > > All being well this means that it will be integrated into the linux-next > tree (usually sometime in the next 24 hours) and sent to Linus during > the next merge window (or sooner if it is a bug fix), however if > problems are discovered then the patch may be dropped or reverted. > > You may get further e-mails resulting from automated or manual testing > and review of the tree, please engage with people reporting problems and > send followup patches addressing any issues that are reported if needed. > > If any updates are required or you are submitting further changes they > should be sent as incremental updates against current git, existing > patches will not be replaced. > > Please add any relevant lists and maintainers to the CCs when replying > to this mail. > > Thanks, > Mark > > From ca14da6e611674cad275f29ac2aaf1e2eb427c6b Mon Sep 17 00:00:00 2001 > From: =?UTF-8?q?Myl=C3=A8ne=20Josserand?= > > Date: Fri, 10 Feb 2017 10:41:31 +0100 > Subject: [PATCH] ASoC: sun8i-codec: Update mixer to use SOC_DAPM_DOUBLE > MIME-Version: 1.0 > Content-Type: text/plain; charset=UTF-8 > Content-Transfer-Encoding: 8bit > > Update the driver to use the new SOC_DAPM_DOUBLE definition > on the digital DAC mixer. > Update the names accordingly as, when they are shared, the > controls are not prefixed with the widget's name anymore. > > Signed-off-by: Mylène Josserand > Acked-by: Chen-Yu Tsai > Signed-off-by: Mark Brown > --- > sound/soc/sunxi/sun8i-codec.c | 45 ++++++++++++++++++++----------------------- > 1 file changed, 21 insertions(+), 24 deletions(-) > > diff --git a/sound/soc/sunxi/sun8i-codec.c b/sound/soc/sunxi/sun8i-codec.c > index d60f6fbd36a2..107fa8213600 100644 > --- a/sound/soc/sunxi/sun8i-codec.c > +++ b/sound/soc/sunxi/sun8i-codec.c > @@ -259,25 +259,20 @@ static int sun8i_codec_hw_params(struct snd_pcm_substream *substream, > return 0; > } > > -static const struct snd_kcontrol_new sun8i_output_left_mixer_controls[] = { > - SOC_DAPM_SINGLE("LSlot 0", SUN8I_DAC_MXR_SRC, > - SUN8I_DAC_MXR_SRC_DACL_MXR_SRC_AIF1DA0L, 1, 0), > - SOC_DAPM_SINGLE("LSlot 1", SUN8I_DAC_MXR_SRC, > - SUN8I_DAC_MXR_SRC_DACL_MXR_SRC_AIF1DA1L, 1, 0), > - SOC_DAPM_SINGLE("DACL", SUN8I_DAC_MXR_SRC, > - SUN8I_DAC_MXR_SRC_DACL_MXR_SRC_AIF2DACL, 1, 0), > - SOC_DAPM_SINGLE("ADCL", SUN8I_DAC_MXR_SRC, > - SUN8I_DAC_MXR_SRC_DACL_MXR_SRC_ADCL, 1, 0), > -}; > - > -static const struct snd_kcontrol_new sun8i_output_right_mixer_controls[] = { > - SOC_DAPM_SINGLE("RSlot 0", SUN8I_DAC_MXR_SRC, > +static const struct snd_kcontrol_new sun8i_dac_mixer_controls[] = { > + SOC_DAPM_DOUBLE("AIF1 Slot 0 Digital DAC Playback Switch", > + SUN8I_DAC_MXR_SRC, > + SUN8I_DAC_MXR_SRC_DACL_MXR_SRC_AIF1DA0L, > SUN8I_DAC_MXR_SRC_DACR_MXR_SRC_AIF1DA0R, 1, 0), > - SOC_DAPM_SINGLE("RSlot 1", SUN8I_DAC_MXR_SRC, > + SOC_DAPM_DOUBLE("AIF1 Slot 1 Digital DAC Playback Switch", > + SUN8I_DAC_MXR_SRC, > + SUN8I_DAC_MXR_SRC_DACL_MXR_SRC_AIF1DA1L, > SUN8I_DAC_MXR_SRC_DACR_MXR_SRC_AIF1DA1R, 1, 0), > - SOC_DAPM_SINGLE("DACR", SUN8I_DAC_MXR_SRC, > + SOC_DAPM_DOUBLE("AIF2 Digital DAC Playback Switch", SUN8I_DAC_MXR_SRC, > + SUN8I_DAC_MXR_SRC_DACL_MXR_SRC_AIF2DACL, > SUN8I_DAC_MXR_SRC_DACR_MXR_SRC_AIF2DACR, 1, 0), > - SOC_DAPM_SINGLE("ADCR", SUN8I_DAC_MXR_SRC, > + SOC_DAPM_DOUBLE("ADC Digital DAC Playback Switch", SUN8I_DAC_MXR_SRC, > + SUN8I_DAC_MXR_SRC_DACL_MXR_SRC_ADCL, > SUN8I_DAC_MXR_SRC_DACR_MXR_SRC_ADCR, 1, 0), > }; > > @@ -293,12 +288,12 @@ static const struct snd_soc_dapm_widget sun8i_codec_dapm_widgets[] = { > SUN8I_AIF1_DACDAT_CTRL_AIF1_DA0R_ENA, 0), > > /* DAC Mixers */ > - SND_SOC_DAPM_MIXER("Left DAC Mixer", SND_SOC_NOPM, 0, 0, > - sun8i_output_left_mixer_controls, > - ARRAY_SIZE(sun8i_output_left_mixer_controls)), > - SND_SOC_DAPM_MIXER("Right DAC Mixer", SND_SOC_NOPM, 0, 0, > - sun8i_output_right_mixer_controls, > - ARRAY_SIZE(sun8i_output_right_mixer_controls)), > + SND_SOC_DAPM_MIXER("Left Digital DAC Mixer", SND_SOC_NOPM, 0, 0, > + sun8i_dac_mixer_controls, > + ARRAY_SIZE(sun8i_dac_mixer_controls)), > + SND_SOC_DAPM_MIXER("Right Digital DAC Mixer", SND_SOC_NOPM, 0, 0, > + sun8i_dac_mixer_controls, > + ARRAY_SIZE(sun8i_dac_mixer_controls)), This patch changes the mixer controls exposed to user space. This driver was just introduced in 4.11-rc1. Would it make sense to merge these cleanup patches as fixes for 4.11, so we don't affect userspace across kernel releases? Thanks ChenYu > > /* Clocks */ > SND_SOC_DAPM_SUPPLY("MODCLK AFI1", SUN8I_MOD_CLK_ENA, > @@ -340,8 +335,10 @@ static const struct snd_soc_dapm_route sun8i_codec_dapm_routes[] = { > { "Digital Right DAC", NULL, "DAC" }, > > /* DAC Mixer Routes */ > - { "Left DAC Mixer", "LSlot 0", "Digital Left DAC"}, > - { "Right DAC Mixer", "RSlot 0", "Digital Right DAC"}, > + { "Left Digital DAC Mixer", "AIF1 Slot 0 Digital DAC Playback Switch", > + "Digital Left DAC"}, > + { "Right Digital DAC Mixer", "AIF1 Slot 0 Digital DAC Playback Switch ", > + "Digital Right DAC"}, > }; > > static struct snd_soc_dai_ops sun8i_codec_dai_ops = { > -- > 2.11.0 >