[U-Boot] ARM: always perform per-CPU GIC init

Tom Rini trini at konsulko.com
Thu Apr 21 13:25:35 CEST 2016


On Fri, Apr 15, 2016 at 02:40:37PM -0600, Stephen Warren wrote:

> From: Stephen Warren <swarren at nvidia.com>
> 
> The current code in ARMv8's lowlevel_init() skips the per-CPU GIC
> initialization ifndef CONFIG_ARMV8_MULTIENTRY. However, the per-CPU init
> should always occur; it's just the one-time init that should only happen
> on the master.
> 
> Once this consideration is taken into account, the only difference between
> the paths when CONFIG_ARMV8_MULTIENTRY is undefined/defined is the use of
> branch_if_slave. Naively, any unified code would need to invoke this
> conditionally upon ifdef CONFIG_ARMV8_MULTIENTRY. However, branch_if_slave
> already checks CONFIG_ARMV8_MULTIENTRY and does nothing if it isn't
> defined, so we don't even need that ifdef at the call site.
> 
> Reported-by: Masahiro Yamada <yamada.masahiro at socionext.com>
> Signed-off-by: Stephen Warren <swarren at nvidia.com>
> Reviewed-by: Masahiro Yamada <yamada.masahiro at socionext.com>

Applied to u-boot/master, thanks!

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


More information about the U-Boot mailing list