Hi, mike. 2010/12/27 Mike Frysinger <vap...@gentoo.org>: > On Saturday, December 25, 2010 18:02:26 Nobuhiro Iwamatsu wrote: >> Current cmd_link_o_target function in config.mk does not support the set >> of the endian. >> >> Some architecture is bi-endian (e.g. mips and sh). >> Therefore, there is case supporting big endian and little endian >> with one toolchain. >> For example, when user builds target of big endian in host of little >> endian, they need set endian. >> >> # If the list of objects to link is empty, just create an empty built-in.o >> cmd_link_o_target = $(if $(strip $1),\ >> - $(LD) -r -o $@ $1 ,\ >> + $(LD) $(ENDIANNESS) -r -o $@ $1 ,\ > > i dont think we should start declaring random new variables with specific > purposes.
I agree. But.... > better to split the "u-boot final" LDFLAGS out into their own > variable (LDFLAGS_u-boot) and keep LDFLAGS as a "these are the flags that need > to be used with $(LD)". cmd_link_o_target is not used in the last of u-boot (u-boot final) . But this is used in the middle of build. For example, when we make libstubs.o in examples/standalone/Makefile. : examples/standalone/Makefile ----- 89 90 all: $(obj).depend $(OBJS) $(LIB) $(SREC) $(BIN) $(ELF) 91 92 ######################################################################### 93 $(LIB): $(obj).depend $(LIBOBJS) 94 $(call cmd_link_o_target, $(LIBOBJS)) 95 96 $(ELF): ----- Therefore, I think that we have to add a new variable to this. Best regards, Nobuhiro -- Nobuhiro Iwamatsu _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
