From: Huan Wang <alison.wang@nxp.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] armv8: Remove the codes about switching to EL1 before jumping to kernel
Date: Mon, 18 Jul 2016 03:24:06 +0000 [thread overview]
Message-ID: <DB5PR04MB21525FB2EDC681416F886376F4360@DB5PR04MB2152.eurprd04.prod.outlook.com> (raw)
In-Reply-To: <CAHTX3dKC7+WQSP8exucqTBBJLf_be2s0Ss7wg7vr8ke-28tcfQ@mail.gmail.com>
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
next prev parent reply other threads:[~2016-07-18 3:24 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-07-07 6:25 [U-Boot] [PATCH] armv8: Remove the codes about switching to EL1 before jumping to kernel Alison Wang
2016-07-07 11:39 ` Ryan Harkin
2016-07-07 11:44 ` Alexander Graf
2016-07-07 17:31 ` Michal Simek
2016-07-18 3:24 ` Huan Wang [this message]
2016-07-18 6:22 ` Alexander Graf
2016-08-29 9:29 ` Huan Wang
2016-09-01 13:56 ` Alexander Graf
2016-09-02 5:27 ` Huan Wang
2016-09-02 5:32 ` Huan Wang
2016-09-05 8:24 ` Huan Wang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=DB5PR04MB21525FB2EDC681416F886376F4360@DB5PR04MB2152.eurprd04.prod.outlook.com \
--to=alison.wang@nxp.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.