[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/braswell: add default option to use public FSP

[Frans Hendriks (Code Review)]

Frans Hendriks has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32381 )

Change subject: soc/intel/braswell: add default option to use public FSP
......................................................................

Patch Set 6: Code-Review-1

(1 comment)

I2CxFrequency must be removed from Braswell and moved to MR1 mainboard directory.

Please have a look into patch: https://review.coreboot.org/c/coreboot/+/29661

https://review.coreboot.org/#/c/32381/6/src/soc/intel/braswell/chip.c 
File src/soc/intel/braswell/chip.c:

https://review.coreboot.org/#/c/32381/6/src/soc/intel/braswell/chip.c@172 
PS6, Line 172: 	params->I2C0Frequency = config->I2C0Frequency;
I2C0Frequency -- I2C6Frequency do not exist in SILICON_INIT_UPD of FSP MR2 
3rdparty Braswell FspUpdVpd.h