linux-kernel - [PATCH 00/14] iio: pressure: mprls0025pa: driver code cleanup

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-Id: <20251218-mprls_cleanup-v1-0-b36a170f1a5c@subdimension.ro>
Date: Thu, 18 Dec 2025 13:05:42 +0200
From: Petre Rodan <petre.rodan@...dimension.ro>
To: Jonathan Cameron <jic23@...nel.org>, 
 David Lechner <dlechner@...libre.com>, 
 Nuno Sá <nuno.sa@...log.com>, 
 Andy Shevchenko <andy@...nel.org>, Andreas Klinger <ak@...klinger.de>
Cc: linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org, 
 Jonathan Cameron <Jonathan.Cameron@...wei.com>
Subject: [PATCH 00/14] iio: pressure: mprls0025pa: driver code cleanup

This series contains a collection of patches to the MPR sensor based
on feedback I received for other drivers.

major changes:
 - drop the use of devm_kzalloc()
 - trigger flag fix (define edge direction only in the device tree)
 - mitigate SPI timing violation by changing the measurement sequence
    (only affects users that do not use the EOC interrupt)
 - fix scan_type struct
 - pressure calculation fix for custom chips (does not affect users that define a sensor via the pressure-triplet)
 - stricter check for the status byte + better error return levels

minor changes:
 - includes added and removed
 - rename generic 'buffer' variable to 'rx_buf'
 - remove redundant locking

Tested on two sensors - MPRLS0015PA0000SA and MPRLS0001BA00001A

Datasheet: https://prod-edam.honeywell.com/content/dam/honeywell-edam/sps/siot/en-us/products/sensors/pressure-sensors/board-mount-pressure-sensors/micropressure-mpr-series/documents/sps-siot-mpr-series-datasheet-32332628-ciid-172626.pdf?download=false
Signed-off-by: Petre Rodan <petre.rodan@...dimension.ro>
---
Petre Rodan (14):
      iio: pressure: mprls0025pa: Kconfig allow bus selection
      iio: pressure: mprls0025pa: remove redundant mutex
      iio: pressure: mprls0025pa: rename buffer variable
      iio: pressure: mprls0025pa: introduce tx buffer
      iio: pressure: mprls0025pa: zero out spi_transfer struct
      iio: pressure: mprls0025pa: memset rx_buf before reading new data
      iio: pressure: mprls0025pa: make ops->write function consistent
      iio: pressure: mprls0025pa: stricter checks for the status byte
      iio: pressure: mprls0025pa: mitigate SPI CS delay violation
      iio: pressure: mprls0025pa: cleanup pressure calculation
      iio: pressure: mprls0025pa: fix scan_type struct
      iio: pressure: mprls0025pa: fix interrupt flag
      iio: pressure: mprls0025pa: cleanup includes and forward declarations
      iio: pressure: mprls0025pa: add copyright line

 drivers/iio/pressure/Kconfig           |  34 +++++----
 drivers/iio/pressure/mprls0025pa.c     | 121 +++++++++++++++------------------
 drivers/iio/pressure/mprls0025pa.h     |  22 ++----
 drivers/iio/pressure/mprls0025pa_i2c.c |  20 +++---
 drivers/iio/pressure/mprls0025pa_spi.c |  28 ++------
 5 files changed, 90 insertions(+), 135 deletions(-)
---
base-commit: f9e05791642810a0cf6237d39fafd6fec5e0b4bb
change-id: 20251215-mprls_cleanup-01de8971b439

Best regards,
-- 
Petre Rodan <petre.rodan@...dimension.ro>