[U-Boot] [UBOOT][PATCH 5/5] am43xx: add delay before xfer

[Jagan Teki]

On Thu, Nov 14, 2013 at 9:09 PM, Sourav Poddar <sourav.poddar at ti.com> wrote:
> Without this delay, write/read is failing.
> Looks like, the WIP always remain set and hence a timeout
> occurs leading to the error.
>
> Signed-off-by: Sourav Poddar <sourav.poddar at ti.com>
> ---
> Hi Jagan,
> This patch seems to be necessary for read/write.
> I tested by changing few timing variables, but it did not help.
> The same driver works on J6 with a differnet flash(S25FL256S).
> Is any one tested macronix flash at uboot?

Delay must be for macronix?
or common for all flashes.

I haven't tested macronix

>
>  drivers/spi/ti_qspi.c |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/spi/ti_qspi.c b/drivers/spi/ti_qspi.c
> index 5666250..aa7b6ae 100644
> --- a/drivers/spi/ti_qspi.c
> +++ b/drivers/spi/ti_qspi.c
> @@ -285,6 +285,9 @@ int spi_xfer(struct spi_slave *slave, unsigned int bitlen, const void *dout,
>                 qslave->cmd |= QSPI_3_PIN;
>         qslave->cmd |= 0xfff;
>
> +#ifdef CONFIG_AM43XX
> +       udelay(100);
> +#endif
>         while (words--) {
>                 if (txp) {
>                         debug("tx cmd %08x dc %08x data %02x\n",
> --
> 1.7.1
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot



-- 
Thanks,
Jagan.
--------
Jagannadha Sutradharudu Teki,
E: jagannadh.teki at gmail.com, P: +91-9676773388
Engineer - System Software Hacker
U-boot - SPI Custodian and Zynq APSOC
Ln: http://www.linkedin.com/in/jaganteki