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

[Maulik V Vaghela (Code Review)]

Maulik V Vaghela 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;
> But this can set 0 right? I think FSP default is 0, so I think it is okay to check non-zero.

HI Eric,

If we don't fill this via devicetree, default value will be 0 in config. Shouldn't this be fine?