[U-Boot] [PATCH v6 03/20] sandbox: Add architecture image support

馬克泡 macpaul at gmail.com
Tue Oct 18 10:28:23 CEST 2011


HI Simon,

2011/10/18 Wolfgang Denk <wd at denx.de>:
> Dear Simon Glass,
>
>> Changes in v5:
>> - Define IH_ARCH_DEFAULT which is now required
>>
>>  arch/sandbox/include/asm/u-boot.h |    3 +++
>>  include/image.h                   |    1 +
>>  2 files changed, 4 insertions(+), 0 deletions(-)
>
> Applied, thanks.
>
> Best regards,
>
> Wolfgang Denk
>

Sorry for late to reply your mail.
I'm doing PATCH v17 fix about NDS32 architecture against to your patches.
I"ve found here you've defined IH_ARCH_SANDBOX in /include/image.h
Just wondering about don't you need to also add this into
../common/image.c in the following table?
static const table_entry_t uimage_arch[]
[deleted]
        {       IH_ARCH_BLACKFIN,       "blackfin",     "Blackfin",     },
        {       IH_ARCH_AVR32,          "avr32",        "AVR32",        },
        {       -1,                     "",             "",             },
};

Because I'm wondering about if I need to fix this if you need to add
IH_ARCH_SANDBOX into this structure.

Thanks!

-- 
Best regards,
Macpaul Lin


More information about the U-Boot mailing list