From patchwork Fri Nov 11 11:26:53 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: =?utf-8?q?Mitja_=C5=A0pes?= X-Patchwork-Id: 13040174 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id C21F2C433FE for ; Fri, 11 Nov 2022 11:28:02 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232506AbiKKL2B (ORCPT ); Fri, 11 Nov 2022 06:28:01 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39162 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231681AbiKKL2B (ORCPT ); Fri, 11 Nov 2022 06:28:01 -0500 Received: from dhl.lxnav.com (dhl.lxnav.com [168.119.248.164]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4292961BAF for ; Fri, 11 Nov 2022 03:28:00 -0800 (PST) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 45F504070D; Fri, 11 Nov 2022 12:27:57 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=lxnav.com; s=dhl; t=1668166078; h=from:subject:date:message-id:to:mime-version: content-transfer-encoding:in-reply-to:references; bh=9BmkVh6jIh/jyhyFFqsPkAGeVwmF2cJMvet6/U1c940=; b=cigeI+HJRe6JIJ2uzZTjx0+QwIL3d2NcFzzLCU7g4uLi1ePXFF50sMtilsU0UpDeOOdkkG gbSo/GwMuqGRzKd7/fAL16JNWFIo2SYjq3eqk7JoTsUFNg4Ey4AL4xdpU1ca5jEfnP4FCx NoVJ/6F/FTicNLbic8Y45Hs7tuc3kzeRcr99xHilkL5FG7Ay0rzL/5a0rnM1OV427OWUjT EL9Zu2n6nlubE3JO/eAxbmwuY67N3o5s84c+m/ECFbu/PmD/Q+HMiJ0kQx/ZtH7NnSGjSu nyhFeeup7/c0KFjJ04iz6pPJ/93BpZ3P0kCvNtv2ZQYcCrFhRlRC+HUNn9ZHfg== From: Mitja Spes To: Jonathan Cameron , Lars-Peter Clausen , Mitja Spes , Angelo Compagnucci , linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 1/4] iio: adc: mcp3422: fix scale read bug Date: Fri, 11 Nov 2022 12:26:53 +0100 Message-Id: <20221111112657.1521307-2-mitja@lxnav.com> In-Reply-To: <20221111112657.1521307-1-mitja@lxnav.com> References: <20221111112657.1521307-1-mitja@lxnav.com> MIME-Version: 1.0 X-Last-TLS-Session-Version: TLSv1.3 Precedence: bulk List-ID: X-Mailing-List: linux-iio@vger.kernel.org Scale was returned for currently active channel instead of the specified channel. Signed-off-by: Mitja Spes Fixes: 07914c84ba30 ("iio: adc: Add driver for Microchip MCP3422/3/4 high resolution ADC") --- drivers/iio/adc/mcp3422.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/iio/adc/mcp3422.c b/drivers/iio/adc/mcp3422.c index da353dcb1e9d..3d53de300c89 100644 --- a/drivers/iio/adc/mcp3422.c +++ b/drivers/iio/adc/mcp3422.c @@ -164,8 +164,9 @@ static int mcp3422_read_raw(struct iio_dev *iio, struct mcp3422 *adc = iio_priv(iio); int err; + u8 req_channel = channel->channel; u8 sample_rate = MCP3422_SAMPLE_RATE(adc->config); - u8 pga = MCP3422_PGA(adc->config); + u8 pga = adc->pga[req_channel]; switch (mask) { case IIO_CHAN_INFO_RAW: @@ -175,7 +176,6 @@ static int mcp3422_read_raw(struct iio_dev *iio, return IIO_VAL_INT; case IIO_CHAN_INFO_SCALE: - *val1 = 0; *val2 = mcp3422_scales[sample_rate][pga]; return IIO_VAL_INT_PLUS_NANO; From patchwork Fri Nov 11 11:26:54 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: =?utf-8?q?Mitja_=C5=A0pes?= X-Patchwork-Id: 13040175 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id C70EFC433FE for ; Fri, 11 Nov 2022 11:28:12 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233455AbiKKL2J (ORCPT ); Fri, 11 Nov 2022 06:28:09 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39228 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233192AbiKKL2E (ORCPT ); Fri, 11 Nov 2022 06:28:04 -0500 Received: from dhl.lxnav.com (dhl.lxnav.com [168.119.248.164]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9C8B86A771 for ; Fri, 11 Nov 2022 03:28:03 -0800 (PST) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 0139D40767; Fri, 11 Nov 2022 12:28:00 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=lxnav.com; s=dhl; t=1668166082; h=from:subject:date:message-id:to:mime-version: content-transfer-encoding:in-reply-to:references; bh=PomnhdIpzQ+pggSr9FKY16Cho/dIxrxhSMt/zS5cSLo=; b=P+PXfovB1fwLFgoWmdrUGW8VQwclQ+LSWBa4D2wnCDpXM0Q2cWskXfTIJlLnZw9fj4G0uH 8Bs30vVh9FN0fr39UV7Ro8bcdXRbEJorK02hJYtKxTOgVbe4wmymuNNDTrG0FI45ghU6VT UN5NDnNyeHRa11zCg3q1pbPgDULfZxTkeRTNP7hHlXDivBcjLif6PzTDmnvsCZRaGNB/AI LP5xcrsBh/QJF8OyYiTCakD+F/iqDVmg32FA01xUO5EoRwWB92dy8/2RamFjS5l8/l6iuA XIM/hY04Hl+F/82Z0jSW2TGNcfw/bX7tjsrttol909mXbQ3uvYbsgPWe8PXrsA== From: Mitja Spes To: Jonathan Cameron , Lars-Peter Clausen , Mitja Spes , Angelo Compagnucci , linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 2/4] iio: adc: mcp3422: allow setting gain and sampling per channel Date: Fri, 11 Nov 2022 12:26:54 +0100 Message-Id: <20221111112657.1521307-3-mitja@lxnav.com> In-Reply-To: <20221111112657.1521307-1-mitja@lxnav.com> References: <20221111112657.1521307-1-mitja@lxnav.com> MIME-Version: 1.0 X-Last-TLS-Session-Version: TLSv1.3 Precedence: bulk List-ID: X-Mailing-List: linux-iio@vger.kernel.org General improvements: - allow setting gain and sampling per channel - setting scale can also set sampling rate (combined setting) - use per channel config setting - do not update mcp register on setting write (we might be reading it...) instead it's updated on next value read - output all scale values (sample rates x gain) Signed-off-by: Mitja Spes --- drivers/iio/adc/mcp3422.c | 119 ++++++++++++++++++++------------------ 1 file changed, 64 insertions(+), 55 deletions(-) diff --git a/drivers/iio/adc/mcp3422.c b/drivers/iio/adc/mcp3422.c index 3d53de300c89..cfb629b964af 100644 --- a/drivers/iio/adc/mcp3422.c +++ b/drivers/iio/adc/mcp3422.c @@ -28,14 +28,19 @@ #define MCP3422_CHANNEL_MASK 0x60 #define MCP3422_PGA_MASK 0x03 #define MCP3422_SRATE_MASK 0x0C -#define MCP3422_SRATE_240 0x0 -#define MCP3422_SRATE_60 0x1 -#define MCP3422_SRATE_15 0x2 -#define MCP3422_SRATE_3 0x3 -#define MCP3422_PGA_1 0 -#define MCP3422_PGA_2 1 -#define MCP3422_PGA_4 2 -#define MCP3422_PGA_8 3 + +#define MCP3422_SRATE_240 0 +#define MCP3422_SRATE_60 1 +#define MCP3422_SRATE_15 2 +#define MCP3422_SRATE_3 3 +#define MCP3422_SRATE_COUNT 4 + +#define MCP3422_PGA_1 0 +#define MCP3422_PGA_2 1 +#define MCP3422_PGA_4 2 +#define MCP3422_PGA_8 3 +#define MCP3422_PGA_COUNT 4 + #define MCP3422_CONT_SAMPLING 0x10 #define MCP3422_CHANNEL(config) (((config) & MCP3422_CHANNEL_MASK) >> 5) @@ -52,32 +57,32 @@ .indexed = 1, \ .channel = _index, \ .info_mask_separate = BIT(IIO_CHAN_INFO_RAW) \ - | BIT(IIO_CHAN_INFO_SCALE), \ - .info_mask_shared_by_type = BIT(IIO_CHAN_INFO_SAMP_FREQ), \ + | BIT(IIO_CHAN_INFO_SCALE) \ + | BIT(IIO_CHAN_INFO_SAMP_FREQ), \ } -static const int mcp3422_scales[4][4] = { +static const int mcp3422_scales[MCP3422_SRATE_COUNT][MCP3422_PGA_COUNT] = { { 1000000, 500000, 250000, 125000 }, { 250000, 125000, 62500, 31250 }, { 62500, 31250, 15625, 7812 }, { 15625, 7812, 3906, 1953 } }; /* Constant msleep times for data acquisitions */ -static const int mcp3422_read_times[4] = { +static const int mcp3422_read_times[MCP3422_SRATE_COUNT] = { [MCP3422_SRATE_240] = 1000 / 240, [MCP3422_SRATE_60] = 1000 / 60, [MCP3422_SRATE_15] = 1000 / 15, [MCP3422_SRATE_3] = 1000 / 3 }; /* sample rates to integer conversion table */ -static const int mcp3422_sample_rates[4] = { +static const int mcp3422_sample_rates[MCP3422_SRATE_COUNT] = { [MCP3422_SRATE_240] = 240, [MCP3422_SRATE_60] = 60, [MCP3422_SRATE_15] = 15, [MCP3422_SRATE_3] = 3 }; /* sample rates to sign extension table */ -static const int mcp3422_sign_extend[4] = { +static const int mcp3422_sign_extend[MCP3422_SRATE_COUNT] = { [MCP3422_SRATE_240] = 11, [MCP3422_SRATE_60] = 13, [MCP3422_SRATE_15] = 15, @@ -87,8 +92,8 @@ static const int mcp3422_sign_extend[4] = { struct mcp3422 { struct i2c_client *i2c; u8 id; - u8 config; - u8 pga[4]; + u8 active_config; // config currently set on mcp + u8 ch_config[4]; // per channel config struct mutex lock; }; @@ -98,7 +103,7 @@ static int mcp3422_update_config(struct mcp3422 *adc, u8 newconfig) ret = i2c_master_send(adc->i2c, &newconfig, 1); if (ret > 0) { - adc->config = newconfig; + adc->active_config = newconfig; ret = 0; } @@ -108,7 +113,7 @@ static int mcp3422_update_config(struct mcp3422 *adc, u8 newconfig) static int mcp3422_read(struct mcp3422 *adc, int *value, u8 *config) { int ret = 0; - u8 sample_rate = MCP3422_SAMPLE_RATE(adc->config); + u8 sample_rate = MCP3422_SAMPLE_RATE(adc->active_config); u8 buf[4] = {0, 0, 0, 0}; u32 temp; @@ -136,18 +141,13 @@ static int mcp3422_read_channel(struct mcp3422 *adc, mutex_lock(&adc->lock); - if (req_channel != MCP3422_CHANNEL(adc->config)) { - config = adc->config; - config &= ~MCP3422_CHANNEL_MASK; - config |= MCP3422_CHANNEL_VALUE(req_channel); - config &= ~MCP3422_PGA_MASK; - config |= MCP3422_PGA_VALUE(adc->pga[req_channel]); - ret = mcp3422_update_config(adc, config); + if (adc->ch_config[req_channel] != adc->active_config) { + ret = mcp3422_update_config(adc, adc->ch_config[req_channel]); if (ret < 0) { mutex_unlock(&adc->lock); return ret; } - msleep(mcp3422_read_times[MCP3422_SAMPLE_RATE(adc->config)]); + msleep(mcp3422_read_times[MCP3422_SAMPLE_RATE(adc->active_config)]); } ret = mcp3422_read(adc, value, &config); @@ -164,9 +164,9 @@ static int mcp3422_read_raw(struct iio_dev *iio, struct mcp3422 *adc = iio_priv(iio); int err; - u8 req_channel = channel->channel; - u8 sample_rate = MCP3422_SAMPLE_RATE(adc->config); - u8 pga = adc->pga[req_channel]; + u8 config = adc->ch_config[channel->channel]; + u8 sample_rate = MCP3422_SAMPLE_RATE(config); + u8 pga = MCP3422_PGA(config); switch (mask) { case IIO_CHAN_INFO_RAW: @@ -181,7 +181,7 @@ static int mcp3422_read_raw(struct iio_dev *iio, return IIO_VAL_INT_PLUS_NANO; case IIO_CHAN_INFO_SAMP_FREQ: - *val1 = mcp3422_sample_rates[MCP3422_SAMPLE_RATE(adc->config)]; + *val1 = mcp3422_sample_rates[sample_rate]; return IIO_VAL_INT; default: @@ -197,26 +197,25 @@ static int mcp3422_write_raw(struct iio_dev *iio, { struct mcp3422 *adc = iio_priv(iio); u8 temp; - u8 config = adc->config; u8 req_channel = channel->channel; - u8 sample_rate = MCP3422_SAMPLE_RATE(config); - u8 i; + u8 config = adc->ch_config[req_channel]; + u8 i, j; switch (mask) { case IIO_CHAN_INFO_SCALE: if (val1 != 0) return -EINVAL; - for (i = 0; i < ARRAY_SIZE(mcp3422_scales[0]); i++) { - if (val2 == mcp3422_scales[sample_rate][i]) { - adc->pga[req_channel] = i; - - config &= ~MCP3422_CHANNEL_MASK; - config |= MCP3422_CHANNEL_VALUE(req_channel); - config &= ~MCP3422_PGA_MASK; - config |= MCP3422_PGA_VALUE(adc->pga[req_channel]); - - return mcp3422_update_config(adc, config); + for (j = 0; j < MCP3422_SRATE_COUNT; j++) { + for (i = 0; i < MCP3422_PGA_COUNT; i++) { + if (val2 == mcp3422_scales[j][i]) { + config &= ~MCP3422_PGA_MASK; + config |= MCP3422_PGA_VALUE(i); + config &= ~MCP3422_SRATE_MASK; + config |= MCP3422_SAMPLE_RATE_VALUE(j); + adc->ch_config[req_channel] = config; + return 0; + } } } return -EINVAL; @@ -241,12 +240,10 @@ static int mcp3422_write_raw(struct iio_dev *iio, return -EINVAL; } - config &= ~MCP3422_CHANNEL_MASK; - config |= MCP3422_CHANNEL_VALUE(req_channel); config &= ~MCP3422_SRATE_MASK; config |= MCP3422_SAMPLE_RATE_VALUE(temp); - - return mcp3422_update_config(adc, config); + adc->ch_config[req_channel] = config; + return 0; default: break; @@ -282,14 +279,18 @@ static ssize_t mcp3422_show_samp_freqs(struct device *dev, static ssize_t mcp3422_show_scales(struct device *dev, struct device_attribute *attr, char *buf) { - struct mcp3422 *adc = iio_priv(dev_to_iio_dev(dev)); - u8 sample_rate = MCP3422_SAMPLE_RATE(adc->config); - - return sprintf(buf, "0.%09u 0.%09u 0.%09u 0.%09u\n", - mcp3422_scales[sample_rate][0], - mcp3422_scales[sample_rate][1], - mcp3422_scales[sample_rate][2], - mcp3422_scales[sample_rate][3]); + ssize_t count = 0; + int i; + + for (i = 0; i < MCP3422_SRATE_COUNT; i++) { + count += sprintf(buf + count, "0.%09u 0.%09u 0.%09u 0.%09u%s", + mcp3422_scales[i][0], + mcp3422_scales[i][1], + mcp3422_scales[i][2], + mcp3422_scales[i][3], + (i < MCP3422_SRATE_COUNT - 1 ? " " : "\n")); + } + return count; } static IIO_DEVICE_ATTR(sampling_frequency_available, S_IRUGO, @@ -336,6 +337,7 @@ static int mcp3422_probe(struct i2c_client *client, struct iio_dev *indio_dev; struct mcp3422 *adc; int err; + int i; u8 config; if (!i2c_check_functionality(client->adapter, I2C_FUNC_I2C)) @@ -376,6 +378,13 @@ static int mcp3422_probe(struct i2c_client *client, } /* meaningful default configuration */ + for (i = 0; i < 4; i++) { + adc->ch_config[i] = (MCP3422_CONT_SAMPLING + | MCP3422_CHANNEL_VALUE(i) + | MCP3422_PGA_VALUE(MCP3422_PGA_1) + | MCP3422_SAMPLE_RATE_VALUE(MCP3422_SRATE_240)); + } + config = (MCP3422_CONT_SAMPLING | MCP3422_CHANNEL_VALUE(0) | MCP3422_PGA_VALUE(MCP3422_PGA_1) From patchwork Fri Nov 11 11:26:55 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: =?utf-8?q?Mitja_=C5=A0pes?= X-Patchwork-Id: 13040176 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 55364C4332F for ; Fri, 11 Nov 2022 11:28:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233446AbiKKL2T (ORCPT ); Fri, 11 Nov 2022 06:28:19 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39242 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232855AbiKKL2Q (ORCPT ); Fri, 11 Nov 2022 06:28:16 -0500 Received: from dhl.lxnav.com (dhl.lxnav.com [168.119.248.164]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2CB61716E8 for ; Fri, 11 Nov 2022 03:28:07 -0800 (PST) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 1E82040768; Fri, 11 Nov 2022 12:28:04 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=lxnav.com; s=dhl; t=1668166085; h=from:subject:date:message-id:to:mime-version: content-transfer-encoding:in-reply-to:references; bh=NqvSjeOULQZbj3EUjH7dknKYQjo14gTb8HTU0Fd3y2A=; b=hYtyX2wIlbpRse+AYLKuwh4QMTwz+d6nnySOhp5qQRVLJviQ613mUP4sJZtegxv8cMGmte nhI8wp+T8NjKNqbOm9W/KRQIwkFC94s6Rv8aXZlmOBJFoxE4Kin3W/Sr7gva9ohi5J043I ZILyXvw1M71C/kaI/gO/KksqNXI5gt1qPD9vdZqkw655oQxRs4Yk0z0ZHk0nLw2hD9nJB7 fCsRc4Lh/uj00LPqmEJVFrrRI9rjLMMO30Ecccdha44/3JURIwe5PrIOk1QrRAiYk6pCm6 1qiKhmLqrLduxacyuSXsE9KikRzdlFFeZTCw+nhw/j3b9hzw3o43lJUgQ/tEkw== From: Mitja Spes To: Jonathan Cameron , Lars-Peter Clausen , Mitja Spes , Angelo Compagnucci , linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 3/4] iio: adc: mcp3422: add hardware gain attribute Date: Fri, 11 Nov 2022 12:26:55 +0100 Message-Id: <20221111112657.1521307-4-mitja@lxnav.com> In-Reply-To: <20221111112657.1521307-1-mitja@lxnav.com> References: <20221111112657.1521307-1-mitja@lxnav.com> MIME-Version: 1.0 X-Last-TLS-Session-Version: TLSv1.3 Precedence: bulk List-ID: X-Mailing-List: linux-iio@vger.kernel.org Allows setting gain separately from scale. Signed-off-by: Mitja Spes --- drivers/iio/adc/mcp3422.c | 32 +++++++++++++++++++++++++++++++- 1 file changed, 31 insertions(+), 1 deletion(-) diff --git a/drivers/iio/adc/mcp3422.c b/drivers/iio/adc/mcp3422.c index cfb629b964af..eef35fb2fc22 100644 --- a/drivers/iio/adc/mcp3422.c +++ b/drivers/iio/adc/mcp3422.c @@ -58,7 +58,8 @@ .channel = _index, \ .info_mask_separate = BIT(IIO_CHAN_INFO_RAW) \ | BIT(IIO_CHAN_INFO_SCALE) \ - | BIT(IIO_CHAN_INFO_SAMP_FREQ), \ + | BIT(IIO_CHAN_INFO_SAMP_FREQ) \ + | BIT(IIO_CHAN_INFO_HARDWAREGAIN), \ } static const int mcp3422_scales[MCP3422_SRATE_COUNT][MCP3422_PGA_COUNT] = { @@ -184,6 +185,10 @@ static int mcp3422_read_raw(struct iio_dev *iio, *val1 = mcp3422_sample_rates[sample_rate]; return IIO_VAL_INT; + case IIO_CHAN_INFO_HARDWAREGAIN: + *val1 = (1 << pga); + return IIO_VAL_INT; + default: break; } @@ -245,6 +250,29 @@ static int mcp3422_write_raw(struct iio_dev *iio, adc->ch_config[req_channel] = config; return 0; + case IIO_CHAN_INFO_HARDWAREGAIN: + switch (val1) { + case 1: + temp = MCP3422_PGA_1; + break; + case 2: + temp = MCP3422_PGA_2; + break; + case 4: + temp = MCP3422_PGA_4; + break; + case 8: + temp = MCP3422_PGA_8; + break; + default: + return -EINVAL; + } + + config &= ~MCP3422_PGA_MASK; + config |= MCP3422_PGA_VALUE(temp); + adc->ch_config[req_channel] = config; + return 0; + default: break; } @@ -260,6 +288,8 @@ static int mcp3422_write_raw_get_fmt(struct iio_dev *indio_dev, return IIO_VAL_INT_PLUS_NANO; case IIO_CHAN_INFO_SAMP_FREQ: return IIO_VAL_INT_PLUS_MICRO; + case IIO_CHAN_INFO_HARDWAREGAIN: + return IIO_VAL_INT_PLUS_MICRO; default: return -EINVAL; } From patchwork Fri Nov 11 11:26:56 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: =?utf-8?q?Mitja_=C5=A0pes?= X-Patchwork-Id: 13040177 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2D496C43219 for ; Fri, 11 Nov 2022 11:28:33 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233756AbiKKL23 (ORCPT ); Fri, 11 Nov 2022 06:28:29 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39580 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233445AbiKKL2S (ORCPT ); Fri, 11 Nov 2022 06:28:18 -0500 Received: from dhl.lxnav.com (dhl.lxnav.com [168.119.248.164]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 29F3D7018C for ; Fri, 11 Nov 2022 03:28:10 -0800 (PST) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 982964070A; Fri, 11 Nov 2022 12:28:07 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=lxnav.com; s=dhl; t=1668166088; h=from:subject:date:message-id:to:mime-version: content-transfer-encoding:in-reply-to:references; bh=sduuwDJ1cpdPo1tG/vMuBFFxLHUHN1zt6Q+P4Jawaig=; b=R25EG2cP4ELm7lYu+1KM6DyIj0ArFnNkdIqozN6tS6rHE7IFVOzk8vSoUTX7eDuwY71eTo PGJqHjaJc7bD4rQnT5gPGbAhDEAph/MMwIC70Hq4A94Y/vBTyE/raLT6xR9zbA1Zd3R7KH ELJfQrO6N8R93ZeIrjhjRlOUjlUs3zs5Q/HMf5a00aMGv292ercvOUMYrSR/evWYJaupYQ iwo9pP7Zx8Gdz00TCYvbueWxtHomDdTHaIFgehqyvJ8MNMmWeX5ORfwnP53pzvZh+bB3yf 1lb7zI5NgbGLuXpORXZMXI+tf1XZXCO9bZiNst+yOxkrQl/A8ITZqA/IgkKRcg== From: Mitja Spes To: Jonathan Cameron , Lars-Peter Clausen , Mitja Spes , Angelo Compagnucci , linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 4/4] iio: adc: mcp3422: reduce sleep for fast sampling rates Date: Fri, 11 Nov 2022 12:26:56 +0100 Message-Id: <20221111112657.1521307-5-mitja@lxnav.com> In-Reply-To: <20221111112657.1521307-1-mitja@lxnav.com> References: <20221111112657.1521307-1-mitja@lxnav.com> MIME-Version: 1.0 X-Last-TLS-Session-Version: TLSv1.3 Precedence: bulk List-ID: X-Mailing-List: linux-iio@vger.kernel.org - reduced sleep time for 240 & 60 sps rates - minor roundup fix for sleep times Signed-off-by: Mitja Spes --- drivers/iio/adc/mcp3422.c | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/drivers/iio/adc/mcp3422.c b/drivers/iio/adc/mcp3422.c index eef35fb2fc22..dbcc8fe91aaa 100644 --- a/drivers/iio/adc/mcp3422.c +++ b/drivers/iio/adc/mcp3422.c @@ -70,10 +70,11 @@ static const int mcp3422_scales[MCP3422_SRATE_COUNT][MCP3422_PGA_COUNT] = { /* Constant msleep times for data acquisitions */ static const int mcp3422_read_times[MCP3422_SRATE_COUNT] = { - [MCP3422_SRATE_240] = 1000 / 240, - [MCP3422_SRATE_60] = 1000 / 60, - [MCP3422_SRATE_15] = 1000 / 15, - [MCP3422_SRATE_3] = 1000 / 3 }; + [MCP3422_SRATE_240] = DIV_ROUND_UP(1000, 240), + [MCP3422_SRATE_60] = DIV_ROUND_UP(1000, 60), + [MCP3422_SRATE_15] = DIV_ROUND_UP(1000, 15), + [MCP3422_SRATE_3] = (100000 + 375 - 100) / 375 /* real rate is 3.75 sps */ +}; /* sample rates to integer conversion table */ static const int mcp3422_sample_rates[MCP3422_SRATE_COUNT] = { @@ -137,6 +138,7 @@ static int mcp3422_read_channel(struct mcp3422 *adc, struct iio_chan_spec const *channel, int *value) { int ret; + int sleep_duration; u8 config; u8 req_channel = channel->channel; @@ -148,7 +150,11 @@ static int mcp3422_read_channel(struct mcp3422 *adc, mutex_unlock(&adc->lock); return ret; } - msleep(mcp3422_read_times[MCP3422_SAMPLE_RATE(adc->active_config)]); + sleep_duration = mcp3422_read_times[MCP3422_SAMPLE_RATE(adc->active_config)]; + if (sleep_duration < 20) + usleep_range(sleep_duration * 1000, sleep_duration * 1300); + else + msleep(sleep_duration); } ret = mcp3422_read(adc, value, &config);