[coreboot-gerrit] Change in coreboot[master]: soc/amd/common: Use SPI settings from EFS

[Felix Held (Code Review)]

Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43303 )

Change subject: soc/amd/common: Use SPI settings from EFS
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/c/coreboot/+/43303/4/src/soc/amd/common/block/sp... 
File src/soc/amd/common/block/spi/fch_spi.c:

https://review.coreboot.org/c/coreboot/+/43303/4/src/soc/amd/common/block/sp... 
PS4, Line 92: 			   *spi_speed, *spi_speed);
> I might not agree about using same max spi_speed for all fast,norm,alt and tpm here. […]

using the same value for all four is very likely wrong as the EFS only contains information on the SPI flash frequency and mode, but not about the other (e)SPI frequencies