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 ... 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 ... Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Do not simplify the design of a program if a way can be found to make it complex and wonderful. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
