Le 19/09/2010 08:04, Heiko Schocher a écrit :
> Hello Wolfgang,
>
> Wolfgang Denk wrote:
>> Dear Albert ARIBAUD,
>>
>> In message<4c953ec4.2090...@free.fr>  you wrote:
>>>> +#if defined(CONFIG_BOARD_EARLY_INIT_F)
>>>> +  board_early_init_f,
>>>> +#endif
>>>> +  timer_init,             /* initialize timer */
>>>> +#ifdef CONFIG_FSL_ESDHC
>>>> +  get_clocks,
>>>> +#endif
>>>> +  env_init,               /* initialize environment */
>>>> +  init_baudrate,          /* initialze baudrate settings */
>>>> +  serial_init,            /* serial communications setup */
>>>> +  console_init_f,         /* stage 1 init of console */
>>>> +  display_banner,         /* say that we are here */
>>>> +#if defined(CONFIG_DISPLAY_CPUINFO)
>>>> +  print_cpuinfo,          /* display cpu info (and speed) */
>>>> +#endif
>>>> +#if defined(CONFIG_DISPLAY_BOARDINFO)
>>>> +  checkboard,             /* display board info */
>>>> +#endif
>>>> +#if defined(CONFIG_HARD_I2C) || defined(CONFIG_SOFT_I2C)
>>>> +  init_func_i2c,
>>>> +#endif
>>>> +  dram_init,              /* configure available RAM banks */
>>>> +#if defined(CONFIG_CMD_PCI) || defined (CONFIG_PCI)
>>>> +  arm_pci_init,
>>>> +#endif
>>>> +  NULL,
>>>> +};
>>> Are all these inits really required at board_init_f stage? I understand
>>
>> Good question...
>>
>>> initializing the console in order to print out diagnostic and error
>>> messages before RAM is initialized, and I understand initializing RAM of
>>> course, but are I2C or PCI init needed before relocation? Can't they
>>> wait until board_init_r?
>>
>> I2C may be needed for example if you have the environment stored in a
>> I2C EEPROM - but then this is needed before init_baudrate ...
>
> or for reading SPD EEprom for DDR setup ...
>
>> CONFIG_PCI - I don't think this is used at all by any ARM board yet.
>>
>>
>> My understanding is that Heiko might be preparing arch/arm/lib/board.c
>> for merging with arch/powerpc/lib/board.c ...
>
> Indeed, I copied this from arch/powerpc/lib/board.c and added
> arm specific code. I think goal should be, that we can make a
> arch/generic/lib/board.c ... ?
>
> bye,
> Heiko

Thanks, that clarifies it.

Amicalement,
-- 
Albert.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to