aboutsummaryrefslogtreecommitdiff
path: root/bl1
diff options
context:
space:
mode:
authorMasahiro Yamada <yamada.masahiro@socionext.com>2020-03-26 10:57:12 +0900
committerMasahiro Yamada <yamada.masahiro@socionext.com>2020-04-02 13:30:17 +0900
commit0a0a7a9ac82cb79af91f098cedc69cc67bca3978 (patch)
treef20e3a4f33e02cafc1c6f888c2c6567938795ed4 /bl1
parent9fb288a03ed2ced7706defbbf78f008e921e17e2 (diff)
downloadtrusted-firmware-a-0a0a7a9ac82cb79af91f098cedc69cc67bca3978.tar.gz
linker_script: replace common read-only data with RODATA_COMMON
The common section data are repeated in many linker scripts (often twice in each script to support SEPARATE_CODE_AND_RODATA). When you add a new read-only data section, you end up with touching lots of places. After this commit, you will only need to touch bl_common.ld.h when you add a new section to RODATA_COMMON. Replace a series of RO section with RODATA_COMMON, which contains 6 sections, some of which did not exist before. This is not a big deal because unneeded data should not be compiled in the first place. I believe this should be controlled by BL*_SOURCES in Makefiles, not by linker scripts. When I was working on this commit, the BL1 image size increased due to the fconf_populator. Commit c452ba159c14 ("fconf: exclude fconf_dyn_cfg_getter.c from BL1_SOURCES") fixed this issue. I investigated BL1, BL2, BL2U, BL31 for plat=fvp, and BL2-AT-EL3, BL31, BL31 for plat=uniphier. I did not see any more unexpected code addition. Change-Id: I5d14d60dbe3c821765bce3ae538968ef266f1460 Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Diffstat (limited to 'bl1')
-rw-r--r--bl1/bl1.ld.S6
1 files changed, 2 insertions, 4 deletions
diff --git a/bl1/bl1.ld.S b/bl1/bl1.ld.S
index 87f1ae82f3..e706ce2866 100644
--- a/bl1/bl1.ld.S
+++ b/bl1/bl1.ld.S
@@ -45,8 +45,7 @@ SECTIONS
__RODATA_START__ = .;
*(SORT_BY_ALIGNMENT(.rodata*))
- PARSER_LIB_DESCS
- CPU_OPS
+ RODATA_COMMON
/*
* No need to pad out the .rodata section to a page boundary. Next is
@@ -67,8 +66,7 @@ SECTIONS
*(SORT_BY_ALIGNMENT(.text*))
*(SORT_BY_ALIGNMENT(.rodata*))
- PARSER_LIB_DESCS
- CPU_OPS
+ RODATA_COMMON
*(.vectors)
__RO_END__ = .;