Re: [PATCH 1/8] spi: dt-bindings: Introduce spi-cs-setup-ns property

From: Michael Walle
Date: Fri Nov 18 2022 - 09:15:17 EST


From: Tudor Ambarus <tudor.ambarus@xxxxxxxxxxxxx>

> 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