[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <alpine.DEB.2.20.2002270906590.29598@lnxricardw1.se.axis.com>
Date: Thu, 27 Feb 2020 09:07:28 +0100
From: Ricard Wanderlof <ricardw@...s.com>
To: Dan Murphy <dmurphy@...com>
CC: "lgirdwood@...il.com" <lgirdwood@...il.com>,
"broonie@...nel.org" <broonie@...nel.org>,
"perex@...ex.cz" <perex@...ex.cz>,
"tiwai@...e.com" <tiwai@...e.com>,
"alsa-devel@...a-project.org" <alsa-devel@...a-project.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Ricard Wanderlöf <Ricard.Wanderlof@...s.com>
Subject: Re: [PATCH for-next] ASoC: tlv320adcx140: Fix MIC_BIAS defines for
ADC full scale
On Wed, 26 Feb 2020, Dan Murphy wrote:
> Change the #defines for the ADC full scale bits from MIC_BIAS to
> ADC_FSCALE. This also changes the error message to incidate ADC full
> scale value error as opposed to the Mic bias.
>
> Reported-by: Ricard Wanderlof <ricardw@...s.com>
> Signed-off-by: Dan Murphy <dmurphy@...com>
> ---
> sound/soc/codecs/tlv320adcx140.c | 12 ++++++------
> sound/soc/codecs/tlv320adcx140.h | 8 ++++----
> 2 files changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/sound/soc/codecs/tlv320adcx140.c
> b/sound/soc/codecs/tlv320adcx140.c
> index 93a0cb8e662c..825ace9b5fa7 100644
> --- a/sound/soc/codecs/tlv320adcx140.c
> +++ b/sound/soc/codecs/tlv320adcx140.c
> @@ -758,12 +758,12 @@ static int adcx140_codec_probe(struct
> snd_soc_component *component)
> ret = device_property_read_u8(adcx140->dev, "ti,vref-source",
> &vref_source);
> if (ret)
> - vref_source = ADCX140_MIC_BIAS_VREF_275V;
> + vref_source = ADCX140_ADC_FSCALE_VREF_275V;
>
> - if (vref_source != ADCX140_MIC_BIAS_VREF_275V &&
> - vref_source != ADCX140_MIC_BIAS_VREF_25V &&
> - vref_source != ADCX140_MIC_BIAS_VREF_1375V) {
> - dev_err(adcx140->dev, "Mic Bias source value is invalid\n");
> + if (vref_source != ADCX140_ADC_FSCALE_VREF_275V &&
> + vref_source != ADCX140_ADC_FSCALE_VREF_25V &&
> + vref_source != ADCX140_ADC_FSCALE_VREF_1375V) {
> + dev_err(adcx140->dev, "ADC full scale setting is
> invalid\n");
> return -EINVAL;
> }
>
> @@ -787,7 +787,7 @@ static int adcx140_codec_probe(struct snd_soc_component
> *component)
>
> ret = regmap_update_bits(adcx140->regmap, ADCX140_BIAS_CFG,
> ADCX140_MIC_BIAS_VAL_MSK |
> - ADCX140_MIC_BIAS_VREF_MSK, bias_source);
> + ADCX140_ADC_FSCALE_VREF_MSK, bias_source);
> if (ret)
> dev_err(adcx140->dev, "setting MIC bias failed %d\n", ret);
> out:
> diff --git a/sound/soc/codecs/tlv320adcx140.h
> b/sound/soc/codecs/tlv320adcx140.h
> index 6d055e55909e..adb9513900b1 100644
> --- a/sound/soc/codecs/tlv320adcx140.h
> +++ b/sound/soc/codecs/tlv320adcx140.h
> @@ -117,10 +117,10 @@
> #define ADCX140_MIC_BIAS_VAL_AVDD 6
> #define ADCX140_MIC_BIAS_VAL_MSK GENMASK(6, 4)
>
> -#define ADCX140_MIC_BIAS_VREF_275V 0
> -#define ADCX140_MIC_BIAS_VREF_25V 1
> -#define ADCX140_MIC_BIAS_VREF_1375V 2
> -#define ADCX140_MIC_BIAS_VREF_MSK GENMASK(1, 0)
> +#define ADCX140_ADC_FSCALE_VREF_275V 0
> +#define ADCX140_ADC_FSCALE_VREF_25V 1
> +#define ADCX140_ADC_FSCALE_VREF_1375V 2
> +#define ADCX140_ADC_FSCALE_VREF_MSK GENMASK(1, 0)
>
> #define ADCX140_PWR_CFG_BIAS_PDZ BIT(7)
> #define ADCX140_PWR_CFG_ADC_PDZ BIT(6)
> --
> 2.25.0
>
>
>
Looks good to me!
/Ricard
--
Ricard Wolf Wanderlof ricardw(at)axis.com
Axis Communications AB, Lund, Sweden www.axis.com
Phone +46 46 272 2016 Fax +46 46 13 61 30
Powered by blists - more mailing lists