Change in ...coreboot[master]: drivers/intel/fsp1_0: Deduplicate code

[Werner Zeh (Code Review)]

Werner Zeh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31713 )

Change subject: drivers/intel/fsp1_0: Deduplicate code
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/#/c/31713/1/src/drivers/intel/fsp1_0/fsp_util.c 
File src/drivers/intel/fsp1_0/fsp_util.c:

https://review.coreboot.org/#/c/31713/1/src/drivers/intel/fsp1_0/fsp_util.c@... 
PS1, Line 109: oid ChipsetFspReturnPoint(EFI_STATUS Status, VOID *HobListPtr)
Once there is no external caller more for this function should we make it static and remove it from fsp_util.h completely? We even can rename it to something like fsp_return_point() or the like since it is not more specific for a given chipset but is more a common function for fsp1.0.