From mboxrd@z Thu Jan  1 00:00:00 1970
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
Date: Fri, 31 May 2013 19:21:11 +0200
Subject: [U-Boot] [PATCH 1/6] ARM: add secure monitor handler to switch
 to non-secure state
In-Reply-To: <51A86C04.3020500@linaro.org>
References: <1367846270-1827-1-git-send-email-andre.przywara@linaro.org>
	<1367846270-1827-2-git-send-email-andre.przywara@linaro.org>
	<20130531010213.GA3189@ubuntu> <51A86C04.3020500@linaro.org>
Message-ID: <20130531192111.68fa2c1d@lilith>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Andre,

On Fri, 31 May 2013 11:23:16 +0200, Andre Przywara
<andre.przywara@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.