From nobody Wed Apr 8 01:35:38 2026 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 3EDADC433FE for ; Fri, 21 Oct 2022 13:59:52 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231252AbiJUN7t (ORCPT ); Fri, 21 Oct 2022 09:59:49 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45458 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231314AbiJUN7m (ORCPT ); Fri, 21 Oct 2022 09:59:42 -0400 Received: from dhl.lxnav.com (dhl.lxnav.com [IPv6:2a01:4f8:c010:2f07::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9B9FBE0B5; Fri, 21 Oct 2022 06:59:21 -0700 (PDT) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 9180640760; Fri, 21 Oct 2022 15:59:18 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=lxnav.com; s=dhl; t=1666360760; h=from:subject:date:message-id:to:cc:mime-version: content-transfer-encoding:in-reply-to:references; bh=t4Yaw2ZLuxRrXeXgMKdBXuSmh/kxDmx/nG04x9JTVpY=; b=ZBuPJEfVPfXW7AaTrMuF4f6GXnU756jb7RTlA09n3h8zdizSwBHkgsDywenn57a6312jnK ZkuAwGQL6glqxbg99PqoG1naqzXn74ZD3nTEeLtu4PIvqS4BEzraNF5vGIpzbjDERjlV4K pQeCLXrnVAIKKFFOuSm+w2VlXb4iC2pM5cxandiEMcwhZjW6/v/n81sQfl68mBm/NOKOK8 COXpzbJwaQyRDxdhBozq0FCqt1gV0MLg+ThkXevkBcgEaqcWk0PNgCgUu6Cavcr3RrTt6a uV8WYWFvv1OvNEwI0T/rqul83v4Zon+Uyqq8fyBF7iAxX1nWkpiaZcDPcSVvPA== From: Mitja Spes Cc: Mitja Spes , Jonathan Cameron , Lars-Peter Clausen , Rob Herring , Krzysztof Kozlowski , Andy Shevchenko , Lee Jones , =?UTF-8?q?J=C3=A9r=C3=B4me=20Pouiller?= , =?UTF-8?q?Uwe=20Kleine-K=C3=B6nig?= , Tomasz Duszynski , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 1/3] iio: pressure: ms5611: fixed value compensation bug Date: Fri, 21 Oct 2022 15:58:20 +0200 Message-Id: <20221021135827.1444793-2-mitja@lxnav.com> In-Reply-To: <20221021135827.1444793-1-mitja@lxnav.com> References: <20221021135827.1444793-1-mitja@lxnav.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Last-TLS-Session-Version: TLSv1.3 To: unlisted-recipients:; (no To-header on input) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" When using multiple instances of this driver the compensation PROM was overwritten by the last initialized sensor. Now each sensor has own PROM storage. Signed-off-by: Mitja Spes --- drivers/iio/pressure/ms5611.h | 12 +++---- drivers/iio/pressure/ms5611_core.c | 51 ++++++++++++++++-------------- 2 files changed, 31 insertions(+), 32 deletions(-) diff --git a/drivers/iio/pressure/ms5611.h b/drivers/iio/pressure/ms5611.h index cbc9349c342a..550b75b7186f 100644 --- a/drivers/iio/pressure/ms5611.h +++ b/drivers/iio/pressure/ms5611.h @@ -25,13 +25,6 @@ enum { MS5607, }; =20 -struct ms5611_chip_info { - u16 prom[MS5611_PROM_WORDS_NB]; - - int (*temp_and_pressure_compensate)(struct ms5611_chip_info *chip_info, - s32 *temp, s32 *pressure); -}; - /* * OverSampling Rate descriptor. * Warning: cmd MUST be kept aligned on a word boundary (see @@ -50,12 +43,15 @@ struct ms5611_state { const struct ms5611_osr *pressure_osr; const struct ms5611_osr *temp_osr; =20 + u16 prom[MS5611_PROM_WORDS_NB]; + int (*reset)(struct ms5611_state *st); int (*read_prom_word)(struct ms5611_state *st, int index, u16 *word); int (*read_adc_temp_and_pressure)(struct ms5611_state *st, s32 *temp, s32 *pressure); =20 - struct ms5611_chip_info *chip_info; + int (*compensate_temp_and_pressure)(struct ms5611_state *st, s32 *temp, + s32 *pressure); struct regulator *vdd; }; =20 diff --git a/drivers/iio/pressure/ms5611_core.c b/drivers/iio/pressure/ms56= 11_core.c index 717521de66c4..c564a1d6cafe 100644 --- a/drivers/iio/pressure/ms5611_core.c +++ b/drivers/iio/pressure/ms5611_core.c @@ -85,7 +85,7 @@ static int ms5611_read_prom(struct iio_dev *indio_dev) struct ms5611_state *st =3D iio_priv(indio_dev); =20 for (i =3D 0; i < MS5611_PROM_WORDS_NB; i++) { - ret =3D st->read_prom_word(st, i, &st->chip_info->prom[i]); + ret =3D st->read_prom_word(st, i, &st->prom[i]); if (ret < 0) { dev_err(&indio_dev->dev, "failed to read prom at %d\n", i); @@ -93,7 +93,7 @@ static int ms5611_read_prom(struct iio_dev *indio_dev) } } =20 - if (!ms5611_prom_is_valid(st->chip_info->prom, MS5611_PROM_WORDS_NB)) { + if (!ms5611_prom_is_valid(st->prom, MS5611_PROM_WORDS_NB)) { dev_err(&indio_dev->dev, "PROM integrity check failed\n"); return -ENODEV; } @@ -114,21 +114,20 @@ static int ms5611_read_temp_and_pressure(struct iio_d= ev *indio_dev, return ret; } =20 - return st->chip_info->temp_and_pressure_compensate(st->chip_info, - temp, pressure); + return st->compensate_temp_and_pressure(st, temp, pressure); } =20 -static int ms5611_temp_and_pressure_compensate(struct ms5611_chip_info *ch= ip_info, +static int ms5611_temp_and_pressure_compensate(struct ms5611_state *st, s32 *temp, s32 *pressure) { s32 t =3D *temp, p =3D *pressure; s64 off, sens, dt; =20 - dt =3D t - (chip_info->prom[5] << 8); - off =3D ((s64)chip_info->prom[2] << 16) + ((chip_info->prom[4] * dt) >> 7= ); - sens =3D ((s64)chip_info->prom[1] << 15) + ((chip_info->prom[3] * dt) >> = 8); + dt =3D t - (st->prom[5] << 8); + off =3D ((s64)st->prom[2] << 16) + ((st->prom[4] * dt) >> 7); + sens =3D ((s64)st->prom[1] << 15) + ((st->prom[3] * dt) >> 8); =20 - t =3D 2000 + ((chip_info->prom[6] * dt) >> 23); + t =3D 2000 + ((st->prom[6] * dt) >> 23); if (t < 2000) { s64 off2, sens2, t2; =20 @@ -154,17 +153,17 @@ static int ms5611_temp_and_pressure_compensate(struct= ms5611_chip_info *chip_inf return 0; } =20 -static int ms5607_temp_and_pressure_compensate(struct ms5611_chip_info *ch= ip_info, +static int ms5607_temp_and_pressure_compensate(struct ms5611_state *st, s32 *temp, s32 *pressure) { s32 t =3D *temp, p =3D *pressure; s64 off, sens, dt; =20 - dt =3D t - (chip_info->prom[5] << 8); - off =3D ((s64)chip_info->prom[2] << 17) + ((chip_info->prom[4] * dt) >> 6= ); - sens =3D ((s64)chip_info->prom[1] << 16) + ((chip_info->prom[3] * dt) >> = 7); + dt =3D t - (st->prom[5] << 8); + off =3D ((s64)st->prom[2] << 17) + ((st->prom[4] * dt) >> 6); + sens =3D ((s64)st->prom[1] << 16) + ((st->prom[3] * dt) >> 7); =20 - t =3D 2000 + ((chip_info->prom[6] * dt) >> 23); + t =3D 2000 + ((st->prom[6] * dt) >> 23); if (t < 2000) { s64 off2, sens2, t2, tmp; =20 @@ -342,15 +341,6 @@ static int ms5611_write_raw(struct iio_dev *indio_dev, =20 static const unsigned long ms5611_scan_masks[] =3D {0x3, 0}; =20 -static struct ms5611_chip_info chip_info_tbl[] =3D { - [MS5611] =3D { - .temp_and_pressure_compensate =3D ms5611_temp_and_pressure_compensate, - }, - [MS5607] =3D { - .temp_and_pressure_compensate =3D ms5607_temp_and_pressure_compensate, - } -}; - static const struct iio_chan_spec ms5611_channels[] =3D { { .type =3D IIO_PRESSURE, @@ -433,7 +423,20 @@ int ms5611_probe(struct iio_dev *indio_dev, struct dev= ice *dev, struct ms5611_state *st =3D iio_priv(indio_dev); =20 mutex_init(&st->lock); - st->chip_info =3D &chip_info_tbl[type]; + + switch (type) { + case MS5611: + st->compensate_temp_and_pressure =3D + ms5611_temp_and_pressure_compensate; + break; + case MS5607: + st->compensate_temp_and_pressure =3D + ms5607_temp_and_pressure_compensate; + break; + default: + return -EINVAL; + } + st->temp_osr =3D &ms5611_avail_temp_osr[ARRAY_SIZE(ms5611_avail_temp_osr) - 1]; st->pressure_osr =3D --=20 2.34.1 From nobody Wed Apr 8 01:35:38 2026 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 11443C4332F for ; Fri, 21 Oct 2022 14:00:49 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231352AbiJUOAr (ORCPT ); Fri, 21 Oct 2022 10:00:47 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46960 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231307AbiJUOAU (ORCPT ); Fri, 21 Oct 2022 10:00:20 -0400 Received: from dhl.lxnav.com (dhl.lxnav.com [IPv6:2a01:4f8:c010:2f07::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E1AC9760FD; Fri, 21 Oct 2022 06:59:46 -0700 (PDT) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 17D2A4068A; Fri, 21 Oct 2022 15:59:25 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=lxnav.com; s=dhl; t=1666360766; h=from:subject:date:message-id:to:cc:mime-version: content-transfer-encoding:in-reply-to:references; bh=aXnlVmknXqlNDzLOlbxpu9TZwfggntzi2DQjXqZO6uU=; b=wMDicj80enjKytifaED4mHK5xQHLe92TDYMkh9FwzOuXtPkCsKlsb8khqIqDVml6vsVLvb PQgzc5GHM466qwX24qdqoJqSAZ1NUzVn0dhpQsgzOljmzQx3LhL6mPHHXycy/GXA8V1Kkl DOjWZlAXrXszAsExiEVdQEgPiIbjicM/wo812c00oigQuW+fU7LHm3ZFlp1rpF2FWsBkB5 DvUma5B8h54muHaM+M+61YxsOk5+2edO4LBbnFrLpxJ4yNDJgfRAGLG4cQMtRu6k3/TJR4 ZCHjx8f5Ec/Bwp3qbW69DRH8QS6NTmZU2EGzZsRnOKNzQp0VA3k/M1JHnYS0Zw== From: Mitja Spes Cc: Mitja Spes , Jonathan Cameron , Lars-Peter Clausen , Rob Herring , Krzysztof Kozlowski , Andy Shevchenko , Lee Jones , Mark Brown , Marcus Folkesson , =?UTF-8?q?Uwe=20Kleine-K=C3=B6nig?= , Tomasz Duszynski , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 2/3] iio: pressure: ms5611: changed hardcoded SPI speed to value limited Date: Fri, 21 Oct 2022 15:58:21 +0200 Message-Id: <20221021135827.1444793-3-mitja@lxnav.com> In-Reply-To: <20221021135827.1444793-1-mitja@lxnav.com> References: <20221021135827.1444793-1-mitja@lxnav.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Last-TLS-Session-Version: TLSv1.3 To: unlisted-recipients:; (no To-header on input) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Don't hardcode the ms5611 SPI speed, limit it instead. Signed-off-by: Mitja Spes --- drivers/iio/pressure/ms5611_spi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/iio/pressure/ms5611_spi.c b/drivers/iio/pressure/ms561= 1_spi.c index 432e912096f4..a0a7205c9c3a 100644 --- a/drivers/iio/pressure/ms5611_spi.c +++ b/drivers/iio/pressure/ms5611_spi.c @@ -91,7 +91,7 @@ static int ms5611_spi_probe(struct spi_device *spi) spi_set_drvdata(spi, indio_dev); =20 spi->mode =3D SPI_MODE_0; - spi->max_speed_hz =3D 20000000; + spi->max_speed_hz =3D min(spi->max_speed_hz, 20000000U); spi->bits_per_word =3D 8; ret =3D spi_setup(spi); if (ret < 0) --=20 2.34.1 From nobody Wed Apr 8 01:35:38 2026 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 08511C4332F for ; Fri, 21 Oct 2022 14:00:43 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231384AbiJUOAj (ORCPT ); Fri, 21 Oct 2022 10:00:39 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46048 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231311AbiJUOAW (ORCPT ); Fri, 21 Oct 2022 10:00:22 -0400 Received: from dhl.lxnav.com (dhl.lxnav.com [IPv6:2a01:4f8:c010:2f07::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9B5FA10069; Fri, 21 Oct 2022 06:59:47 -0700 (PDT) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id C9CAD40764; Fri, 21 Oct 2022 15:59:31 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=lxnav.com; s=dhl; t=1666360773; h=from:subject:date:message-id:to:cc:mime-version: content-transfer-encoding:in-reply-to:references; bh=URJvSDdj3A8sCZTu24AcBhZBWiNAekT3eymuvA4CTvQ=; b=G+rLlFNGUJHICgyDaEQ4utGFtVOgemmaEn0rDs2i73omeThkUrTeDP6cAdE+zfI7PgwjzX kwqtp8L5nKmm/8RLkUIgSnXy8orX2u/ayh192CKme68jgPWgLihWb+VBc+nKQOmEJV+6C4 OStRoty1o52+u28fRJvRP8PcmnVRMYTyn8Ucv2lF12GVzInr7hhEZBztSlRLBnvbaI70aC 7DosExs8+aoQwMjvWqKf+UUfjUZF6p2f3NL6Ls3qfeSYqXI5cgw7Gx4NkhWCJRIXN8SeAD mxybCK65UMS1qHripm9cC0PSr2DvckSCRwD05wiaywC9uN+HBnIuK222EXEXYA== From: Mitja Spes Cc: Mitja Spes , Jonathan Cameron , Lars-Peter Clausen , Rob Herring , Krzysztof Kozlowski , Andy Shevchenko , Geert Uytterhoeven , Lee Jones , Alexandre Belloni , =?UTF-8?q?Uwe=20Kleine-K=C3=B6nig?= , Tomasz Duszynski , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 3/3] dt-bindings: iio: pressure: meas,ms5611: add max SPI frequency to the example Date: Fri, 21 Oct 2022 15:58:22 +0200 Message-Id: <20221021135827.1444793-4-mitja@lxnav.com> In-Reply-To: <20221021135827.1444793-1-mitja@lxnav.com> References: <20221021135827.1444793-1-mitja@lxnav.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Last-TLS-Session-Version: TLSv1.3 To: unlisted-recipients:; (no To-header on input) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Added max SPI frequency setting to the example. It is now honored by the driver. Signed-off-by: Mitja Spes Acked-by: Krzysztof Kozlowski --- Documentation/devicetree/bindings/iio/pressure/meas,ms5611.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/iio/pressure/meas,ms5611.yam= l b/Documentation/devicetree/bindings/iio/pressure/meas,ms5611.yaml index 4f06707450bf..08bd06e6dabe 100644 --- a/Documentation/devicetree/bindings/iio/pressure/meas,ms5611.yaml +++ b/Documentation/devicetree/bindings/iio/pressure/meas,ms5611.yaml @@ -52,6 +52,7 @@ examples: compatible =3D "meas,ms5611"; reg =3D <0>; vdd-supply =3D <&ldo_3v3_gnss>; + spi-max-frequency =3D <20000000>; }; }; ... --=20 2.34.1