[U-Boot] [PATCH 2/6] exynos: spi: Fix calculation of SPI transaction start time

[Ajay kumar]

On Mon, Jul 7, 2014 at 9:46 PM, Simon Glass <sjg at chromium.org> wrote:
> The SPI transaction delay is supposed to be measured from the end of one
> transaction to the start of the next. The code does not work that way, so
> fix it.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
>  drivers/spi/exynos_spi.c | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/spi/exynos_spi.c b/drivers/spi/exynos_spi.c
> index c92276f..2969184 100644
> --- a/drivers/spi/exynos_spi.c
> +++ b/drivers/spi/exynos_spi.c
> @@ -428,10 +428,6 @@ void spi_cs_activate(struct spi_slave *slave)
>         clrbits_le32(&spi_slave->regs->cs_reg, SPI_SLAVE_SIG_INACT);
>         debug("Activate CS, bus %d\n", spi_slave->slave.bus);
>         spi_slave->skip_preamble = spi_slave->mode & SPI_PREAMBLE;
> -
> -       /* Remember time of this transaction so we can honour the bus delay */
> -       if (spi_slave->bus->deactivate_delay_us)
> -               spi_slave->last_transaction_us = timer_get_us();
>  }
>
>  /**
> @@ -445,6 +441,11 @@ void spi_cs_deactivate(struct spi_slave *slave)
>         struct exynos_spi_slave *spi_slave = to_exynos_spi(slave);
>
>         setbits_le32(&spi_slave->regs->cs_reg, SPI_SLAVE_SIG_INACT);
> +
> +       /* Remember time of this transaction so we can honour the bus delay */
> +       if (spi_slave->bus->deactivate_delay_us)
> +               spi_slave->last_transaction_us = timer_get_us();
> +
>         debug("Deactivate CS, bus %d\n", spi_slave->slave.bus);
>  }
>
> --
> 2.0.0.526.g5318336
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
Able to test display on peach_pit(which needs tps65090) with these patches.
Also, crosec commands from u-boot prompt works as expected on peach_pit.

Tested-by: Ajay Kumar <ajaykumar.rs at samsung.com>