On Tuesday 18 October 2011 04:28:23 馬克泡 wrote: > 2011/10/18 Wolfgang Denk <w...@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot