[coreboot-gerrit] New patch to review for coreboot: 85bba3e arm64: remove _stack and _estack symbols in linker script

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Thu Mar 26 10:56:12 CET 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/9024

-gerrit

commit 85bba3e8d33cb1dd39f98ffa6d972bb517d888a5
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Fri Aug 29 10:01:39 2014 -0500

    arm64: remove _stack and _estack symbols in linker script
    
    These symbols should have been removed with the stack
    refactoring. I'm not sure how it was missed.
    
    BUG=chrome-os-partner:31545
    BRANCH=None
    TEST=Built and booted into kernel with both cpus.
    
    Change-Id: Ia6c2103d7b5e2c9d74cdc5d1b5f42f8954812231
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: d9432b5cf0cce3bfdbfd5371fb3280e3cc746a42
    Original-Change-Id: I17bc9a7aaaf133f427b15f803a6003fa2ca8f8a6
    Original-Signed-off-by: Aaron Durbin <adurbin at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/215541
---
 src/arch/arm64/ramstage.ld | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/src/arch/arm64/ramstage.ld b/src/arch/arm64/ramstage.ld
index c1c2221..d767778 100644
--- a/src/arch/arm64/ramstage.ld
+++ b/src/arch/arm64/ramstage.ld
@@ -105,10 +105,6 @@ SECTIONS
 		_eheap = .;
 	}
 
-	/* arm64 chipsets need to define CONFIG_RAMSTAGE_STACK_(TOP|BOTTOM) */
-	_stack = CONFIG_RAMSTAGE_STACK_BOTTOM;
-	_estack = CONFIG_RAMSTAGE_STACK_TOP;
-
 	/* The ram segment. This includes all memory used by the memory
 	 * resident copy of coreboot, except the tables that are produced on
 	 * the fly, but including stack and heap.



More information about the coreboot-gerrit mailing list