Messages in this thread | | | From | Michael Walle <> | Subject | Re: [PATCH 1/8] spi: dt-bindings: Introduce spi-cs-setup-ns property | Date | Fri, 18 Nov 2022 15:14:58 +0100 |
| |
From: Tudor Ambarus <tudor.ambarus@microchip.com>
> diff --git a/Documentation/devicetree/bindings/spi/spi-peripheral-props.yaml b/Documentation/devicetree/bindings/spi/spi-peripheral-props.yaml > index dca677f9e1b9..ead2cccf658f 100644 > --- a/Documentation/devicetree/bindings/spi/spi-peripheral-props.yaml > +++ b/Documentation/devicetree/bindings/spi/spi-peripheral-props.yaml > @@ -44,6 +44,11 @@ properties: > description: > Maximum SPI clocking speed of the device in Hz. > > + spi-cs-setup-ns: > + description: > + Delay in nanosecods to be introduced by the controller after CS is > + asserted. > +
Does this need a type as the spi-cs-setup-ns is apparently just 16bit? At least the driver uses it that way.
But IMHO this should just be a normal uint32 value to be consistent with all the other properties. Also the max value with 16bit will be 'just' 65us.
-michael
| |