[coreboot-gerrit] Change in coreboot[master]: mb/google/hatch: Change SerialIoDevMode for I2C, SPI, and UART

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34246 )

Change subject: mb/google/hatch: Change SerialIoDevMode for I2C, SPI, and UART
......................................................................

Patch Set 1:

(1 comment)

I

https://review.coreboot.org/c/coreboot/+/34246/1/src/mainboard/google/hatch/... 
File src/mainboard/google/hatch/variants/hatch/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/34246/1/src/mainboard/google/hatch/... 
PS1, Line 4: PchSerialIoSkipInit
It looks like PchSerialIoSkipInit is not a valid setting for I2C and SPI. See https://review.coreboot.org/cgit/coreboot.git/tree/src/vendorcode/intel/fsp/....

We need to update soc common code to ensure that it does not allow this setting.