[coreboot-gerrit] Change in coreboot[master]: soc/intel/jasperlake: Enable Intel FIVR RFI settings

[EricR Lai (Code Review)]

EricR Lai has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47286 )

Change subject: soc/intel/jasperlake: Enable Intel FIVR RFI settings
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/47286/2/src/soc/intel/jasperlake/fs... 
File src/soc/intel/jasperlake/fsp_params.c:

https://review.coreboot.org/c/coreboot/+/47286/2/src/soc/intel/jasperlake/fs... 
PS2, Line 220: 	params->FivrRfiFrequency = config->FivrRfiFrequency;
> HI Eric, […]

If FivrSpreadSpectrum default is 0, this should be fine. In general, it better to honor FSP setting if device tree is NULL. I have no concern here, your call.