[coreboot-gerrit] Change in coreboot[master]: include/spi-generic: moving common flash timeouts

[Werner Zeh (Code Review)]

Werner Zeh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34345 )

Change subject: include/spi-generic: moving common flash timeouts
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/c/coreboot/+/34345/3/src/include/spi-generic.h 
File src/include/spi-generic.h:

https://review.coreboot.org/c/coreboot/+/34345/3/src/include/spi-generic.h@2... 
PS3, Line 26: #define SPI_FLASH_SECTOR_ERASE_TIMEOUT	(10 * CONF_SYS_HZ)
> Hz is a unit. Specifically, it is 1/second (the inverse of a second). […]

Yes, we stumbled over this _HZ naming already.
I guess we should change the defines into something like ..._TIMEOUT_MS and drop the define for CONF_SYS_HZ 100 completely. It seems not to be used elsewhere anyway.