summaryrefslogtreecommitdiff
path: root/bl32/sp_min
diff options
context:
space:
mode:
authorMasahiro Yamada <yamada.masahiro@socionext.com>2020-04-22 10:50:12 +0900
committerMasahiro Yamada <yamada.masahiro@socionext.com>2020-04-25 20:09:08 +0900
commitcaa3e7e0a4aeb657873bbd2c002c0e33a614eb1d (patch)
treeeefef8ca9add77dbd7524741d632b367d1d0f99f /bl32/sp_min
parenta926a9f60aa94a034b0a06eed296996363245d30 (diff)
downloadarm-trusted-firmware-caa3e7e0a4aeb657873bbd2c002c0e33a614eb1d.tar.gz
linker_script: move .data section to bl_common.ld.h
Move the data section to the common header. I slightly tweaked some scripts as follows: [1] bl1.ld.S has ALIGN(16). I added DATA_ALIGN macro, which is 1 by default, but overridden by bl1.ld.S. Currently, ALIGN(16) of the .data section is redundant because commit 412865907699 ("Fix boot failures on some builds linked with ld.lld.") padded out the previous section to work around the issue of LLD version <= 10.0. This will be fixed in the future release of LLVM, so I am keeping the proper way to align LMA. [2] bl1.ld.S and bl2_el3.ld.S define __DATA_RAM_{START,END}__ instead of __DATA_{START,END}__. I put them out of the .data section. [3] SORT_BY_ALIGNMENT() is missing tsp.ld.S, sp_min.ld.S, and mediatek/mt6795/bl31.ld.S. This commit adds SORT_BY_ALIGNMENT() for all images, so the symbol order in those three will change, but I do not think it is a big deal. Change-Id: I215bb23c319f045cd88e6f4e8ee2518c67f03692 Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Diffstat (limited to 'bl32/sp_min')
-rw-r--r--bl32/sp_min/sp_min.ld.S6
1 files changed, 1 insertions, 5 deletions
diff --git a/bl32/sp_min/sp_min.ld.S b/bl32/sp_min/sp_min.ld.S
index 8e91cec91..9e0596f1f 100644
--- a/bl32/sp_min/sp_min.ld.S
+++ b/bl32/sp_min/sp_min.ld.S
@@ -91,11 +91,7 @@ SECTIONS
*/
__RW_START__ = . ;
- .data . : {
- __DATA_START__ = .;
- *(.data*)
- __DATA_END__ = .;
- } >RAM
+ DATA_SECTION >RAM
#ifdef BL32_PROGBITS_LIMIT
ASSERT(. <= BL32_PROGBITS_LIMIT, "BL32 progbits has exceeded its limit.")