RE: [PATCH v6 0/6] iio: Add output buffer support

From: Chindris, Mihail
Date: Thu Oct 07 2021 - 04:10:44 EST


> -----Original Message-----
> From: Mihail Chindris <mihail.chindris@xxxxxxxxxx>
> Sent: Thursday, 7 October 2021 11:00
> To: linux-kernel@xxxxxxxxxxxxxxx; linux-iio@xxxxxxxxxxxxxxx
> Cc: lars@xxxxxxxxxx; Hennerich, Michael
> <Michael.Hennerich@xxxxxxxxxx>; jic23@xxxxxxxxxx; Sa, Nuno
> <Nuno.Sa@xxxxxxxxxx>; Bogdan, Dragos <Dragos.Bogdan@xxxxxxxxxx>;
> alexandru.ardelean@xxxxxxxxxx; Chindris, Mihail
> <Mihail.Chindris@xxxxxxxxxx>
> Subject: [PATCH v6 0/6] iio: Add output buffer support
>
> Changelog v5 -> v6:
> * https://lore.kernel.org/all/20210916182914.1810-1-
> mihail.chindris@xxxxxxxxxx
> * Fix check in iio_update_buffers adding test on insert_buffer
> * Return if check in iio_update_buffers fails.
> * Fix length typo
> * Add rb->direction check in both read and write functions.
> * Add fixes tags
> * Add Reviewed-by: Alexandru Ardelean <ardeleanalex@xxxxxxxxx> line
> * Remove unnecessary (u8 *) cast
> * Remove unnecessary AD5766_NUM_CH define
* Write to input register in ad5766_trigger_handler for simultaneous update
>
> Changelog v4 -> v5:
> * https://lore.kernel.org/all/20210820165927.4524-1-
> mihail.chindris@xxxxxxxxxx
> * Remove ad3552r example from series and replace with the update of an
> existing driver: ad5662. Will add ad3552r in other another series.
> * Make normal comment from kernel-doc comment. (A bot was
> complaining about that)
> * Add indio_dev->info check
> * Rename iio_buffer_remove_sample -> iio_pop_from_buffer
> * Fix comment of remove_from: sample -> scan
> * Change iio_pop_from_buffer data type to void * to be consistent with
> iio_push_to_buffers
> * Remove use watermark, in our kernel is not used and I can't think of an
> usecase for it.
> * Reimplement write to increment buffer index and handle blocking and
> noblocking calls
> * Move `if (insert_buffer->direction == IIO_BUFFER_DIRECTION_OUT)`
> outside lock
> * Remove redundant checks of `if (insert_buffer->direction ==
> IIO_BUFFER_DIRECTION_OUT)`
>
> Alexandru Ardelean (1):
> iio: triggered-buffer: extend support to configure output buffers
>
> Lars-Peter Clausen (1):
> iio: kfifo-buffer: Add output buffer support
>
> Mihail Chindris (4):
> iio: Add output buffer support
> drivers: iio: dac: ad5766: Fix dt property name
> Documentation:devicetree:bindings:iio:dac: Fix val
> drivers:iio:dac:ad5766.c: Add trigger buffer
>
> .../bindings/iio/dac/adi,ad5766.yaml | 2 +-
> drivers/iio/accel/adxl372.c | 1 +
> drivers/iio/accel/bmc150-accel-core.c | 1 +
> drivers/iio/adc/at91-sama5d2_adc.c | 4 +-
> .../buffer/industrialio-triggered-buffer.c | 8 +-
> drivers/iio/buffer/kfifo_buf.c | 50 +++++++
> .../cros_ec_sensors/cros_ec_sensors_core.c | 5 +-
> .../common/hid-sensors/hid-sensor-trigger.c | 5 +-
> drivers/iio/dac/ad5766.c | 48 ++++++-
> drivers/iio/iio_core.h | 4 +
> drivers/iio/industrialio-buffer.c | 127 +++++++++++++++++-
> drivers/iio/industrialio-core.c | 1 +
> include/linux/iio/buffer.h | 7 +
> include/linux/iio/buffer_impl.h | 11 ++
> include/linux/iio/triggered_buffer.h | 11 +-
> 15 files changed, 269 insertions(+), 16 deletions(-)
>
>
> base-commit: 94a853eca720ac9e385e59f27e859b4a01123f58
> --
> 2.27.0