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

Gabriel Rondon posted 2 patches 1 month, 1 week ago
drivers/iio/accel/kionix-kx022a.c | 19 ++++++++++---------
1 file changed, 10 insertions(+), 9 deletions(-)
[PATCH v3 0/2] iio: accel: kionix-kx022a: unify staging buffer and convert to iio_push_to_buffers_with_ts()
Posted by Gabriel Rondon 1 month, 1 week ago
This is the kx022a staging-buffer unification plus the deprecated-API
conversion. Patch 1 drops the redundant buffer and routes the one-shot
read and the triggered handler through scan; patch 2 does the
iio_push_to_buffers_with_ts() conversion.

Changes in v3:
- get_axis(): use a local __le16 *buf and le16_to_cpup() (Andy)
- Add Suggested-by: Jonathan Cameron to patch 1
- Trim patch 2 changelog

v2: https://lore.kernel.org/linux-iio/20260818215122.52715-1-grondon@gmail.com/

Gabriel Rondon (2):
  iio: accel: kionix-kx022a: use scan struct for one-shot and trigger
    reads
  iio: accel: kionix-kx022a: use iio_push_to_buffers_with_ts()

 drivers/iio/accel/kionix-kx022a.c | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

-- 
2.50.1 (Apple Git-155)
Re: [PATCH v3 0/2] iio: accel: kionix-kx022a: unify staging buffer and convert to iio_push_to_buffers_with_ts()
Posted by Jonathan Cameron 1 month ago
On Wed, 19 Aug 2026 11:37:44 +0100
Gabriel Rondon <grondon@gmail.com> wrote:

> This is the kx022a staging-buffer unification plus the deprecated-API
> conversion. Patch 1 drops the redundant buffer and routes the one-shot
> read and the triggered handler through scan; patch 2 does the
> iio_push_to_buffers_with_ts() conversion.
> 
Ah. I didn't see you'd posted a new version with the changes requested
post my having picked this up (to fast as it turned out!)

If you are going to do this - which is fine - then reply to original
thread to call that out.  I think we ended up pretty much with the
same thing upstream though as I went ahead and tweaked it in my tree.
Mostly once a patch has been accepted, it's fixes on top.  Things are
a little more flexible when I haven't yet pushed it out as a non rebasing
(ish) branch like iio.git/togreg

Give me a shout if I messed anything up when tweaking it.

Jonathan

> Changes in v3:
> - get_axis(): use a local __le16 *buf and le16_to_cpup() (Andy)
> - Add Suggested-by: Jonathan Cameron to patch 1
> - Trim patch 2 changelog
> 
> v2: https://lore.kernel.org/linux-iio/20260818215122.52715-1-grondon@gmail.com/
> 
> Gabriel Rondon (2):
>   iio: accel: kionix-kx022a: use scan struct for one-shot and trigger
>     reads
>   iio: accel: kionix-kx022a: use iio_push_to_buffers_with_ts()
> 
>  drivers/iio/accel/kionix-kx022a.c | 19 ++++++++++---------
>  1 file changed, 10 insertions(+), 9 deletions(-)
>