[PATCH 03/18] common: board_f: Move setup_machine code to setup_bdinfo

Simon Glass sjg at chromium.org
Sat Nov 14 16:17:25 CET 2020


Hi Ovidiu

On Thu, 5 Nov 2020 at 03:10, Ovidiu Panait <ovidiu.panait at windriver.com> wrote:
>
> setup_bdinfo is used to populate various bdinfo fields, so move
> setup_machine code there, as all it does is setting
> gd->bd->bi_arch_number.

But you are moving it to reserve_global_data() aren't you? I don't
like the sound of that.

>
> Signed-off-by: Ovidiu Panait <ovidiu.panait at windriver.com>
> ---
>  common/board_f.c | 13 ++++---------
>  1 file changed, 4 insertions(+), 9 deletions(-)
>
> diff --git a/common/board_f.c b/common/board_f.c
> index a3c353a4b5..408b95826a 100644
> --- a/common/board_f.c
> +++ b/common/board_f.c
> @@ -503,14 +503,6 @@ static int reserve_board(void)
>         return 0;
>  }
>
> -static int setup_machine(void)
> -{
> -#ifdef CONFIG_MACH_TYPE
> -       gd->bd->bi_arch_number = CONFIG_MACH_TYPE; /* board id for Linux */
> -#endif
> -       return 0;
> -}
> -
>  static int reserve_global_data(void)
>  {
>         gd->start_addr_sp = reserve_stack_aligned(sizeof(gd_t));
> @@ -605,6 +597,10 @@ int setup_bdinfo(void)
>                 bd->bi_sramsize = CONFIG_SYS_SRAM_SIZE;  /* size  of SRAM */
>         }
>
> +#ifdef CONFIG_MACH_TYPE
> +       bd->bi_arch_number = CONFIG_MACH_TYPE; /* board id for Linux */
> +#endif
> +
>         return arch_setup_bdinfo();
>  }
>
> @@ -916,7 +912,6 @@ static const init_fnc_t init_sequence_f[] = {
>         reserve_uboot,
>         reserve_malloc,
>         reserve_board,
> -       setup_machine,
>         reserve_global_data,
>         reserve_fdt,
>         reserve_bootstage,
> --
> 2.17.1
>

Regards,
SImon


More information about the U-Boot mailing list