Attention is currently required from: Arthur Heymans, Subrata Banik, Wonkyu Kim.

Appukuttan V K uploaded patch set #13 to this change.

View Change

The following approvals got outdated and were removed: Code-Review+1 by Subrata Banik, Code-Review+1 by Wonkyu Kim, Verified+1 by build bot (Jenkins)

drivers/intel/fsp2_0: Make coreboot FSP stack 16-bytes aligned

- Stack alignment:

1. FSP functions must be called with the stack 16-bytes aligned
in x86_64 mode.This is already setup properly with the default
value of the `mpreferred-stack-boundary' compiler option (4).
Similarly for x86_32 it is set to 2 through compiler option.

2. The FSP heap buffer supplied by coreboot through the `StackBase'
UPD must be 16-bytes aligned for x86_64 and 4-bytes aligned for
x86_32.

BUG=b:329034258
TEST=Verified on Meteor Lake board (Rex)

Change-Id: I86048c5d3623a29f17a5e492cd67568e4844589c
Signed-off-by: Appukuttan V K <appukuttan.vk@intel.com>
---
M src/drivers/intel/fsp2_0/memory_init.c
1 file changed, 5 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/61/81661/13

To view, visit change 81661. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I86048c5d3623a29f17a5e492cd67568e4844589c
Gerrit-Change-Number: 81661
Gerrit-PatchSet: 13
Gerrit-Owner: Appukuttan V K <appukuttan.vk@intel.com>
Gerrit-Reviewer: Andrey Petrov <andrey.petrov@gmail.com>
Gerrit-Reviewer: Ronak Kanabar <ronak.kanabar@intel.com>
Gerrit-Reviewer: Subrata Banik <subratabanik@google.com>
Gerrit-Reviewer: Wonkyu Kim <wonkyu.kim@intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-CC: Paul Menzel <paulepanter@mailbox.org>
Gerrit-Attention: Wonkyu Kim <wonkyu.kim@intel.com>
Gerrit-Attention: Subrata Banik <subratabanik@google.com>
Gerrit-Attention: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-MessageType: newpatchset