[XS] Change in coreboot[main]: soc/intel/mtl: Increase CAR_STACK_SIZE by 31KB for coreboot compatibi...

[Subrata Banik (Code Review)]

Attention is currently required from: Eran Mitrani, Jakub Czapiga, Julius Werner, Kapil Porwal, Karthikeyan Ramasubramanian, Nico Huber, Paul Menzel, Rishika Raj, Tarun.

Subrata Banik has posted comments on this change by Rishika Raj. ( https://review.coreboot.org/c/coreboot/+/83540?usp=email )

Change subject: soc/intel/mtl: Increase CAR_STACK_SIZE by 31KB for coreboot compatibility
......................................................................

Patch Set 8:

(1 comment)

Patchset:

PS8:
> FWIW, Karthik just mentioned that the numbers in alderlake/Kconfig are actually wrong, and the FSP on that platform only requires 256K stack according to Intel's FSP integration guide (not 512K as the Kconfig help text says). It would be good if we could double-check all platforms to ensure our documentation is accurate.
>
> Do we have an FSP integration guide for MeteorLake as well? I couldn't find one on Drive. If the number there is also smaller than 512K maybe this patch isn't that necessary (it still shouldn't hurt, though).

Please look at my CL https://review.coreboot.org/c/coreboot/+/47837 , coreboot code for ADL/MTL is correct and FSP integration guide might not have the latest requirement captured. We shall bring this into FSP sync meeting.