On Thursday 02 April 2009 07:20:13 Haavard Skinnemoen wrote: > Mike Frysinger wrote: > > - /* Up to 2 seconds */ > > - ret = stmicro_wait_ready(flash, 2 * CONFIG_SYS_HZ); > > + ret = stmicro_wait_ready(flash, SPI_FLASH_PAGE_ERASE_TIMEOUT); > > 50 ms is an awful lot less than 2 seconds. Sure this is safe?
the 2 sec mark was copied in all spi flash drivers based on the original port rather than referring to a datasheet, and it works on my Blackfin boards that have stmicro parts. personally i think the timeouts in the current spi flash common code is too low in general, so i'd propose we raise them. after all, the timeouts only matter in when something goes wrong, so they wouldnt be reached. and the low threshold seems like it makes the presumption of faster SPI bus speeds. i.e. how about raising the timeout values 50x or 100x ? -mike
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot