RE: [PATCH V1 21/26] DT bindings: spi: add tx/rx clock delay SPI client properties

[Sowjanya Komatineni]

> On Tue, Mar 26, 2019 at 10:56:42PM -0700, Sowjanya Komatineni wrote:
> > This patch adds Tegra SPI master tx and rx clock delay properties.
> > 
> > TX/RX clock delays may vary depending on the platform design trace 
> > lengths for each client on the Tegra SPI bus. These properties helps 
> > to tune the clock delays.
> > 
> > Signed-off-by: Sowjanya Komatineni <skomatineni@xxxxxxxxxx>
> > ---
> >  .../devicetree/bindings/spi/nvidia,tegra114-spi.txt      | 16 ++++++++++++++++
> >  1 file changed, 16 insertions(+)
>
> Just combine this with patch 19.
>
> > diff --git 
> > a/Documentation/devicetree/bindings/spi/nvidia,tegra114-spi.txt 
> > b/Documentation/devicetree/bindings/spi/nvidia,tegra114-spi.txt
> > index 6167c5234b64..2b84b7b726ce 100644
> > --- a/Documentation/devicetree/bindings/spi/nvidia,tegra114-spi.txt
> > +++ b/Documentation/devicetree/bindings/spi/nvidia,tegra114-spi.txt
> > @@ -29,6 +29,12 @@ spi-client device controller properties:
> >  - nvidia,cs-hold-clk-count: CS hold timing parameter.
> >  - nvidia,cs-inactive-cycles: CS inactive delay in terms of clock between
> >    transfers.
> > +- nvidia,tx-clk-tap-delay: Delays the clock going out to the external 
> > +device
> > +  with this tap value. This property is used to tune the outgoing 
> > +data from
> > +  Tegra SPI master with respect to outgoing Tegra SPI master clock.
> > +- nvidia,rx-clk-tap-delay: Delays the clock coming in from the 
> > +external device
> > +  with this tap value. This property is used to adjust the Tegra SPI 
> > +master
> > +  clock with respect to the data from the SPI slave device.
>
> Are there units? What's the range of values.

TX/RX Clock delays are tap counts and there is internal tap-to-tap delay.
Will update to specify tap to delay correlation along with supported tap values range.

As per feedback from mark, will move CS timing to API implementation as they are not
Tegra SPI specific and applicable in general.

>
> >  
> >  Example:
> >  
> > @@ -45,4 +51,14 @@ spi@7000d600 {
> >  	reset-names = "spi";
> >  	dmas = <&apbdma 16>, <&apbdma 16>;
> >  	dma-names = "rx", "tx";
> > +
> > +	<spi-client>@<bus_num> {
> > +		...
> > +		...
> > +		nvidia,cs-setup-clk-count = <10>;
> > +		nvidia,cs-hold-clk-count = <10>;
> > +		nvidia,rx-clk-tap-delay = <0>;
> > +		nvidia,tx-clk-tap-delay = <16>;
> > +		...
> > +	};
> >  };
> > --
> > 2.7.4
> >