Re: [U-Boot] [PATCH 1/4] imx: mx7: avoid some initialization if low level is skipped

2018-09-05 Thread Peng Fan


> -Original Message-
> From: Bryan O'Donoghue [mailto:bryan.odonog...@linaro.org]
> Sent: 2018年9月5日 18:56
> To: u-boot@lists.denx.de; Fabio Estevam 
> Cc: ryan.har...@linaro.org; Rui Miguel Silva ; Bryan
> O'Donoghue ; Stefano Babic ;
> Albert Aribaud ; Peng Fan 
> Subject: [PATCH 1/4] imx: mx7: avoid some initialization if low level is 
> skipped
> 
> From: Rui Miguel Silva 
> 
> We can have the case where u-boot is launched after some other low level
> enabler, like for example when u-boot runs after arm-trusted-firmware and/or
> optee. So, because of that we may need to jump the initialization of some IP
> blocks even because we may no longer have the permission for that.
> 
> So, if the config option to skip low level init is set disable also timer, 
> board and
> csu initialization.
> 
> Signed-off-by: Rui Miguel Silva 
> Signed-off-by: Bryan O'Donoghue 
> Cc: Stefano Babic 
> Cc: Fabio Estevam 
> Cc: Albert Aribaud 
> Cc: Peng Fan 
> Cc: u-boot@lists.denx.de
> ---
>  arch/arm/mach-imx/mx7/soc.c| 2 ++
>  arch/arm/mach-imx/syscounter.c | 2 ++
>  2 files changed, 4 insertions(+)
> 
> diff --git a/arch/arm/mach-imx/mx7/soc.c b/arch/arm/mach-imx/mx7/soc.c
> index 7334ca9eb8..c38bd1ce46 100644
> --- a/arch/arm/mach-imx/mx7/soc.c
> +++ b/arch/arm/mach-imx/mx7/soc.c
> @@ -133,6 +133,7 @@ u32 __weak get_board_rev(void)  }  #endif
> 
> +#ifndef CONFIG_SKIP_LOWLEVEL_INIT
>  /* enable all periherial can be accessed in nosec mode */  static void
> init_csu(void)  { @@ -182,6 +183,7 @@ int arch_cpu_init(void)
> 
>   return 0;
>  }
> +#endif
> 
>  #ifdef CONFIG_ARCH_MISC_INIT
>  int arch_misc_init(void)
> diff --git a/arch/arm/mach-imx/syscounter.c b/arch/arm/mach-imx/syscounter.c
> index 676bb3caa9..2c319681fc 100644
> --- a/arch/arm/mach-imx/syscounter.c
> +++ b/arch/arm/mach-imx/syscounter.c
> @@ -55,6 +55,7 @@ static inline unsigned long long us_to_tick(unsigned long
> long usec)
>   return usec;
>  }
> 
> +#ifndef CONFIG_SKIP_LOWLEVEL_INIT
>  int timer_init(void)
>  {
>   struct sctr_regs *sctr = (struct sctr_regs *)SCTR_BASE_ADDR; @@ -76,6
> +77,7 @@ int timer_init(void)
> 
>   return 0;
>  }
> +#endif
> 
>  unsigned long long get_ticks(void)
>  {

Reviewed-by: Peng Fan 

> --
> 2.18.0

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


[U-Boot] [PATCH 1/4] imx: mx7: avoid some initialization if low level is skipped

2018-09-05 Thread Bryan O'Donoghue
From: Rui Miguel Silva 

We can have the case where u-boot is launched after some other low level
enabler, like for example when u-boot runs after arm-trusted-firmware
and/or optee. So, because of that we may need to jump the initialization of
some IP blocks even because we may no longer have the permission for that.

So, if the config option to skip low level init is set disable also timer,
board and csu initialization.

Signed-off-by: Rui Miguel Silva 
Signed-off-by: Bryan O'Donoghue 
Cc: Stefano Babic 
Cc: Fabio Estevam 
Cc: Albert Aribaud 
Cc: Peng Fan 
Cc: u-boot@lists.denx.de
---
 arch/arm/mach-imx/mx7/soc.c| 2 ++
 arch/arm/mach-imx/syscounter.c | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/arch/arm/mach-imx/mx7/soc.c b/arch/arm/mach-imx/mx7/soc.c
index 7334ca9eb8..c38bd1ce46 100644
--- a/arch/arm/mach-imx/mx7/soc.c
+++ b/arch/arm/mach-imx/mx7/soc.c
@@ -133,6 +133,7 @@ u32 __weak get_board_rev(void)
 }
 #endif
 
+#ifndef CONFIG_SKIP_LOWLEVEL_INIT
 /* enable all periherial can be accessed in nosec mode */
 static void init_csu(void)
 {
@@ -182,6 +183,7 @@ int arch_cpu_init(void)
 
return 0;
 }
+#endif
 
 #ifdef CONFIG_ARCH_MISC_INIT
 int arch_misc_init(void)
diff --git a/arch/arm/mach-imx/syscounter.c b/arch/arm/mach-imx/syscounter.c
index 676bb3caa9..2c319681fc 100644
--- a/arch/arm/mach-imx/syscounter.c
+++ b/arch/arm/mach-imx/syscounter.c
@@ -55,6 +55,7 @@ static inline unsigned long long us_to_tick(unsigned long 
long usec)
return usec;
 }
 
+#ifndef CONFIG_SKIP_LOWLEVEL_INIT
 int timer_init(void)
 {
struct sctr_regs *sctr = (struct sctr_regs *)SCTR_BASE_ADDR;
@@ -76,6 +77,7 @@ int timer_init(void)
 
return 0;
 }
+#endif
 
 unsigned long long get_ticks(void)
 {
-- 
2.18.0

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot