Re: [PATCH v5 5/5] iio: mcp9600: Add support for IIR filter

[Ben Collins <ben.collins@linux.dev>]

[-- Attachment #1: Type: text/plain, Size: 1346 bytes --]

On Mon, Aug 18, 2025 at 01:59:34PM -0500, David Lechner wrote:
> On 8/18/25 1:47 PM, Ben Collins wrote:
> > On Mon, Aug 18, 2025 at 07:15:39PM -0500, Jonathan Cameron wrote:
> >> On Sun, 17 Aug 2025 23:59:53 -0400
> >> Ben Collins <bcollins@kernel.org> wrote:
> >>
> >>> From: Ben Collins <bcollins@watter.com>
> >>>
> >>> MCP9600 supports an IIR filter with 7 levels. Add IIR attribute
> >>> to allow get/set of this value.
> >>>
> >>> Use a filter_type[none, ema] for enabling the IIR filter.
> >> Hi Ben,
> >>
> >> A few comments inline. You also need to send an additional patch to update
> >> the filter_type docs in Documentation/ABI/testing/sysfs-bus-iio
> > 
> > Hi Jonathan,
> > 
> > I just sent a v6 because I was getting too many comments on the
> > dt-bindings patch.
> 
> Actually, folks will be happier if you slow down a bit. General
> advice is only submit one revision per week since some people only
> have time to review once per week.

I appreciate the info. Admittedly, when I was more prolific with kernel
development, the pipline and protocol was much leaner, so I'm not used
to the more recent "norm" quite yet.

Not complaining, just noting. I'll get there.

-- 
 Ben Collins
 https://libjwt.io
 https://github.com/benmcollins
 --
 3EC9 7598 1672 961A 1139  173A 5D5A 57C7 242B 22CF

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]