diff options
| author | Chen-Yu Tsai <wens@csie.org> | 2018-09-06 11:56:28 +0800 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2018-09-07 09:11:42 -0400 |
| commit | 792b204798453d96b00e0817e8472c19455e92a2 (patch) | |
| tree | 39bb1e47c2e830f25b71fd475a161409d07cbd17 /arch/arm/cpu/u-boot.lds | |
| parent | 4cdeda511f8037015b568396e6dcc3d8fb41e8c0 (diff) | |
| download | u-boot-792b204798453d96b00e0817e8472c19455e92a2.zip u-boot-792b204798453d96b00e0817e8472c19455e92a2.tar.gz u-boot-792b204798453d96b00e0817e8472c19455e92a2.tar.bz2 | |
ARM: Specify aligned address for secure section instead of using attributes
In commit a1274cc94a20 ("ARM: Page align secure section only when it is
executed in situ"), we used output section attributes (the "ALIGN"
keyword after the colon) to specify the alignment requirements. Using
the constant "COMMONPAGE" there was recently broken in binutils 2.31 [1].
Binutils maintainer Alan Modra suggested the former method would still
work. Since both methods achieve the same result, this patch does just
that. This fixes the "reboot after bootm" issue we've been seeing on
sunxi when booting non-secure.
[1] https://sourceware.org/bugzilla/show_bug.cgi?id=23571
Suggested-by: Alan Modra <amodra@gmail.com>
Signed-off-by: Chen-Yu Tsai <wens@csie.org>
Reviewed-by: Tom Rini <trini@konsulko.com>
Acked-by: Jagan Teki <jagan@openedev.com>
Diffstat (limited to 'arch/arm/cpu/u-boot.lds')
| -rw-r--r-- | arch/arm/cpu/u-boot.lds | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/arm/cpu/u-boot.lds b/arch/arm/cpu/u-boot.lds index 834dc99..0eb164d 100644 --- a/arch/arm/cpu/u-boot.lds +++ b/arch/arm/cpu/u-boot.lds @@ -68,11 +68,11 @@ SECTIONS #ifdef CONFIG_ARMV7_NONSEC /* Align the secure section only if we're going to use it in situ */ - .__secure_start : + .__secure_start #ifndef CONFIG_ARMV7_SECURE_BASE ALIGN(CONSTANT(COMMONPAGESIZE)) #endif - { + : { KEEP(*(.__secure_start)) } |