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

[Mike Frysinger]

On Tuesday 18 October 2011 04:28:23 馬克泡 wrote:
> 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.
> 
> 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.

for now, i don't think so.  "sandbox" isn't an arch you can create uimages for 
(atm?).
-mike