[coreboot-gerrit] [M] Change in coreboot[master]: mb/google/skyrim: Override SPI flash bus speed

[Karthik Ramasubramanian (Code Review)]

Attention is currently required from: Jason Nien, Jon Murphy, Martin Roth, Tim Van Patten.

Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68116 )

Change subject: mb/google/skyrim: Override SPI flash bus speed
......................................................................

Patch Set 1:

(1 comment)

File src/mainboard/google/skyrim/Kconfig:

https://review.coreboot.org/c/coreboot/+/68116/comment/66c082a3_9f53871e 
PS1, Line 96: 	default 0x5 if BOARD_GOOGLE_SKYRIM
            : 	default 0xffffffff
> Does this mean 100mhz only works on skyrim, and not any of the variants? […]

Even on skyrim, it works only on the board versions where the stubs to debug header is removed to meet SI requirements.

Similar exercise has to be done for other variants before enabling 100 MHz on them. Usually hardware team work along with the partner to measure the SI and then pass the bug to SW team to enable it.