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

[Ovidiu Panait <ovidiu.panait@windriver.com>]

Hi Simon,

On 14.11.2020 17:17, Simon Glass wrote:
> [Please note this e-mail is from an EXTERNAL e-mail address]
>
> Hi Ovidiu
>
> On Thu, 5 Nov 2020 at 03:10, Ovidiu Panait <ovidiu.panait@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.

reserve_global_data() is part of context of the previous diff:

@@ -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));


setup_machine() contents get moved to setup_bdinfo() in the next diff:

@@ -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();
  }


Ovidiu

>> Signed-off-by: Ovidiu Panait <ovidiu.panait@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