Re: [PATCH v7 2/2] iio: adc: add MAX40080 current-sense amplifier driver
"Siratul Islam" <[email protected]>
| Newsgroups | org.kernel.vger.linux-iio,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <[email protected]> |
August 18, 2026 at 8:29 PM, "Stefan Popa" <[email protected]> wrote: > > The MAX40080 is a bidirectional current-sense amplifier with an > integrated 12-bit ADC and an I2C/SMBus interface. It measures the > voltage across an external shunt resistor and the input bus voltage, > storing the results in an internal FIFO. > > Add a direct-mode IIO driver exposing the current and voltage channels > with raw, scale and hardware-gain attributes, a configurable > oversampling (digital averaging) ratio, and PEC-protected register > access. The current scale is derived from the shunt resistor value > described in the device tree. > > The driver operates in single-measurement mode: each raw read triggers > an on-demand conversion via SMBus Quick Command and returns a matched > current/voltage pair. This avoids the latency and complexity of the > continuous FIFO mode while ensuring each read reflects the current > state. The two selectable current-sense ranges are exposed through > scale/scale_available. > > Continuous FIFO buffering, threshold events and the alert interrupt are > intentionally left out of this initial submission and may be added > later. > > Co-developed-by: Ciprian Hegbeli <[email protected]> > Signed-off-by: Ciprian Hegbeli <[email protected]> > Signed-off-by: Stefan Popa <[email protected]> > --- > Looks like my concerns have been addressed. Assuming you are going to send v8 with fixes for Andy and Jonathan, feel free to add my tag. Reviewed-by: Siratul Islam <[email protected]>