Re: [PATCH v3 06/10] iio: adc: ad4170: Add support for buffered data capture

[Jonathan Cameron <jic23@kernel.org>]

On Tue, 13 May 2025 09:35:23 -0300
Marcelo Schmitt <marcelo.schmitt@analog.com> wrote:

> Extend the AD4170 driver to allow buffered data capture in continuous read
> mode. In continuous read mode, the chip skips the instruction phase and
> outputs just ADC sample data, enabling faster sample rates to be reached.
> The internal channel sequencer always starts sampling from channel 0 and
> channel 0 must be enabled if more than one channel is selected for data
> capture. The scan mask validation callback checks if the aforementioned
> condition is met.
> 
> Signed-off-by: Marcelo Schmitt <marcelo.schmitt@analog.com>

I'm curious - why no timestamps?

> diff --git a/drivers/iio/adc/ad4170.c b/drivers/iio/adc/ad4170.c
> index 218f768042fe..e8856c911dfd 100644
> --- a/drivers/iio/adc/ad4170.c
> +++ b/drivers/iio/adc/ad4170.c

>  
>  enum ad4170_ref_buf {
> @@ -309,9 +322,13 @@ struct ad4170_state {
>  	struct spi_device *spi;
>  	struct regmap *regmap;
>  	int sps_tbl[ARRAY_SIZE(ad4170_filt_names)][AD4170_MAX_FS_TBL_SIZE][2];
> +	__be32 bounce_buffer[AD4170_MAX_CHANNELS];
> +	struct spi_message msg;
> +	struct spi_transfer xfer;
> +	struct iio_trigger *trig;
> +	struct completion completion;
>  	int pins_fn[AD4170_NUM_ANALOG_PINS];
>  	u32 int_pin_sel;
> -	struct completion completion;

Probably better to move this to it's new location when it's introduced and
reduce noise a little here.

>  	/*