[U-Boot] [PATCH 1/6] ARM: add secure monitor handler to switch to non-secure state

Albert ARIBAUD albert.u.boot at aribaud.net
Fri May 31 19:21:11 CEST 2013


Hi Andre,

On Fri, 31 May 2013 11:23:16 +0200, Andre Przywara
<andre.przywara at linaro.org> wrote:

> software_interrupt is currently a panic routine. So it is not actually 
> used by u-boot, it's just there to dump some state and eventually call 
> reset_cpu().
> So I feel that since I am now the only user of software_interrupt I 
> don't need any special precautions and consider this routine now 
> actually part of the HYP mode switcher. But of course I can retain the 
> original "functionality" if CONFIG_ARMV7_VIRT is not defined.

You should, actually. Rule #1 goes (*) 'a new config option should
have zero effect on the code when not defined'.

> Regards,
> Andre.

(*) Rule #1 actually goes, 'there is an infinite number of "Rule #1"',
but that's slightly beyond the point.

Amicalement,
-- 
Albert.


More information about the U-Boot mailing list