From: Stefan Popa <stefan.popa@analog.com>
To: <linux-iio@vger.kernel.org>
Cc: <linux-hwmon@vger.kernel.org>, <devicetree@vger.kernel.org>,
<linux-kernel@vger.kernel.org>, <jic23@kernel.org>,
<andy@kernel.org>, <nuno.sa@analog.com>, <linux@roeck-us.net>,
<robh@kernel.org>, <krzk+dt@kernel.org>, <conor+dt@kernel.org>,
<dlechner@baylibre.com>, <ciprian.hegbeli@analog.com>,
<siratul.islam@linux.dev>, <u.kleine-koenig@baylibre.com>,
Stefan Popa <stefan.popa@analog.com>
Subject: [PATCH v5 0/2] iio: adc: add MAX40080 current-sense amplifier driver
Date: Thu, 23 Jul 2026 09:50:34 +0300 [thread overview]
Message-ID: <20260723065036.2683075-1-stefan.popa@analog.com> (raw)
This series adds support for the MAX40080, a bidirectional current-sense
amplifier with an integrated 12-bit ADC.
The driver is placed in IIO rather than hwmon because:
- IIO's scale/raw model naturally exposes the selectable gain ranges and
lets userspace apply the correct scale per range.
- Buffered capture support is planned for a follow-up series -- each
conversion returns a matched current+voltage pair, which maps cleanly
to IIO's buffered/triggered model.
Changes since v4:
- Initialize tmp = 0 in max40080_read_iv() before read_poll_timeout to
avoid using potentially uninitialized value in the condition check
(Sashiko, Jonathan).
- Simplify max40080_get_oversampling_ratio() to use cached value instead
of reading from the device (Jonathan).
- Remove DMA-safe buffer from struct -- I2C layer already bounces data
internally for i2c_smbus_* calls (Jonathan).
- Make shunt-resistor-micro-ohms required in DT binding. The default
1 Ohm was arbitrary and rarely correct for real hardware (Jonathan).
- Remove PM ops entirely. Having only resume without suspend was flagged
as "papering over" possible power loss issues (Jonathan).
- Remove the {} scope around propname, declare it at function top (Andy).
- Remove mod_devicetable.h include -- i2c.h provides necessary types
(Jonathan).
Changes since v3:
- Removed all regmap usage -- regmap doesn't support SMBus PEC or Quick
Command, both required by this device (Nuno).
- Fixed locking: max40080_update_bits() now takes the lock internally,
and callers update cached state after the call returns (Nuno).
- Consistent comment style: capital first letter + period (Andy).
- Use propname variable to deduplicate "shunt-resistor-micro-ohms" (Andy).
- Return IIO_VAL_* inline instead of break + return pattern (Nuno).
Changes since v2:
- Added shunt-resistor-micro-ohms DT property with default value.
- Use single-shot mode with SMBus Quick Command trigger for correlated
current/voltage reads.
- Precompute current scale per range at probe time.
Link: https://lore.kernel.org/linux-iio/20260717123852.1140893-1-stefan.popa@analog.com/ [v4]
Link: https://lore.kernel.org/linux-iio/20260715063652.368501-1-stefan.popa@analog.com/ [v3]
Link: https://lore.kernel.org/linux-iio/20260713120226.90303-1-stefan.popa@analog.com/ [v2]
Link: https://lore.kernel.org/linux-iio/20260703102941.1141341-1-stefan.popa@analog.com/ [v1]
Stefan Popa (2):
dt-bindings: iio: adc: add maxim,max40080
iio: adc: add MAX40080 current-sense amplifier driver
.../bindings/iio/adc/maxim,max40080.yaml | 56 ++
MAINTAINERS | 9 +
drivers/iio/adc/Kconfig | 11 +
drivers/iio/adc/Makefile | 1 +
drivers/iio/adc/max40080.c | 586 ++++++++++++++++++
5 files changed, 663 insertions(+)
create mode 100644 Documentation/devicetree/bindings/iio/adc/maxim,max40080.yaml
create mode 100644 drivers/iio/adc/max40080.c
--
2.53.0
next reply other threads:[~2026-07-23 6:51 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-23 6:50 Stefan Popa [this message]
2026-07-23 6:50 ` [PATCH v5 1/2] dt-bindings: iio: adc: add maxim,max40080 Stefan Popa
2026-07-23 16:23 ` Siratul Islam
2026-07-25 23:31 ` Jonathan Cameron
2026-07-26 9:59 ` Siratul Islam
2026-07-26 13:46 ` Guenter Roeck
2026-07-23 16:40 ` Conor Dooley
2026-07-26 15:37 ` David Lechner
2026-07-23 6:50 ` [PATCH v5 2/2] iio: adc: add MAX40080 current-sense amplifier driver Stefan Popa
2026-07-23 17:21 ` Siratul Islam
2026-07-25 23:58 ` Jonathan Cameron
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260723065036.2683075-1-stefan.popa@analog.com \
--to=stefan.popa@analog.com \
--cc=andy@kernel.org \
--cc=ciprian.hegbeli@analog.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dlechner@baylibre.com \
--cc=jic23@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=linux-hwmon@vger.kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=nuno.sa@analog.com \
--cc=robh@kernel.org \
--cc=siratul.islam@linux.dev \
--cc=u.kleine-koenig@baylibre.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®