[PATCH] board: xilinx/zynqmp: Remove 1-second delay at boot

[PATCH] board: xilinx/zynqmp: Remove 1-second delay at boot

[Mike Looijmans]

SPL boot is slow on the ZynqMP because of this delay. The initialization done
in psu_init no longer requires an external delay, so this can be removed to
speed up the SPL boot flow considerably.

Signed-off-by: Mike Looijmans <mike.looijmans@topic.nl>
---
 board/xilinx/zynqmp/zynqmp.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/board/xilinx/zynqmp/zynqmp.c b/board/xilinx/zynqmp/zynqmp.c
index cb72914adb..d00235f236 100644
--- a/board/xilinx/zynqmp/zynqmp.c
+++ b/board/xilinx/zynqmp/zynqmp.c
@@ -32,7 +32,6 @@
 #include <zynqmp_firmware.h>
 #include <g_dnl.h>
 #include <linux/bitops.h>
-#include <linux/delay.h>
 #include <linux/sizes.h>
 #include "../common/board.h"
 
@@ -334,9 +333,6 @@ int board_early_init_f(void)
 	ret = psu_init();
 	if (ret)
 		return ret;
-
-	/* Delay is required for clocks to be propagated */
-	udelay(1000000);
 #endif
 
 #ifdef CONFIG_DEBUG_UART
-- 
2.17.1

[PATCH] board: xilinx/zynqmp: Remove 1-second delay at boot

[Michal Simek]

On 26. 05. 20 10:20, Mike Looijmans wrote:
> SPL boot is slow on the ZynqMP because of this delay. The initialization done
> in psu_init no longer requires an external delay, so this can be removed to
> speed up the SPL boot flow considerably.
> 
> Signed-off-by: Mike Looijmans <mike.looijmans@topic.nl>
> ---
>  board/xilinx/zynqmp/zynqmp.c | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/board/xilinx/zynqmp/zynqmp.c b/board/xilinx/zynqmp/zynqmp.c
> index cb72914adb..d00235f236 100644
> --- a/board/xilinx/zynqmp/zynqmp.c
> +++ b/board/xilinx/zynqmp/zynqmp.c
> @@ -32,7 +32,6 @@
>  #include <zynqmp_firmware.h>
>  #include <g_dnl.h>
>  #include <linux/bitops.h>
> -#include <linux/delay.h>
>  #include <linux/sizes.h>
>  #include "../common/board.h"
>  
> @@ -334,9 +333,6 @@ int board_early_init_f(void)
>  	ret = psu_init();
>  	if (ret)
>  		return ret;
> -
> -	/* Delay is required for clocks to be propagated */
> -	udelay(1000000);
>  #endif
>  
>  #ifdef CONFIG_DEBUG_UART
> 

Mike reported me that removing this 1s is causing issue in SD boot mode
that's why I am not applying this patch till we root caused it.

Thanks,
Michal