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=-7.0 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS, URIBL_BLOCKED autolearn=unavailable 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 4C7D2C43381 for ; Sun, 31 Mar 2019 09:46:27 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 1A8772146F for ; Sun, 31 Mar 2019 09:46:27 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="l8WEV42m"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="wbu3ASz6" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1A8772146F Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=8QyT2IEgxejNWUlkp2wKsJcU9/OO7FM9QycF08MWaFs=; b=l8WEV42mk20pGu XK6Hi9I0Ga8fcr2vI9DtvjhbSpGfDZ7wMdJRYt/g4I5yMPM6gdfFlbYomrC9Zxt1pzCi0wzsMWfmZ U5LvA/c0NAzcktKAhMre+yCuLJFvVrd3vcScQKy/wooZAfQSOz8o5cWj/3sgcEe9mnc+4Lm+DyiXe +PyEFBIKqivkGMQtrK5nQkqH6jWWCj779sBPPj9L/EC4Cg3RzO2E/Ey3/zfadWa7jCqX8pLmsQpEj dQaMe4DyAIlZVjrTZkmwWwGztPDrs4ro53jh1oLj0Un/MzewWxGjBmWeidb94m12eEUG5WKOprPAo PQRF1BQeQkLrvA4avxig==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1hAX2d-0008Ci-5e; Sun, 31 Mar 2019 09:46:15 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1hAX2Y-0008CN-A2 for linux-arm-kernel@lists.infradead.org; Sun, 31 Mar 2019 09:46:12 +0000 Received: from archlinux (cpc91196-cmbg18-2-0-cust659.5-4.cable.virginm.net [81.96.234.148]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 3D6AE20850; Sun, 31 Mar 2019 09:46:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1554025569; bh=mvatAZ8EMowd44cb0Yyla/0+OwnoQ3rNaVBn3xAnz4c=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=wbu3ASz6Loab3F9fmV43p9D6pUTLuvgNJ/qs4QrwEiAS1luQCMlC4O46HG7gxLVSa ElGtUTUSQu9DAtfrTii8e2MU5pvtrjB7Drbptmoc48ke2U6TiD6htUQAd6O4y9J0rK CzCmEig2wHjUKH6tgQm4fLEKsL3JrTj4bX+cRvFA= Date: Sun, 31 Mar 2019 10:46:02 +0100 From: Jonathan Cameron To: Fabrice Gasnier Subject: Re: [PATCH] iio: adc: stm32-dfsdm: improve sampling frequency accuracy Message-ID: <20190331104602.209cb9a0@archlinux> In-Reply-To: <1553523841-16315-1-git-send-email-fabrice.gasnier@st.com> References: <1553523841-16315-1-git-send-email-fabrice.gasnier@st.com> X-Mailer: Claws Mail 3.17.3 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190331_024610_916124_3C17767A X-CRM114-Status: GOOD ( 17.10 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: lars@metafoo.de, olivier.moysan@st.com, alexandre.torgue@st.com, linux-iio@vger.kernel.org, pmeerw@pmeerw.net, arnaud.pouliquen@st.com, linux-kernel@vger.kernel.org, mcoquelin.stm32@gmail.com, knaack.h@gmx.de, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, 25 Mar 2019 15:24:01 +0100 Fabrice Gasnier wrote: > The sample frequency is driven using the oversampling ratio depending > on the SPI bus frequency. > Currently, oversampling ratio is computed by an entire division: > - spi_freq / sample_freq. This may result in inaccurate value. > Using DIV_ROUND_CLOSEST improves resulting sample frequency, which is > useful for audio that requests fixed rates (such as: 8, 16 or 32 kHz). > BTW, introduce new routine to re-factor sample frequency setting, and > move frequency accuracy message from warning to debug level. > > Signed-off-by: Fabrice Gasnier Would have preferred this as two patches as it would have made review easier by making it clear what the functional change is. Still that is mostly me just wanting an easy life ;) Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it. Thanks, Jonathan > --- > drivers/iio/adc/stm32-dfsdm-adc.c | 56 +++++++++++++++++++++------------------ > 1 file changed, 30 insertions(+), 26 deletions(-) > > diff --git a/drivers/iio/adc/stm32-dfsdm-adc.c b/drivers/iio/adc/stm32-dfsdm-adc.c > index 531ca7e..051561c 100644 > --- a/drivers/iio/adc/stm32-dfsdm-adc.c > +++ b/drivers/iio/adc/stm32-dfsdm-adc.c > @@ -558,13 +558,38 @@ static ssize_t dfsdm_adc_audio_get_spiclk(struct iio_dev *indio_dev, > return snprintf(buf, PAGE_SIZE, "%d\n", adc->spi_freq); > } > > +static int dfsdm_adc_set_samp_freq(struct iio_dev *indio_dev, > + unsigned int sample_freq, > + unsigned int spi_freq) > +{ > + struct stm32_dfsdm_adc *adc = iio_priv(indio_dev); > + struct stm32_dfsdm_filter *fl = &adc->dfsdm->fl_list[adc->fl_id]; > + unsigned int oversamp; > + int ret; > + > + oversamp = DIV_ROUND_CLOSEST(spi_freq, sample_freq); > + if (spi_freq % sample_freq) > + dev_dbg(&indio_dev->dev, > + "Rate not accurate. requested (%u), actual (%u)\n", > + sample_freq, spi_freq / oversamp); > + > + ret = stm32_dfsdm_set_osrs(fl, 0, oversamp); > + if (ret < 0) { > + dev_err(&indio_dev->dev, "No filter parameters that match!\n"); > + return ret; > + } > + adc->sample_freq = spi_freq / oversamp; > + adc->oversamp = oversamp; > + > + return 0; > +} > + > static ssize_t dfsdm_adc_audio_set_spiclk(struct iio_dev *indio_dev, > uintptr_t priv, > const struct iio_chan_spec *chan, > const char *buf, size_t len) > { > struct stm32_dfsdm_adc *adc = iio_priv(indio_dev); > - struct stm32_dfsdm_filter *fl = &adc->dfsdm->fl_list[adc->fl_id]; > struct stm32_dfsdm_channel *ch = &adc->dfsdm->ch_list[chan->channel]; > unsigned int sample_freq = adc->sample_freq; > unsigned int spi_freq; > @@ -583,17 +608,9 @@ static ssize_t dfsdm_adc_audio_set_spiclk(struct iio_dev *indio_dev, > return -EINVAL; > > if (sample_freq) { > - if (spi_freq % sample_freq) > - dev_warn(&indio_dev->dev, > - "Sampling rate not accurate (%d)\n", > - spi_freq / (spi_freq / sample_freq)); > - > - ret = stm32_dfsdm_set_osrs(fl, 0, (spi_freq / sample_freq)); > - if (ret < 0) { > - dev_err(&indio_dev->dev, > - "No filter parameters that match!\n"); > + ret = dfsdm_adc_set_samp_freq(indio_dev, sample_freq, spi_freq); > + if (ret < 0) > return ret; > - } > } > adc->spi_freq = spi_freq; > > @@ -1068,22 +1085,9 @@ static int stm32_dfsdm_write_raw(struct iio_dev *indio_dev, > spi_freq = adc->spi_freq; > } > > - if (spi_freq % val) > - dev_warn(&indio_dev->dev, > - "Sampling rate not accurate (%d)\n", > - spi_freq / (spi_freq / val)); > - > - ret = stm32_dfsdm_set_osrs(fl, 0, (spi_freq / val)); > - if (ret < 0) { > - dev_err(&indio_dev->dev, > - "Not able to find parameter that match!\n"); > - iio_device_release_direct_mode(indio_dev); > - return ret; > - } > - adc->sample_freq = val; > + ret = dfsdm_adc_set_samp_freq(indio_dev, val, spi_freq); > iio_device_release_direct_mode(indio_dev); > - > - return 0; > + return ret; > } > > return -EINVAL; _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel