Re: [PATCH v6 4/4] iio: adc: ad4080: add support for AD4880 dual-channel ADC
From: Andy Shevchenko
Date: Fri Mar 13 2026 - 10:30:16 EST
On Fri, Mar 13, 2026 at 01:58:53PM +0200, Antoniu Miclaus wrote:
> Add support for the AD4880, a dual-channel 20-bit 40MSPS SAR ADC with
> integrated fully differential amplifiers (FDA).
>
> The AD4880 has two independent ADC channels, each with its own SPI
> configuration interface. The driver uses spi_new_ancillary_device() to
> create an additional SPI device for the second channel, allowing both
> channels to share the same SPI bus with different chip selects.
I am still not sure this is the best approach we can have.
In any case, I have immediate questions here about regmap usage.
- Why do we need to have a separate regmap per channel?
- What is special about channel 0?
- Is it okay to communicate with different channels simultaneously?
Wouldn't be a nasty race with HW IO?
--
With Best Regards,
Andy Shevchenko