2023-10-27 14:05:29

by Gradinariu, Ramona

[permalink] [raw]
Subject: [PATCH v3 0/3] iio: imu: adis: Use spi cs inactive delay

A delay is needed each time the chip selected becomes inactive,
even after burst data readings are performed.
Currently, there is no delay added after a burst reading
and in case a new SPI transfer is performed before
the needed delay, the adis device becomes unresponsive until
reset.

This commit is adding the needed delay directly to the spi driver,
using the cs_inactive parameter, in case it is not set and is
removing the additional chip select change delay present in adis
APIs to remove the double delay.

Ramona Gradinariu (3):
iio: imu: adis: Use spi cs inactive delay
dt-bindings: adis16475: Add 'spi-cs-inactive-delay-ns' property
dt-bindings: adis16460: Add 'spi-cs-inactive-delay-ns' property

.../bindings/iio/imu/adi,adis16460.yaml | 4 ++++
.../bindings/iio/imu/adi,adis16475.yaml | 4 ++++
drivers/iio/imu/adis.c | 18 ++++++------------
3 files changed, 14 insertions(+), 12 deletions(-)

--
2.34.1


2023-10-28 15:24:09

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH v3 0/3] iio: imu: adis: Use spi cs inactive delay

On Fri, 27 Oct 2023 17:03:55 +0300
Ramona Gradinariu <[email protected]> wrote:

> A delay is needed each time the chip selected becomes inactive,
> even after burst data readings are performed.
> Currently, there is no delay added after a burst reading
> and in case a new SPI transfer is performed before
> the needed delay, the adis device becomes unresponsive until
> reset.
>
> This commit is adding the needed delay directly to the spi driver,
> using the cs_inactive parameter, in case it is not set and is
> removing the additional chip select change delay present in adis
> APIs to remove the double delay.

Series applied.

I'm a bit confused though as this one is turning up in patchwork.
Meh, maybe a delay and it will show up soon.

Jonathan

>
> Ramona Gradinariu (3):
> iio: imu: adis: Use spi cs inactive delay
> dt-bindings: adis16475: Add 'spi-cs-inactive-delay-ns' property
> dt-bindings: adis16460: Add 'spi-cs-inactive-delay-ns' property
>
> .../bindings/iio/imu/adi,adis16460.yaml | 4 ++++
> .../bindings/iio/imu/adi,adis16475.yaml | 4 ++++
> drivers/iio/imu/adis.c | 18 ++++++------------
> 3 files changed, 14 insertions(+), 12 deletions(-)
>
> --
> 2.34.1
>