On Mon, 2023-10-23 at 17:05 +0300, Ramona Gradinariu wrote: > The adis16460 device requires a stall time between SPI > transactions (during which the chip select is inactive), > with a minimum value equal to 16 microseconds. > This commit adds 'spi-cs-inactive-delay-ns' property, which should > indicate the stall time between consecutive SPI transactions. > > Signed-off-by: Ramona Gradinariu <ramona.gradinariu@xxxxxxxxxx> > --- > changes in v2: > - added default value > - updated description > - updated commit message > .../devicetree/bindings/iio/imu/adi,adis16460.yaml | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml > b/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml > index 4e43c80e5119..f10469b86ee0 100644 > --- a/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml > +++ b/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml > @@ -25,6 +25,12 @@ properties: > > spi-cpol: true > > + spi-cs-inactive-delay-ns: > + minimum: 16000 > + default: 16000 > + description: > + Indicates the stall time between consecutive SPI transactions. > + You should drop the description... Also, give more time before posting a v2 so others get a chance to review your patches. It's also better for you since you can gather more change requests. - Nuno Sá
