[U-Boot] [PATCH v2] bootm: Move to using a function pointer table for the boot os function

Wolfgang Denk wd at denx.de
Mon Oct 20 09:28:36 CEST 2008


Dear Kumar Gala,

In message <6FA5D356-5762-4185-BD0C-1CE3BDEDE2D9 at kernel.crashing.org> you wrote:
> 
> >> This removes a bit of code and makes it easier for the upcoming sub  
> >> bootm
> >> command support to call into the proper OS specific handler.
> >>
> >> Signed-off-by: Kumar Gala <galak at kernel.crashing.org>
> >> ---
...
> >> @@ -119,6 +119,22 @@ int do_bootelf (cmd_tbl_t *cmdtp, int flag,  
> >> int argc, char *argv[]);
> >> static boot_os_fn do_bootm_integrity;
> >> #endif
> >>
> >> +boot_os_fn * boot_os[] = {
> >> +	[IH_OS_LINUX] = do_bootm_linux,
> >> +	[IH_OS_NETBSD] = do_bootm_netbsd,
> > why not add config to ative or not netbsd, linux and rterms to  
> > reduce U-Boot
> > size?
> 
> ative?
> 
> that would be an orthogonal change.

One that we should implement, I think.

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: wd at denx.de
Drawing on my fine command of language, I said nothing.


More information about the U-Boot mailing list