[U-Boot] [PATCH] ARM: Specify aligned address for secure section instead of using attributes

Tom Rini trini at konsulko.com
Thu Sep 6 13:45:50 UTC 2018


On Thu, Sep 06, 2018 at 11:56:28AM +0800, Chen-Yu Tsai wrote:

> 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 at gmail.com>
> Signed-off-by: Chen-Yu Tsai <wens at csie.org>

Reviewed-by: Tom Rini <trini at konsulko.com>

After a little more time for review, should I take this directly or a PR
from the sunxi team?  Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180906/85cdad4e/attachment.sig>


More information about the U-Boot mailing list