diff mbox

[U-Boot] armv8: Flush dcache before switching to EL2

Message ID 1396302032-31292-1-git-send-email-yorksun@freescale.com
State Accepted
Delegated to: Albert ARIBAUD
Headers show

Commit Message

York Sun March 31, 2014, 9:40 p.m. UTC
For ARMv8, U-boot has been running at EL3 with cache and MMU enabled.
Without proper setup for EL2, cache and MMU are both disabled (out of
reset). Before switching, we need to flush the dcache to make sure the
data is in the main memory.

Signed-off-by: York Sun <yorksun@freescale.com>
---
 arch/arm/lib/bootm.c |    1 +
 1 file changed, 1 insertion(+)

Comments

fenghua@phytium.com.cn April 2, 2014, 10:05 a.m. UTC | #1
> -----Original Messages-----
> From: "York Sun" <yorksun@freescale.com>
> Sent Time: 2014-04-01 05:40:32 (Tuesday)
> To: albert.u.boot@aribaud.net
> Cc: u-boot@lists.denx.de, "York Sun" <yorksun@freescale.com>
> Subject: [U-Boot] [PATCH] armv8: Flush dcache before switching to EL2
> 
> For ARMv8, U-boot has been running at EL3 with cache and MMU enabled.
> Without proper setup for EL2, cache and MMU are both disabled (out of
> reset). Before switching, we need to flush the dcache to make sure the
> data is in the main memory.
> 
> Signed-off-by: York Sun <yorksun@freescale.com>
> ---
>  arch/arm/lib/bootm.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/lib/bootm.c b/arch/arm/lib/bootm.c
> index a8295bf..9782ddb 100644
> --- a/arch/arm/lib/bootm.c
> +++ b/arch/arm/lib/bootm.c
> @@ -199,6 +199,7 @@ static void do_nonsec_virt_switch(void)
>  
>  #ifdef CONFIG_ARM64
>  	smp_kick_all_cpus();
> +	flush_dcache_all();	/* flush cache before swtiching to EL2 */
>  	armv8_switch_to_el2();
>  #ifdef CONFIG_ARMV8_SWITCH_TO_EL1
>  	armv8_switch_to_el1();
> -- 
> 1.7.9.5

fine!

Acked-by: David.Feng <fenghua@phytium.com.cn>
Albert ARIBAUD April 7, 2014, 8:23 p.m. UTC | #2
Hi York,

On Mon, 31 Mar 2014 14:40:32 -0700, York Sun <yorksun@freescale.com>
wrote:

> For ARMv8, U-boot has been running at EL3 with cache and MMU enabled.
> Without proper setup for EL2, cache and MMU are both disabled (out of
> reset). Before switching, we need to flush the dcache to make sure the
> data is in the main memory.
> 
> Signed-off-by: York Sun <yorksun@freescale.com>
> ---
>  arch/arm/lib/bootm.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/lib/bootm.c b/arch/arm/lib/bootm.c
> index a8295bf..9782ddb 100644
> --- a/arch/arm/lib/bootm.c
> +++ b/arch/arm/lib/bootm.c
> @@ -199,6 +199,7 @@ static void do_nonsec_virt_switch(void)
>  
>  #ifdef CONFIG_ARM64
>  	smp_kick_all_cpus();
> +	flush_dcache_all();	/* flush cache before swtiching to EL2 */
>  	armv8_switch_to_el2();
>  #ifdef CONFIG_ARMV8_SWITCH_TO_EL1
>  	armv8_switch_to_el1();

Applied as a bugfix to u-boot-arm/master, thanks!

Amicalement,
diff mbox

Patch

diff --git a/arch/arm/lib/bootm.c b/arch/arm/lib/bootm.c
index a8295bf..9782ddb 100644
--- a/arch/arm/lib/bootm.c
+++ b/arch/arm/lib/bootm.c
@@ -199,6 +199,7 @@  static void do_nonsec_virt_switch(void)
 
 #ifdef CONFIG_ARM64
 	smp_kick_all_cpus();
+	flush_dcache_all();	/* flush cache before swtiching to EL2 */
 	armv8_switch_to_el2();
 #ifdef CONFIG_ARMV8_SWITCH_TO_EL1
 	armv8_switch_to_el1();