[PATCH 2/2] board: stm32: Remove the bi_boot_params initialization

[Patrice CHOTARD]

Hi Patrick

On 9/2/21 12:02 PM, Patrick Delaunay wrote:
> The stm32 platforms never had to support an ATAGs-based Linux Kernel,
> so remove the bi_boot_params initialization.
> 
> Signed-off-by: Patrick Delaunay <patrick.delaunay at foss.st.com>
> ---
> 
>  board/dhelectronics/dh_stm32mp1/board.c              | 3 ---
>  board/engicam/stm32mp1/stm32mp1.c                    | 3 ---
>  board/st/stm32f429-discovery/stm32f429-discovery.c   | 2 --
>  board/st/stm32f429-evaluation/stm32f429-evaluation.c | 2 --
>  board/st/stm32f469-discovery/stm32f469-discovery.c   | 2 --
>  board/st/stm32f746-disco/stm32f746-disco.c           | 2 --
>  board/st/stm32h743-disco/stm32h743-disco.c           | 1 -
>  board/st/stm32h743-eval/stm32h743-eval.c             | 1 -
>  board/st/stm32h750-art-pi/stm32h750-art-pi.c         | 1 -
>  board/st/stm32mp1/stm32mp1.c                         | 3 ---
>  10 files changed, 20 deletions(-)
> 
> diff --git a/board/dhelectronics/dh_stm32mp1/board.c b/board/dhelectronics/dh_stm32mp1/board.c
> index d7c1857c16..765b54a4a4 100644
> --- a/board/dhelectronics/dh_stm32mp1/board.c
> +++ b/board/dhelectronics/dh_stm32mp1/board.c
> @@ -590,9 +590,6 @@ static void board_init_fmc2(void)
>  /* board dependent setup after realloc */
>  int board_init(void)
>  {
> -	/* address of boot parameters */
> -	gd->bd->bi_boot_params = STM32_DDR_BASE + 0x100;
> -
>  	if (CONFIG_IS_ENABLED(DM_GPIO_HOG))
>  		gpio_hog_probe_all();
>  
> diff --git a/board/engicam/stm32mp1/stm32mp1.c b/board/engicam/stm32mp1/stm32mp1.c
> index 8bf9c9c67d..20d8603c78 100644
> --- a/board/engicam/stm32mp1/stm32mp1.c
> +++ b/board/engicam/stm32mp1/stm32mp1.c
> @@ -40,9 +40,6 @@ int checkboard(void)
>  /* board dependent setup after realloc */
>  int board_init(void)
>  {
> -	/* address of boot parameters */
> -	gd->bd->bi_boot_params = STM32_DDR_BASE + 0x100;
> -
>  	if (IS_ENABLED(CONFIG_DM_REGULATOR))
>  		regulators_enable_boot_on(_DEBUG);
>  
> diff --git a/board/st/stm32f429-discovery/stm32f429-discovery.c b/board/st/stm32f429-discovery/stm32f429-discovery.c
> index 46fcf907fc..5a50e98dd0 100644
> --- a/board/st/stm32f429-discovery/stm32f429-discovery.c
> +++ b/board/st/stm32f429-discovery/stm32f429-discovery.c
> @@ -53,8 +53,6 @@ u32 get_board_rev(void)
>  
>  int board_init(void)
>  {
> -	gd->bd->bi_boot_params = gd->bd->bi_dram[0].start + 0x100;
> -
>  	return 0;
>  }
>  
> diff --git a/board/st/stm32f429-evaluation/stm32f429-evaluation.c b/board/st/stm32f429-evaluation/stm32f429-evaluation.c
> index 3b6df1f3ab..cf3056163c 100644
> --- a/board/st/stm32f429-evaluation/stm32f429-evaluation.c
> +++ b/board/st/stm32f429-evaluation/stm32f429-evaluation.c
> @@ -47,8 +47,6 @@ u32 get_board_rev(void)
>  
>  int board_init(void)
>  {
> -	gd->bd->bi_boot_params = gd->bd->bi_dram[0].start + 0x100;
> -
>  	return 0;
>  }
>  
> diff --git a/board/st/stm32f469-discovery/stm32f469-discovery.c b/board/st/stm32f469-discovery/stm32f469-discovery.c
> index c5df9b0d9c..056c9dff2a 100644
> --- a/board/st/stm32f469-discovery/stm32f469-discovery.c
> +++ b/board/st/stm32f469-discovery/stm32f469-discovery.c
> @@ -47,8 +47,6 @@ u32 get_board_rev(void)
>  
>  int board_init(void)
>  {
> -	gd->bd->bi_boot_params = gd->bd->bi_dram[0].start + 0x100;
> -
>  	return 0;
>  }
>  
> diff --git a/board/st/stm32f746-disco/stm32f746-disco.c b/board/st/stm32f746-disco/stm32f746-disco.c
> index efa38a0e26..2543e2a5f8 100644
> --- a/board/st/stm32f746-disco/stm32f746-disco.c
> +++ b/board/st/stm32f746-disco/stm32f746-disco.c
> @@ -122,8 +122,6 @@ int board_late_init(void)
>  
>  int board_init(void)
>  {
> -	gd->bd->bi_boot_params = gd->bd->bi_dram[0].start + 0x100;
> -
>  #ifdef CONFIG_ETH_DESIGNWARE
>  	const char *phy_mode;
>  	int node;
> diff --git a/board/st/stm32h743-disco/stm32h743-disco.c b/board/st/stm32h743-disco/stm32h743-disco.c
> index 4091d5f9fd..e493786f11 100644
> --- a/board/st/stm32h743-disco/stm32h743-disco.c
> +++ b/board/st/stm32h743-disco/stm32h743-disco.c
> @@ -43,6 +43,5 @@ u32 get_board_rev(void)
>  
>  int board_init(void)
>  {
> -	gd->bd->bi_boot_params = gd->bd->bi_dram[0].start + 0x100;
>  	return 0;
>  }
> diff --git a/board/st/stm32h743-eval/stm32h743-eval.c b/board/st/stm32h743-eval/stm32h743-eval.c
> index 4091d5f9fd..e493786f11 100644
> --- a/board/st/stm32h743-eval/stm32h743-eval.c
> +++ b/board/st/stm32h743-eval/stm32h743-eval.c
> @@ -43,6 +43,5 @@ u32 get_board_rev(void)
>  
>  int board_init(void)
>  {
> -	gd->bd->bi_boot_params = gd->bd->bi_dram[0].start + 0x100;
>  	return 0;
>  }
> diff --git a/board/st/stm32h750-art-pi/stm32h750-art-pi.c b/board/st/stm32h750-art-pi/stm32h750-art-pi.c
> index 5785b2e575..bec26465d2 100644
> --- a/board/st/stm32h750-art-pi/stm32h750-art-pi.c
> +++ b/board/st/stm32h750-art-pi/stm32h750-art-pi.c
> @@ -53,6 +53,5 @@ int board_late_init(void)
>  
>  int board_init(void)
>  {
> -	gd->bd->bi_boot_params = gd->bd->bi_dram[0].start + 0x100;
>  	return 0;
>  }
> diff --git a/board/st/stm32mp1/stm32mp1.c b/board/st/stm32mp1/stm32mp1.c
> index 032f08d795..1bceb41494 100644
> --- a/board/st/stm32mp1/stm32mp1.c
> +++ b/board/st/stm32mp1/stm32mp1.c
> @@ -646,9 +646,6 @@ static void board_ev1_init(void)
>  /* board dependent setup after realloc */
>  int board_init(void)
>  {
> -	/* address of boot parameters */
> -	gd->bd->bi_boot_params = STM32_DDR_BASE + 0x100;
> -
>  	if (CONFIG_IS_ENABLED(DM_GPIO_HOG))
>  		gpio_hog_probe_all();
>  
> 

Reviewed-by: Patrice Chotard <patrice.chotard at foss.st.com>
Tested-by: Patrice Chotard <patrice.chotard at foss.st.com>

Thanks
Patrice