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

[Furquan Shaikh (Code Review)]

Furquan Shaikh 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 feature? I believe the feature was already enabled here CB:28358, just that the stack base pointer was set incorrectly?