mbox series

[v3,0/3] iio: gyro: adxrs290: Add triggered buffer & debugfs support

Message ID 20200910123000.16860-1-nish.malpani25@gmail.com (mailing list archive)
Headers show
Series iio: gyro: adxrs290: Add triggered buffer & debugfs support | expand

Message

Nishant Malpani Sept. 10, 2020, 12:29 p.m. UTC 
Introduce DATA_RDY trigger for triggered buffer setup; this enables continuous
data capture. Additionally, add support for direct register access using the debugfs 
iio interface. 

The device-tree bindings documentation illustrates an example of using a GPIO irq
line to trigger a data capture.

Nishant Malpani (3):
  iio: gyro: adxrs290: Add triggered buffer support
  dt-bindings: iio: gyro: adxrs290: Add interrupts support
  iio: gyro: adxrs290: Add debugfs register access support

 .../bindings/iio/gyroscope/adi,adxrs290.yaml  |   6 +
 drivers/iio/gyro/Kconfig                      |   2 +
 drivers/iio/gyro/adxrs290.c                   | 237 ++++++++++++++++--
 3 files changed, 231 insertions(+), 14 deletions(-)

Comments

Andy Shevchenko Sept. 10, 2020, 1:51 p.m. UTC | #1 
On Thu, Sep 10, 2020 at 3:30 PM Nishant Malpani
<nish.malpani25@gmail.com> wrote:
>
> Introduce DATA_RDY trigger for triggered buffer setup; this enables continuous
> data capture. Additionally, add support for direct register access using the debugfs
> iio interface.
>
> The device-tree bindings documentation illustrates an example of using a GPIO irq
> line to trigger a data capture.
>

Patches 1 and 3 looks good to me now
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>

> Nishant Malpani (3):
>   iio: gyro: adxrs290: Add triggered buffer support
>   dt-bindings: iio: gyro: adxrs290: Add interrupts support
>   iio: gyro: adxrs290: Add debugfs register access support
>
>  .../bindings/iio/gyroscope/adi,adxrs290.yaml  |   6 +
>  drivers/iio/gyro/Kconfig                      |   2 +
>  drivers/iio/gyro/adxrs290.c                   | 237 ++++++++++++++++--
>  3 files changed, 231 insertions(+), 14 deletions(-)
>
> --
> 2.20.1
>
Jonathan Cameron Sept. 10, 2020, 5:46 p.m. UTC | #2 
On Thu, 10 Sep 2020 17:59:57 +0530
Nishant Malpani <nish.malpani25@gmail.com> wrote:

> Introduce DATA_RDY trigger for triggered buffer setup; this enables continuous
> data capture. Additionally, add support for direct register access using the debugfs 
> iio interface. 
> 
> The device-tree bindings documentation illustrates an example of using a GPIO irq
> line to trigger a data capture.
> 
> Nishant Malpani (3):
>   iio: gyro: adxrs290: Add triggered buffer support
>   dt-bindings: iio: gyro: adxrs290: Add interrupts support
>   iio: gyro: adxrs290: Add debugfs register access support
> 
>  .../bindings/iio/gyroscope/adi,adxrs290.yaml  |   6 +
>  drivers/iio/gyro/Kconfig                      |   2 +
>  drivers/iio/gyro/adxrs290.c                   | 237 ++++++++++++++++--
>  3 files changed, 231 insertions(+), 14 deletions(-)
> 

Hi Nishant,

Something odd went on with this version.  It's not come across threaded
and patch 1 is missing.  Please resend.

Thanks,

Jonathan