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

Kumar Gala galak at kernel.crashing.org
Mon Oct 20 15:14:29 CEST 2008


On Oct 20, 2008, at 2:28 AM, Wolfgang Denk wrote:

> 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.

So this means introducing CONFIG_BOOT_RTEMS, CONFIG_BOOT_NETBSD, and  
CONFIG_BOOT_LINUX.

Should these be enabled in all board configs by default?

- k



More information about the U-Boot mailing list