HI Simon, 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. > > 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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot