Re: [PATCH 1/3] iio: filter: admv8818: Add missing error code
From: Andy Shevchenko
Date: Fri Feb 27 2026 - 11:14:18 EST
On Fri, Feb 27, 2026 at 10:01:25AM -0600, David Lechner wrote:
> On 2/27/26 1:06 AM, Andy Shevchenko wrote:
> > On Fri, Feb 27, 2026 at 12:42:11AM -0600, Ethan Tidmore wrote:
> >> On Fri Feb 27, 2026 at 12:14 AM CST, Ethan Tidmore wrote:
...
> >> Just realized I should have used -EIO for the error code. Will wait
> >> however on reviews before v2.
> >
> > Why -EIO? Is it used in the same situation in other cases in this driver?
>
> I think EIO makes sense here. We are reading from a chip. And if the
> values we read aren't one of the expected values, the mostly likely
> reason is the chip is malfunctioning or noise on the wires.
Makes sense, but needs to be properly spoken in the commit message.
--
With Best Regards,
Andy Shevchenko