[U-Boot] [PATCH] MIPS: Add board_early_init_f() to init_sequence

Stefan Roese sr at denx.de
Tue Nov 18 16:56:38 CET 2008


On Tuesday 18 November 2008, Shinya Kuribayashi wrote:
> >> +++ b/lib_mips/board.c
> >> @@ -70,6 +70,15 @@ static ulong mem_malloc_brk;
> >>   */
> >>  unsigned long mips_io_port_base = -1;
> >>
> >> +int __board_early_init_f(void)
> >> +{
> >> +	/*
> >> +	 * Nothing to do in this dummy implementation
> >> +	 */
> >> +	return 0;
> >> +}
> >
> > why not remove this and create a generic dummy function
> > to reduce to code size impact
>
> +1.  Such dummy function could be shared among all architectures.

OK, makes perfect sense. We would need different dummy functions for each 
different parameter though, such as:

void dummy_void_void(void);
int dummy_int_void(void);
int dummy_int_int(int);
int dummy_int_pint(* int);
...

And where should those functions be placed? We could create a new file 
probably in lib_generic for this. I suggest we postpone this consolidation 
until after the next release though.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================


More information about the U-Boot mailing list