[coreboot-gerrit] Change in ...coreboot[master]: drivers/intel/fsp2_0: Enable FSP feature to make use of same stack wi...

[Subrata Banik (Code Review)]

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

Change subject: drivers/intel/fsp2_0: Enable FSP feature to make use of same stack with bootloader
......................................................................

Patch Set 2:

(2 comments)

https://review.coreboot.org/#/c/32079/2//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/32079/2//COMMIT_MSG@7 
PS2, Line 7: drivers/intel/fsp2_0: Enable FSP feature to make use of same stack with bootloader
> Please short the summary.
> > drivers/intel/fsp2_0: Use same stack with bootloader

Done
> What is the bootloader? GRUB is a bootloader for me.

referring to coreboot here

https://review.coreboot.org/#/c/32079/2//COMMIT_MSG@10 
PS2, Line 10:
> Please describe the problem in more detail.

its not a problem for say, its feature request in FSP and coreboot need to adhere the same.
We were seeing some issue as described in test section which is fixed now with this CL.