[U-Boot-Users] [PATCH] SPI Flash: Support the ST Microelectronics M25P80 and M25P40 SPI Flash
Wolfgang Denk
wd at denx.de
Tue Jun 10 12:01:53 CEST 2008
In message <20080610024534.77FA14813D at mcmullan-linux.hq.netapp.com> you wrote:
> This commit adds MTD support for the M25P80 (1Mx8) and the M25P40 (512kx8)
> SPI Flash components from ST Microelectronics.
...
> --- /dev/null
> +++ b/drivers/mtd/spi/stmicro.c
> @@ -0,0 +1,326 @@
> +/*
> + * $Id$
> + *
Please remove these two lines.
> + do {
> + ret = spi_xfer(spi, 8, NULL, &status, 0);
> + if (ret)
> + return -1;
> +
> + if ((status & STMICRO_SR_WIP) == 0)
> + break;
> +
> + } while (1 || get_timer(timebase) < timeout);
Hm... the timeout code seems to be disabled here?
I think you should remove either the first or the second expression.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Mistakes are often the stepping stones to utter failure.
More information about the U-Boot
mailing list