Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41062 )
Change subject: soc/intel/jasperlake: Apply FiVR related settings ......................................................................
Patch Set 14:
(1 comment)
https://review.coreboot.org/c/coreboot/+/41062/14/src/soc/intel/jasperlake/f... File src/soc/intel/jasperlake/fsp_params.c:
https://review.coreboot.org/c/coreboot/+/41062/14/src/soc/intel/jasperlake/f... PS14, Line 162: /* FiVR */ Is this configuration required even if FIVR is not used/enabled. Can you please mention why?