Re: [PATCH v3 0/2] iio: adc: Add the Ingenic X1600 ADC

From: Jonathan Cameron

Date: Mon Oct 26 2026 - 21:16:07 EST


On Mon, 26 Oct 2026 09:36:29 +0100
Olivier Louvignes <olivier@xxxxxxxxx> wrote:

> A driver for the SAR ADC of the Ingenic X1600: four auxiliary channels,
> 12 bits, one interrupt per channel. Tested on a HiBy R1, whose keys are on
> channel 0 behind adc-keys.
>
> By their programming manuals, the X1830 (3 channels, 10 bits) and the
> X2000 (5 channels, 10 bits, one more data register) have the same
> register layout, two results to a data register. As Jonathan said on v2,
> the X1600 name stays, and supporting them now is welcome but not
> required: I have neither to test on, so this version is X1600-only, and
> a chip_info with the channel count and the resolution can add them
> later. Nikolaus, if Letux has either at hand, adding them on top with a
> test would be welcome.

Nothing to add from me for this version but perhaps wait a few days to
see if this suggestion gets more feedback.

Jonathan