Re: [PATCH v2 0/2] iio: accel: kionix-kx022a: unify staging buffer and convert to iio_push_to_buffers_with_ts()

[Andy Shevchenko]

On Wed, Aug 19, 2026 at 08:15:40AM +0300, Matti Vaittinen wrote:
> On 19/08/2026 03:09, Jonathan Cameron wrote:
> > On Tue, 18 Aug 2026 22:51:20 +0100
> > Gabriel Rondon <grondon@xxxxxxxxx> wrote:
> > 
> > > v1 was a single patch converting the two push sites to
> > > iio_push_to_buffers_with_ts(). Reviewing it, Jonathan pointed out that
> > > the driver carries two separate staging areas holding the same thing
> > > (buffer[8] and the scan struct), and asked to fold the cleanup into this
> > > set.
> > > 
> > > So v2 is a two-patch series: patch 1 drops the redundant buffer and
> > > routes the one-shot read and the triggered handler through scan, and
> > > patch 2 does the deprecated-API conversion, now with a single buffer to
> > > push at both sites.
> > > 
> > > Changes in v2:
> > > - New patch 1: drop buffer[8], use scan for the one-shot read and the
> > >    triggered handler, move IIO_DMA_MINALIGN onto scan (Jonathan)
> > > - Patch 2 now pushes data->scan at both sites instead of data->buffer
> > > 
> > Nice.  All looks good to me, so I'll queue it up.
> > 
> > Applied to the testing branch of iio.git which will be rebased on rc1 once
> > available.
> > 
> > Note that there is plenty of time for additional feedback, tags or indeed
> > me to drop it again if someone spots something I missed.
> 
> Ah, Jonathan was quick and efficient :)
> I'll drop my comments to 'nits' in order to not generate more work for

And I, in the opposite, insist on mine against patch 1 as I consider that that
makes code easier to read and follow.

> Jonathan. Hence, acting on my comments is not required.

-- 
With Best Regards,
Andy Shevchenko