From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.5 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 08890C33CB1 for ; Tue, 14 Jan 2020 13:16:42 +0000 (UTC) Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 88E0724676 for ; Tue, 14 Jan 2020 13:16:41 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gerhold.net header.i=@gerhold.net header.b="K7YmquUJ" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 88E0724676 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=gerhold.net Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=alsa-devel-bounces@alsa-project.org Received: from alsa1.perex.cz (unknown [207.180.221.201]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id 9358D2FD3; Tue, 14 Jan 2020 13:08:38 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz 9358D2FD3 Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id 1AEE0F8014D; Tue, 14 Jan 2020 13:08:38 +0100 (CET) Received: by alsa1.perex.cz (Postfix, from userid 50401) id 0577CF80121; Tue, 14 Jan 2020 13:08:35 +0100 (CET) Received: from mo6-p02-ob.smtp.rzone.de (mo6-p02-ob.smtp.rzone.de [IPv6:2a01:238:20a:202:5302::3]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id 9D2FBF80121 for ; Tue, 14 Jan 2020 13:08:28 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz 9D2FBF80121 Authentication-Results: alsa1.perex.cz; dkim=pass (2048-bit key) header.d=gerhold.net header.i=@gerhold.net header.b="K7YmquUJ" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1579003707; s=strato-dkim-0002; d=gerhold.net; h=In-Reply-To:References:Message-ID:Subject:Cc:To:From:Date: X-RZG-CLASS-ID:X-RZG-AUTH:From:Subject:Sender; bh=w0hCOs0pUCXzBGfEbGi023I9TPLGyinZfevmistvGHE=; b=K7YmquUJH/3O2JtUBApy4XIloM5aUgfQ2WzZtbt6Qo9aiVCWoa+n2FRUiNvSlCX4to eRkb5T8HwqphpNXuKl39S3qELvdIlakdRkFnZdT/r3MWSYn8VNV3dC7JF93+qLDHby9R y8ZP+W/1NIAVqaiTrUxOPyJ8R24uGePZe2E7QJT8SwtY+xUMnZX2ZURRJRwsAPESqhJB 1qHQ2o3iOnRBQS983A6LWQUuF1TLQdL6v+a5ogUs4xeHBOXOdOh1ZWshjJwc44/EedMs WS2cPyuoTwfZ7BPcQHTFAmtjZ86vFlFrHj6Ypf5ZtQlmSRoGkyzjIj5heU5u2ek3rapD SyCw== X-RZG-AUTH: ":P3gBZUipdd93FF5ZZvYFPugejmSTVR2nRPhVOQ/OcYgojyw4j34+u267EpF+OQRc4oDWF5yUxHE=" X-RZG-CLASS-ID: mo00 Received: from gerhold.net by smtp.strato.de (RZmta 46.1.4 AUTH) with ESMTPSA id h048a6w0EC8OUdL (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256 bits)) (Client did not present a certificate); Tue, 14 Jan 2020 13:08:24 +0100 (CET) Date: Tue, 14 Jan 2020 13:08:17 +0100 From: Stephan Gerhold To: Srinivas Kandagatla Message-ID: <20200114120817.GA52379@gerhold.net> References: <20200111164006.43074-1-stephan@gerhold.net> <20200111164006.43074-4-stephan@gerhold.net> <314b9a9b-13cd-94f5-0967-f0d4e66d65a1@linaro.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <314b9a9b-13cd-94f5-0967-f0d4e66d65a1@linaro.org> Cc: alsa-devel@alsa-project.org, Takashi Iwai , Liam Girdwood , Mark Brown , ~postmarketos/upstreaming@lists.sr.ht, Nikita Travkin Subject: Re: [alsa-devel] [PATCH 3/4] ASoC: msm8916-wcd-analog: Simplify MIC BIAS Internal X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" On Tue, Jan 14, 2020 at 10:54:44AM +0000, Srinivas Kandagatla wrote: > > > On 11/01/2020 16:40, Stephan Gerhold wrote: > > At the moment, MIC BIAS Internal* and MIC BIAS External* both reference > > the same register, and have a part of their initialization sequence > > duplicated. > > > > In general, the sequence for enabling MIC BIAS InternalX seems to be: > > 1. Enable MIC BIAS ExternalX > > 2. Enable internal RBIAS resistor > > Does not sound correct to me. > > What external means here is if the MICBIAS has external pull up resistor or > not. And this is very much board specific. In order for driver to > enable/disable internal pull up resistor in such cases we use two dapm paths > to differentiate it. > You are right. Maybe the naming is a bit confusing here. Let me try to clarify it: If I understand it correctly, setting the MICB_EN bit in CDC_A_MICB_1_EN enables the MIC_BIAS1 supply. This supply can be either used with an external pull up resistor ("MIC BIAS External1") or with the internal pull up resistor ("MIC BIAS Internal1"). Which one of them, is board-specific. To use the internal pull up resistor, we need to set the TX1_INT_RBIAS_EN bit in CDC_A_MICB_1_INT_RBIAS additionally. In other words, the sequence for enabling MIC BIAS Internal1 is: I1. Enable MIC_BIAS1 supply (MICB_EN bit in CDC_A_MICB_1_EN) I2. Enable internal RBIAS (TX1_INT_RBIAS_EN bit in CDC_A_MICB_1_INT_RBIAS) The sequence for enabling MIC BIAS External1 is: E1. Enable MIC_BIAS1 supply (MICB_EN bit in CDC_A_MICB_1_EN) Right now we have: SND_SOC_DAPM_SUPPLY("MIC BIAS Internal1", CDC_A_MICB_1_EN, 7, 0, ...) // I1 SND_SOC_DAPM_SUPPLY("MIC BIAS External1", CDC_A_MICB_1_EN, 7, 0, ...) // E1 I2 is done in the PM event handler (pm8916_wcd_analog_enable_micbias_int1). The idea of this patch is to simplify this, and use one DAPM supply that handles the common (I1/E1), and one DAPM supply that handles (I2). Translated to code we get: SND_SOC_DAPM_SUPPLY("MIC BIAS1", CDC_A_MICB_1_EN, 7, 0, ...) // I1/E1 SND_SOC_DAPM_SUPPLY("MIC BIAS Internal1", CDC_A_MICB_1_INT_RBIAS, 7, 0, ...) // I2 SND_SOC_DAPM_SUPPLY("MIC BIAS External1", SND_SOC_NOPM, ...) // dummy And the routes: {"MIC BIAS Internal1", NULL, "MIC BIAS1"} // If I2, also do I1 {"MIC BIAS External1", NULL, "MIC BIAS1"} // Only do E1 As you can see, for "MIC BIAS External1" we just do "MIC BIAS1". The confusing element of this patch might be that I simplified it a bit further and combined "MIC BIAS1" with "MIC BIAS External1". This works because we don't do anything extra for "MIC BIAS External1". Does that make sense? Thanks for taking the time to review all my patches. Stephan > > --srini > > > > > > > > This means that we can simplify the code by modelling MIC BIAS InternalX > > as supply connected to MIC BIAS ExternalX. MIC BIAS InternalX is only > > responsible for enabling the internal RBIAS resistor (2). The DAPM enable > > sequence will automatically enable MIC BIAS ExternalX (1). > > > > This makes it much easier to add support for MIC BIAS Internal3 > > as a next step. > > > > Tested-by: Nikita Travkin # longcheer-l8150 > > Cc: Srinivas Kandagatla > > Signed-off-by: Stephan Gerhold > > --- > > sound/soc/codecs/msm8916-wcd-analog.c | 57 ++++++--------------------- > > 1 file changed, 13 insertions(+), 44 deletions(-) > > > > diff --git a/sound/soc/codecs/msm8916-wcd-analog.c b/sound/soc/codecs/msm8916-wcd-analog.c > > index 1f7964beb20c..930baae6eff5 100644 > > --- a/sound/soc/codecs/msm8916-wcd-analog.c > > +++ b/sound/soc/codecs/msm8916-wcd-analog.c > > @@ -389,23 +389,17 @@ static int pm8916_wcd_analog_enable_micbias_ext(struct snd_soc_component > > return 0; > > } > > -static int pm8916_wcd_analog_enable_micbias_int(struct snd_soc_component > > - *component, int event, > > - int reg, u32 cap_mode) > > +static int pm8916_wcd_analog_enable_micbias_int(struct snd_soc_dapm_widget *w, > > + struct snd_kcontrol *kcontrol, > > + int event) > > { > > + struct snd_soc_component *component = snd_soc_dapm_to_component(w->dapm); > > switch (event) { > > case SND_SOC_DAPM_PRE_PMU: > > - snd_soc_component_update_bits(component, reg, MICB_1_EN_PULL_DOWN_EN_MASK, 0); > > snd_soc_component_update_bits(component, CDC_A_MICB_1_EN, > > MICB_1_EN_OPA_STG2_TAIL_CURR_MASK, > > MICB_1_EN_OPA_STG2_TAIL_CURR_1_60UA); > > - > > - break; > > - case SND_SOC_DAPM_POST_PMU: > > - pm8916_wcd_analog_micbias_enable(component); > > - snd_soc_component_update_bits(component, CDC_A_MICB_1_EN, > > - MICB_1_EN_BYP_CAP_MASK, cap_mode); > > break; > > } > > @@ -437,26 +431,6 @@ static int pm8916_wcd_analog_enable_micbias_ext2(struct > > } > > -static int pm8916_wcd_analog_enable_micbias_int1(struct > > - snd_soc_dapm_widget > > - *w, struct snd_kcontrol > > - *kcontrol, int event) > > -{ > > - struct snd_soc_component *component = snd_soc_dapm_to_component(w->dapm); > > - struct pm8916_wcd_analog_priv *wcd = snd_soc_component_get_drvdata(component); > > - > > - switch (event) { > > - case SND_SOC_DAPM_PRE_PMU: > > - snd_soc_component_update_bits(component, CDC_A_MICB_1_INT_RBIAS, > > - MICB_1_INT_TX1_INT_RBIAS_EN_MASK, > > - MICB_1_INT_TX1_INT_RBIAS_EN_ENABLE); > > - break; > > - } > > - > > - return pm8916_wcd_analog_enable_micbias_int(component, event, w->reg, > > - wcd->micbias1_cap_mode); > > -} > > - > > static int pm8916_mbhc_configure_bias(struct pm8916_wcd_analog_priv *priv, > > bool micbias2_enabled) > > { > > @@ -564,9 +538,8 @@ static int pm8916_wcd_analog_enable_micbias_int2(struct > > switch (event) { > > case SND_SOC_DAPM_PRE_PMU: > > - snd_soc_component_update_bits(component, CDC_A_MICB_1_INT_RBIAS, > > - MICB_1_INT_TX2_INT_RBIAS_EN_MASK, > > - MICB_1_INT_TX2_INT_RBIAS_EN_ENABLE); > > + snd_soc_component_update_bits(component, CDC_A_MICB_2_EN, > > + CDC_A_MICB_2_PULL_DOWN_EN_MASK, 0); > > break; > > case SND_SOC_DAPM_POST_PMU: > > pm8916_mbhc_configure_bias(wcd, true); > > @@ -576,8 +549,7 @@ static int pm8916_wcd_analog_enable_micbias_int2(struct > > break; > > } > > - return pm8916_wcd_analog_enable_micbias_int(component, event, w->reg, > > - wcd->micbias2_cap_mode); > > + return pm8916_wcd_analog_enable_micbias_int(w, kcontrol, event); > > } > > static int pm8916_wcd_analog_enable_adc(struct snd_soc_dapm_widget *w, > > @@ -878,12 +850,10 @@ static const struct snd_soc_dapm_route pm8916_wcd_analog_audio_map[] = { > > {"SPK PA", NULL, "SPK DAC"}, > > {"SPK DAC", "Switch", "PDM_RX3"}, > > - {"MIC BIAS Internal1", NULL, "INT_LDO_H"}, > > - {"MIC BIAS Internal2", NULL, "INT_LDO_H"}, > > + {"MIC BIAS Internal1", NULL, "MIC BIAS External1"}, > > + {"MIC BIAS Internal2", NULL, "MIC BIAS External2"}, > > {"MIC BIAS External1", NULL, "INT_LDO_H"}, > > {"MIC BIAS External2", NULL, "INT_LDO_H"}, > > - {"MIC BIAS Internal1", NULL, "vdd-micbias"}, > > - {"MIC BIAS Internal2", NULL, "vdd-micbias"}, > > {"MIC BIAS External1", NULL, "vdd-micbias"}, > > {"MIC BIAS External2", NULL, "vdd-micbias"}, > > }; > > @@ -937,11 +907,10 @@ static const struct snd_soc_dapm_widget pm8916_wcd_analog_dapm_widgets[] = { > > SND_SOC_DAPM_SUPPLY("RX_BIAS", CDC_A_RX_COM_BIAS_DAC, 7, 0, NULL, 0), > > /* TX */ > > - SND_SOC_DAPM_SUPPLY("MIC BIAS Internal1", CDC_A_MICB_1_EN, 7, 0, > > - pm8916_wcd_analog_enable_micbias_int1, > > - SND_SOC_DAPM_PRE_PMU | SND_SOC_DAPM_POST_PMU | > > - SND_SOC_DAPM_POST_PMD), > > - SND_SOC_DAPM_SUPPLY("MIC BIAS Internal2", CDC_A_MICB_2_EN, 7, 0, > > + SND_SOC_DAPM_SUPPLY("MIC BIAS Internal1", CDC_A_MICB_1_INT_RBIAS, 7, 0, > > + pm8916_wcd_analog_enable_micbias_int, > > + SND_SOC_DAPM_PRE_PMU), > > + SND_SOC_DAPM_SUPPLY("MIC BIAS Internal2", CDC_A_MICB_1_INT_RBIAS, 4, 0, > > pm8916_wcd_analog_enable_micbias_int2, > > SND_SOC_DAPM_PRE_PMU | SND_SOC_DAPM_POST_PMU | > > SND_SOC_DAPM_POST_PMD), > > _______________________________________________ Alsa-devel mailing list Alsa-devel@alsa-project.org https://mailman.alsa-project.org/mailman/listinfo/alsa-devel