2020-09-10 12:36:15

by Nishant Malpani

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

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(-)

--
2.20.1

2020-09-10 17:53:07

by Jonathan Cameron

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

On Thu, 10 Sep 2020 17:59:57 +0530
Nishant Malpani <[email protected]> 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

2020-09-10 21:33:57

by Andy Shevchenko

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

On Thu, Sep 10, 2020 at 3:30 PM Nishant Malpani
<[email protected]> 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 <[email protected]>

> 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
>


--
With Best Regards,
Andy Shevchenko