[coreboot-gerrit] Change in coreboot[master]: drivers/intel/fsp2_0: Add FSP 2.2 specific support

[Subrata Banik (Code Review)]

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41728 )

Change subject: drivers/intel/fsp2_0: Add FSP 2.2 specific support
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/41728/1/src/drivers/intel/fsp2_0/si... 
File src/drivers/intel/fsp2_0/silicon_init.c:

https://review.coreboot.org/c/coreboot/+/41728/1/src/drivers/intel/fsp2_0/si... 
PS1, Line 100: 	multi_phase_params = xmalloc(sizeof(struct fsp_multi_phase_params));
             : 	multi_phase_get_number =
             : 			xmalloc(sizeof(struct fsp_multi_phase_get_number_of_phase_params));
> Do we really need to allocate heap for these? Seems like they could be local variables on the stack.

Ack