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

Jagan Teki jagan at amarulasolutions.com
Fri Sep 7 04:32:47 UTC 2018


On Thu, Sep 6, 2018 at 7:15 PM, Tom Rini <trini at konsulko.com> wrote:
> 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>

Acked-by: Jagan Teki <jagan at openedev.com>

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

As of now, I don't have any patches for the release please take it directly.


More information about the U-Boot mailing list