[coreboot-gerrit] Change in coreboot[master]: mb/google/skyrim: Update SPI settings for skyrim

[Karthik Ramasubramanian (Code Review)]

Attention is currently required from: Raul Rangel, Jon Murphy, Chris Wang, Felix Held.
Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63717 )

Change subject: mb/google/skyrim: Update SPI settings for skyrim
......................................................................

Patch Set 1:

(2 comments)

File src/mainboard/google/skyrim/Kconfig:

https://review.coreboot.org/c/coreboot/+/63717/comment/c5dd5b63_f60223e9 
PS1, Line 78: EFS_SPI_READ_MODE
> PSP doesn't use EFS I thought?

Yes, it doesn't. But both bootblock and PSP verstage will be using speed from EFS to configure.

PSP team is working to not override the speed configuration for Chrome platforms.

https://review.coreboot.org/c/coreboot/+/63717/comment/334f7013_cd00774a 
PS1, Line 79: Dual IO (1-1-2)
> Check with Johnathan... I would like to use 1-2-2, but not sure the CPLD supports it.

This bug indicates CPLD supports only 1-1-2 - b/225213679#comment18