Re: [PATCH v4 3/6] iio: adc: adi-axi-adc: add axi_adc_oversampling_ratio_set

From: David Lechner
Date: Mon May 19 2025 - 11:16:32 EST


On 5/19/25 9:02 AM, Pop Ioan Daniel wrote:
> Add support for setting decimation rate.
>
> Signed-off-by: Pop Ioan Daniel <pop.ioan-daniel@xxxxxxxxxx>
> ---

With the bit below fixed:

Reviewed-by: David Lechner <dlechner@xxxxxxxxxxxx>

...
> @@ -381,7 +397,8 @@ static int axi_adc_ad485x_data_size_set(struct iio_backend *back,
> }
>
> static int axi_adc_ad485x_oversampling_ratio_set(struct iio_backend *back,
> - unsigned int ratio)
> + unsigned int chan,
> + unsigned int ratio)

I think this change belongs in the previous patch. Most importantly because
it could cause a compile error during a git bisect, but also because that
is where it logically belongs.

> {
> struct adi_axi_adc_state *st = iio_backend_get_priv(back);
>