diff mbox series

[U-Boot] vexpress/aemv8a: drop CONFIG_ARMV8_SWITCH_TO_EL1

Message ID 20190821172910.13921-2-sudeep.holla@arm.com
State Accepted
Commit 208bdaf2ae5f31cf8bd20beedd36686e8624d52b
Delegated to: Tom Rini
Headers show
Series [U-Boot] vexpress/aemv8a: drop CONFIG_ARMV8_SWITCH_TO_EL1 | expand

Commit Message

Sudeep Holla Aug. 21, 2019, 5:29 p.m. UTC
To support KVM, we need to drop at EL2 and not EL1 before we boot Linux
kernel. This causes issues on platform with VHE and secondaries booting
at EL2 via TF-A PSCI CPU_ON call.

Cc: Ryan Harkin <ryan.harkin@linaro.org>
Cc: Liviu Dudau <liviu.dudau@foss.arm.com>
Cc: Linus Walleij <linus.walleij@linaro.org>
Cc: David Feng <fenghua@phytium.com.cn>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
---
 include/configs/vexpress_aemv8a.h | 1 -
 1 file changed, 1 deletion(-)

Comments

Peng Fan Aug. 22, 2019, 1:24 a.m. UTC | #1
> Subject: [U-Boot] [PATCH] vexpress/aemv8a: drop
> CONFIG_ARMV8_SWITCH_TO_EL1
> 
> To support KVM, we need to drop at EL2 and not EL1 before we boot Linux
> kernel. This causes issues on platform with VHE and secondaries booting at
> EL2 via TF-A PSCI CPU_ON call.
> 
> Cc: Ryan Harkin <ryan.harkin@linaro.org>
> Cc: Liviu Dudau <liviu.dudau@foss.arm.com>
> Cc: Linus Walleij <linus.walleij@linaro.org>
> Cc: David Feng <fenghua@phytium.com.cn>
> Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
> ---
>  include/configs/vexpress_aemv8a.h | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/include/configs/vexpress_aemv8a.h
> b/include/configs/vexpress_aemv8a.h
> index 89cd8b550ae6..974647055221 100644
> --- a/include/configs/vexpress_aemv8a.h
> +++ b/include/configs/vexpress_aemv8a.h
> @@ -11,7 +11,6 @@
>  #ifndef CONFIG_SEMIHOSTING
>  #error CONFIG_TARGET_VEXPRESS64_BASE_FVP requires
> CONFIG_SEMIHOSTING  #endif -#define CONFIG_ARMV8_SWITCH_TO_EL1
> #endif
> 
>  #define CONFIG_REMAKE_ELF

Reviewed-by: Peng Fan <peng.fan@nxp.com>

> --
> 2.17.1
> 
> _______________________________________________
> U-Boot mailing list
> U-Boot@lists.denx.de
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.d
> enx.de%2Flistinfo%2Fu-boot&amp;data=02%7C01%7CPeng.Fan%40nxp.com
> %7C68ad94406f6942be9d5408d7265d3489%7C686ea1d3bc2b4c6fa92cd99c
> 5c301635%7C0%7C0%7C637020054108251189&amp;sdata=k%2B4Fod6oxv
> 01%2BrKzUmUCR8I5WSVXmiyptJ%2B%2FIzRmlYg%3D&amp;reserved=0
Tom Rini Aug. 27, 2019, 12:18 a.m. UTC | #2
On Wed, Aug 21, 2019 at 06:29:10PM +0100, Sudeep Holla wrote:

> To support KVM, we need to drop at EL2 and not EL1 before we boot Linux
> kernel. This causes issues on platform with VHE and secondaries booting
> at EL2 via TF-A PSCI CPU_ON call.
> 
> Cc: Ryan Harkin <ryan.harkin@linaro.org>
> Cc: Liviu Dudau <liviu.dudau@foss.arm.com>
> Cc: Linus Walleij <linus.walleij@linaro.org>
> Cc: David Feng <fenghua@phytium.com.cn>
> Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
> Reviewed-by: Peng Fan <peng.fan@nxp.com>

Applied to u-boot/master, thanks!
diff mbox series

Patch

diff --git a/include/configs/vexpress_aemv8a.h b/include/configs/vexpress_aemv8a.h
index 89cd8b550ae6..974647055221 100644
--- a/include/configs/vexpress_aemv8a.h
+++ b/include/configs/vexpress_aemv8a.h
@@ -11,7 +11,6 @@ 
 #ifndef CONFIG_SEMIHOSTING
 #error CONFIG_TARGET_VEXPRESS64_BASE_FVP requires CONFIG_SEMIHOSTING
 #endif
-#define CONFIG_ARMV8_SWITCH_TO_EL1
 #endif
 
 #define CONFIG_REMAKE_ELF