Re: [PATCH v2 0/5] iio: Add output buffer support

2021-02-18 Thread Jonathan Cameron
On Wed, 17 Feb 2021 10:34:33 +0200
Alexandru Ardelean  wrote:

> This patchset is based on the new multibuffer set.
> It doesn't require the high-speed/mmap interface.
> That will come later on a v2.
> 
> Changelog v1 -> v2:
> * 
> https://lore.kernel.org/linux-iio/20210212102021.47276-4-alexandru.ardel...@analog.com/T/#u
> * removed DMA patches for now
> * in patch 'iio: Add output buffer support'
>   - added /sys/bus/iio/devices/iio:deviceX/bufferY/direction attribute
>   - thinking about this, an update to the new buffer infrastructure is
> required when adding ADDAC/MxFE/transceivers; right now there is no
> problem, because we have only ADCs and DACs; but when we get
> transceivers, a bufferY/ directory needs to filter in/out
> scan_elements/ ; this only occured to me recently (it's one of those
> things that pops up later in mind)

For that we can rely on review in the short term, but agreed a sanity
check that everything matches would make sense.

> * added 'iio: Documentation: update definitions for bufferY and scan_elements'
>   - seems I forgot this on the original multibuffer patchset

Likewise. :)  Who needs docs?

> * added 'iio: triggered-buffer: extend support to configure output buffers'
>   - basically output triggered buffer support
> * added 'iio: dac: ad5686: Add PWM as a trigger source'
>   - this is a first user of this infrastructure
> 
> Alexandru Ardelean (2):
>   iio: Documentation: update definitions for bufferY and scan_elements
>   iio: triggered-buffer: extend support to configure output buffers
> 
> Lars-Peter Clausen (2):
>   iio: Add output buffer support
>   iio: kfifo-buffer: Add output buffer support
> 
> Mircea Caprioru (1):
>   iio: dac: ad5686: Add PWM as a trigger source
> 
>  Documentation/ABI/testing/sysfs-bus-iio   |  92 +++
>  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|   1 +
>  .../common/hid-sensors/hid-sensor-trigger.c   |   5 +-
>  drivers/iio/dac/ad5686-spi.c  |   2 +-
>  drivers/iio/dac/ad5686.c  | 146 +-
>  drivers/iio/dac/ad5686.h  |   7 +-
>  drivers/iio/dac/ad5696-i2c.c  |   2 +-
>  drivers/iio/industrialio-buffer.c | 128 ++-
>  include/linux/iio/buffer.h|   7 +
>  include/linux/iio/buffer_impl.h   |  11 ++
>  include/linux/iio/triggered_buffer.h  |  11 +-
>  16 files changed, 459 insertions(+), 17 deletions(-)
> 



[PATCH v2 0/5] iio: Add output buffer support

2021-02-17 Thread Alexandru Ardelean
This patchset is based on the new multibuffer set.
It doesn't require the high-speed/mmap interface.
That will come later on a v2.

Changelog v1 -> v2:
* 
https://lore.kernel.org/linux-iio/20210212102021.47276-4-alexandru.ardel...@analog.com/T/#u
* removed DMA patches for now
* in patch 'iio: Add output buffer support'
  - added /sys/bus/iio/devices/iio:deviceX/bufferY/direction attribute
  - thinking about this, an update to the new buffer infrastructure is
required when adding ADDAC/MxFE/transceivers; right now there is no
problem, because we have only ADCs and DACs; but when we get
transceivers, a bufferY/ directory needs to filter in/out
scan_elements/ ; this only occured to me recently (it's one of those
things that pops up later in mind)
* added 'iio: Documentation: update definitions for bufferY and scan_elements'
  - seems I forgot this on the original multibuffer patchset
* added 'iio: triggered-buffer: extend support to configure output buffers'
  - basically output triggered buffer support
* added 'iio: dac: ad5686: Add PWM as a trigger source'
  - this is a first user of this infrastructure

Alexandru Ardelean (2):
  iio: Documentation: update definitions for bufferY and scan_elements
  iio: triggered-buffer: extend support to configure output buffers

Lars-Peter Clausen (2):
  iio: Add output buffer support
  iio: kfifo-buffer: Add output buffer support

Mircea Caprioru (1):
  iio: dac: ad5686: Add PWM as a trigger source

 Documentation/ABI/testing/sysfs-bus-iio   |  92 +++
 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|   1 +
 .../common/hid-sensors/hid-sensor-trigger.c   |   5 +-
 drivers/iio/dac/ad5686-spi.c  |   2 +-
 drivers/iio/dac/ad5686.c  | 146 +-
 drivers/iio/dac/ad5686.h  |   7 +-
 drivers/iio/dac/ad5696-i2c.c  |   2 +-
 drivers/iio/industrialio-buffer.c | 128 ++-
 include/linux/iio/buffer.h|   7 +
 include/linux/iio/buffer_impl.h   |  11 ++
 include/linux/iio/triggered_buffer.h  |  11 +-
 16 files changed, 459 insertions(+), 17 deletions(-)

-- 
2.17.1