From mboxrd@z Thu Jan  1 00:00:00 1970
From: Huan Wang <alison.wang@nxp.com>
Date: Mon, 18 Jul 2016 03:24:06 +0000
Subject: [U-Boot] [PATCH] armv8: Remove the codes about switching to EL1
 before jumping to kernel
In-Reply-To: <CAHTX3dKC7+WQSP8exucqTBBJLf_be2s0Ss7wg7vr8ke-28tcfQ@mail.gmail.com>
References: <1467872700-45942-1-git-send-email-b18965@freescale.com>
 <577E4092.8070007@suse.de>
 <CAHTX3dKC7+WQSP8exucqTBBJLf_be2s0Ss7wg7vr8ke-28tcfQ@mail.gmail.com>
Message-ID: <DB5PR04MB21525FB2EDC681416F886376F4360@DB5PR04MB2152.eurprd04.prod.outlook.com>
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, Alex,

              As there is strong objection to remove the codes about switching to EL1, I think we have to remain it, do you agree?

                If it is remained, I think your suggestion about *always* jumping to ep for both switching to AArch64 and AArch32 modes will make the code hard to realize and very complicated. So I prefer to  keep the process in v4 patches. What is your opinion?


Best Regards,
Alison Wang

From: Michal Simek [mailto:monstr at monstr.eu]
Sent: Friday, July 08, 2016 1:31 AM
To: Alexander Graf
Cc: Alison Wang; york sun; Scott Wood; Stuart Yoder; Yang-Leo Li; David Feng; Linus Walleij; ryan.harkin at linaro.org; U-Boot; Zhengxiong Jin
Subject: Re: [U-Boot] [PATCH] armv8: Remove the codes about switching to EL1 before jumping to kernel

Hi,

2016-07-07 13:44 GMT+02:00 Alexander Graf <agraf at suse.de<mailto:agraf@suse.de>>:
On 07/07/2016 08:25 AM, Alison Wang wrote:
As CONFIG_ARMV8_SWITCH_TO_EL1 is not used now, this patch is to remove
CONFIG_ARMV8_SWITCH_TO_EL1 and the corresponding functions.

Signed-off-by: Alison Wang <alison.wang at nxp.com<mailto:alison.wang@nxp.com>>

I'll CC the maintainers for ZynqMP and Exynos as well, but from my side this patch is a great step forward.

Reviewed-by: Alexander Graf <agraf at suse.de<mailto:agraf@suse.de>>


I am against this patch. The reason is simple. We are using this option for testing to make sure that hyperviser and OS
behaves correctly when they start from different level.
Thanks,
Michal

--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu<http://www.monstr.eu> p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform