[U-Boot] [PATCH 1/6 V4] common: Add symbol handling for generic lists into Makefile
Marek Vasut
marex at denx.de
Fri Oct 19 15:19:19 CEST 2012
Dear Wolfgang Denk,
[...]
> Maybe we can sae one call to objdump by storing the intermediate
> result?
I removed the other call in subsequent patch anyway, so this is fine now.
> > + cd $(LNDIR) && $(LD) $(LDFLAGS) $(LDFLAGS_$(@F)) \
> > + $$UNDEF_SYM $$UNDEF_LST $(__OBJS) \
>
> Hm.... we use UNDEF_SYM and UNDEF_LST in exactly the same way? The
> why cannot we do this all in a single run, like:
>
> UNDEF_SYM=`$(OBJDUMP) -x $(LIBBOARD) $(LIBS) | \
> sed -n -e 's/.*\($(SYM_PREFIX)__u_boot_cmd_.*\)/-u\1/p' \
> -e 's/.*\($(SYM_PREFIX)_u_boot_list_.*\)/-u\1/p' | sort | uniq`;
\
> cd $(LNDIR) && $(LD) $(LDFLAGS) $(LDFLAGS_$(@F)) $$UNDEF_SYM $(__OBJS) \
> ...
>
> ?
See above, otherwise I do agree.
> > @rm -f $(obj)board/cray/L1/{bootscript.c,bootscript.image} \
> >
> > $(obj)board/matrix_vision/*/bootscript.img \
> > $(obj)board/voiceblue/eeprom \
> >
> > - $(obj)u-boot.lds \
> > + $(obj)u-boot.lds,include/u-boot.lst \
>
> s/,/ / here!!!
Argh, it's been merged already, right? Fixup patch on the way.
> > + @rm -f
> > $(obj)nand_spl/{u-boot.{lds,lst},u-boot-nand_spl.lds,u-boot-spl,u-boot-s
> > pl.map,System.map} + @rm -f
> > $(obj)spl/{u-boot-spl,u-boot-spl.bin,u-boot-spl.{lds,lst},u-boot-spl.map
> > }
>
> Line length?
True.
> Best regards,
>
> Wolfgang Denk
Best regards,
Marek Vasut
More information about the U-Boot
mailing list