diff options
| author | Frans Hendriks <fhendriks@eltan.com> | 2020-11-19 08:10:47 +0100 |
|---|---|---|
| committer | Arthur Heymans <arthur@aheymans.xyz> | 2020-11-20 09:21:45 +0000 |
| commit | 7ba970aa78816dd29052cd34304b6ea67953c003 (patch) | |
| tree | 352dd79f6d0f3c6c28bb32be4c4d11f516023dfd /src | |
| parent | 2832d11dd1aa59a195c67296a2a39ae4689b74eb (diff) | |
| download | coreboot-7ba970aa78816dd29052cd34304b6ea67953c003.tar.xz | |
drivers/intel/fsp1_1/cache_as_ram.S: Use _car_stack area for stack
Top of Temp RAM is used as bootloader stack, which is the
_car_region_end area. This area is not equal to CAR stack area as
defined in car.ld file.
Use _ecar_stack (end of CAR stack) as starting stack location.
Tested VBOOT, Vendorboot security and no security on Facebook FBG1701.
Change-Id: I16b077f60560de334361b1f0d3758ab1a5cbe895
Signed-off-by: Frans Hendriks <fhendriks@eltan.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/47737
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Wim Vervoorn <wvervoorn@eltan.com>
Reviewed-by: Michał Żygowski <michal.zygowski@3mdeb.com>
Reviewed-by: Christian Walter <christian.walter@9elements.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src')
| -rw-r--r-- | src/drivers/intel/fsp1_1/cache_as_ram.S | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/drivers/intel/fsp1_1/cache_as_ram.S b/src/drivers/intel/fsp1_1/cache_as_ram.S index 31c3580aac..3be9eb92df 100644 --- a/src/drivers/intel/fsp1_1/cache_as_ram.S +++ b/src/drivers/intel/fsp1_1/cache_as_ram.S @@ -133,7 +133,7 @@ CAR_init_done: jne halt2 /* Setup bootloader stack */ - movl %edx, %esp + movl $_ecar_stack, %esp /* * ebp: FSP_INFO_HEADER address |