Re: [PATCH v1 0/1] iio: adc: ad7192: Refactor filter config

[David Lechner <dlechner@baylibre.com>]

On 4/25/25 8:20 AM, Alisa-Dariana Roman wrote:
> Dear maintainers,
> 
> Here is a pretty voluminous patch improving the filter setup for AD7192
> and similar chips.
> 
> I removed the write options for two attributes and I know that is
> questionable. I am sending this version just in case it is still viable
> since I think the driver is a lot cleaner like this.

If you can make the case that no one is using them, e.g. because the function
didn't work correctly, then it could be OK. But best not to break userspace if
we can help it.

For IIO_CHAN_INFO_LOW_PASS_FILTER_3DB_FREQUENCY, it seems easy enough to keep
ad7192_set_3db_filter_freq() and just modify it a bit to call the new
ad7192_set_filter_mode() with the 4 previously supported filter types.

And IIO_CHAN_INFO_OVERSAMPLING_RATIO seems like it could be preserved too, but
will take a bit more work. I'll comment more in the patch.

> 
> I also modified the size of the 3db filter low pass available attribute
> since AD7193/AD7194 have 16 filter modes, unlike the other chips that
> have 4.
> 
> Note that I moved a few of the functions around to be able to use them
> where needed.
> 
> Have a great weekend!
> 
> Kind regards,
> Alisa-Dariana Roman.
>