Change in ...coreboot[master]: drivers/intel/fsp2_0: Use same stack with coreboot

[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: Use same stack with coreboot
......................................................................


Patch Set 3:

(1 comment)

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

https://review.coreboot.org/#/c/32079/3//COMMIT_MSG@9 
PS3, Line 9: This patch ensures to have same stack base for FSP and coreboot.
           : 
           : Feature added in FSP2.1
           : - Remove stack swapping from FSP.
           : - Stack will be shared between coreboot and FSP.
> Isn't this CL actually correcting the stack base pointer passed into FSP rather than enabling the fe […]
> that was the actual intention at that time but if you calculate stack usage in CB:28358 then you will realize it was using still using dedicated stack for cb and fsp. unless we point the same stack base, it can't make use of same stack. hence i have given commit title as "make use of same stack" and in description given the correct expectation from this CL