Martin Roth has submitted this change. ( https://review.coreboot.org/c/coreboot/+/52688 )
Change subject: psp_verstage: make temp_stack optional ......................................................................
psp_verstage: make temp_stack optional
Temp stack for verstage is only needed for picasso, so make it optional in the layout file.
Signed-off-by: Kangheui Won khwon@chromium.org Change-Id: I44196103a3531e9d01c96ab8f454c8b580fe9807 Reviewed-on: https://review.coreboot.org/c/coreboot/+/52688 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Martin Roth martinroth@google.com --- M src/soc/amd/common/block/cpu/noncar/memlayout_psp_verstage.ld 1 file changed, 2 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Martin Roth: Looks good to me, approved
diff --git a/src/soc/amd/common/block/cpu/noncar/memlayout_psp_verstage.ld b/src/soc/amd/common/block/cpu/noncar/memlayout_psp_verstage.ld index cc9ebc0..f386823 100644 --- a/src/soc/amd/common/block/cpu/noncar/memlayout_psp_verstage.ld +++ b/src/soc/amd/common/block/cpu/noncar/memlayout_psp_verstage.ld @@ -29,7 +29,9 @@
#include "memlayout_transfer_buffer.inc"
+#if defined(PSP_VERSTAGE_TEMP_STACK_START) PSP_VERSTAGE_TEMP_STACK_END = (PSP_VERSTAGE_TEMP_STACK_START + PSP_VERSTAGE_TEMP_STACK_SIZE ); +#endif
REGION(stack, PSP_VERSTAGE_STACK_START, PSP_VERSTAGE_STACK_SIZE, 64) PSP_VERSTAGE_STACK_BASE = _stack;