[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 00:46:28 CEST 2008


On Oct 19, 2008, at 1:44 PM, Jean-Christophe PLAGNIOL-VILLARD wrote:

> On 13:07 Sun 19 Oct     , Kumar Gala 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>
>> ---
>>
>> Fix a bug that showed up in sub command testing in that we didn't  
>> sent the relocated
>> flag.
>>
>> common/cmd_bootm.c |   68 ++++++++++++++++++++++ 
>> +----------------------------
>> 1 files changed, 31 insertions(+), 37 deletions(-)
>>
>> diff --git a/common/cmd_bootm.c b/common/cmd_bootm.c
>> index b02da3e..956e1a0 100644
>> --- a/common/cmd_bootm.c
>> +++ b/common/cmd_bootm.c
>> @@ -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.

- k


More information about the U-Boot mailing list