[coreboot-gerrit] Change in coreboot[master]: soc/intel/jasperlake: Allow mainboard to override chip configuration

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45856 )

Change subject: soc/intel/jasperlake: Allow mainboard to override chip configuration
......................................................................

Patch Set 1:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/45856/1/src/soc/intel/jasperlake/fs... 
PS1, Line 94: override
Shouldn't this be run at the end so that it can actually override something?
This way it's more like the board can provide defaults and the code below
would have the last word.