Re: [U-Boot] [U-Boot, RFC] ARM: create .secure_stack section only for PSCI

2016-10-14 Thread Tom Rini
On Mon, Sep 26, 2016 at 02:21:30PM +0900, Masahiro Yamada wrote:

> Jon Master reports that QEMU refuses to load a U-Boot image built
> with CONFIG_ARMV7_NONSEC, but without CONFIG_ARMV7_PSCI since
> commit 5a3aae68c74e ("ARM: armv7: guard memory reserve for PSCI
> with #ifdef CONFIG_ARMV7_PSCI").
> 
> It looks like only PSCI that needs the Secure stack, so move
> the #ifdef to guard the whole of .secure_stack allocation in order
> not to create the empty section.
> 
> Signed-off-by: Masahiro Yamada 
> Reported-by: Jon Masters 
> Link: http://patchwork.ozlabs.org/patch/664025/

Applied to u-boot/master, thanks!

... and no, I don't know how / why this caused build failures before,
but it did and I replicated it at the time too.  But I just got a
clean build everywhere.  And I've added vexpress-a15 and vexpress-a9 via
QEMU to my test setup now.

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [U-Boot, RFC] ARM: create .secure_stack section only for PSCI

2016-10-07 Thread Tom Rini
On Mon, Sep 26, 2016 at 02:21:30PM +0900, Masahiro Yamada wrote:

> Jon Master reports that QEMU refuses to load a U-Boot image built
> with CONFIG_ARMV7_NONSEC, but without CONFIG_ARMV7_PSCI since
> commit 5a3aae68c74e ("ARM: armv7: guard memory reserve for PSCI
> with #ifdef CONFIG_ARMV7_PSCI").
> 
> It looks like only PSCI that needs the Secure stack, so move
> the #ifdef to guard the whole of .secure_stack allocation in order
> not to create the empty section.
> 
> Signed-off-by: Masahiro Yamada 
> Reported-by: Jon Masters 
> Link: http://patchwork.ozlabs.org/patch/664025/

We're close I think, but this breaks boards like ls1021aqds_nor_lpuart

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot