From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/6 V4] common: Add symbol handling for generic lists into Makefile
Date: Fri, 19 Oct 2012 15:19:19 +0200 [thread overview]
Message-ID: <201210191519.19153.marex@denx.de> (raw)
In-Reply-To: <20121018194021.7932B204030@gemini.denx.de>
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
next prev parent reply other threads:[~2012-10-19 13:19 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-09-30 0:17 [U-Boot] [PATCH 0/5] Linker-generated arrays (take 2) Marek Vasut
2012-09-30 0:17 ` [U-Boot] [PATCH 1/5] common: Add symbol handling for generic lists into Makefile Marek Vasut
2012-10-02 23:40 ` Joe Hershberger
2012-09-30 0:17 ` [U-Boot] [PATCH 2/5] common: Implement support for linker-generated arrays Marek Vasut
2012-10-02 23:40 ` Joe Hershberger
2012-09-30 0:17 ` [U-Boot] [PATCH 3/5] common: Add .u_boot_list into all linker files Marek Vasut
2012-10-02 23:41 ` Joe Hershberger
2012-10-03 14:55 ` Daniel Schwierzeck
2012-10-03 15:01 ` Marek Vasut
2012-10-03 15:21 ` Daniel Schwierzeck
2012-10-03 16:44 ` Marek Vasut
2012-10-03 18:05 ` Daniel Schwierzeck
2012-10-03 20:49 ` [U-Boot] [PATCH 3/5 V2] " Marek Vasut
2012-09-30 0:17 ` [U-Boot] [PATCH 4/5] common: Convert the U-Boot commands to LG-arrays Marek Vasut
2012-10-02 23:43 ` Joe Hershberger
2012-09-30 0:17 ` [U-Boot] [PATCH 5/5] common: Discard the __u_boot_cmd section Marek Vasut
2012-10-02 23:43 ` Joe Hershberger
2012-10-03 14:56 ` Daniel Schwierzeck
2012-10-03 20:50 ` [U-Boot] [PATCH 5/5 V2] " Marek Vasut
2012-10-07 0:06 ` [U-Boot] [PATCH 0/6 V2] Linker-generated arrays (take 2) Marek Vasut
2012-10-07 0:06 ` [U-Boot] [PATCH 1/6 V2] common: Add symbol handling for generic lists into Makefile Marek Vasut
2012-10-07 0:06 ` [U-Boot] [PATCH 2/6 V2] common: Implement support for linker-generated arrays Marek Vasut
2012-10-07 0:06 ` [U-Boot] [PATCH 3/6 V2] common: Add .u_boot_list into all linker files Marek Vasut
2012-10-07 0:06 ` [U-Boot] [PATCH 4/6 V2] common: Convert the U-Boot commands to LG-arrays Marek Vasut
2012-10-07 0:06 ` [U-Boot] [PATCH 5/6 V2] common: Discard the __u_boot_cmd section Marek Vasut
2012-10-07 0:06 ` [U-Boot] [PATCH 6/6 V2] kerneldoc: tmpl: Implement template for LG-arrays Marek Vasut
2012-10-12 20:27 ` [U-Boot] [PATCH 0/6 V3] Linker-generated arrays (take 2) Marek Vasut
2012-10-12 20:27 ` [U-Boot] [PATCH 1/6 V3] common: Add symbol handling for generic lists into Makefile Marek Vasut
2012-10-14 23:28 ` Daniel Schwierzeck
2012-10-15 1:49 ` Marek Vasut
2012-10-15 2:04 ` Daniel Schwierzeck
2012-10-15 2:09 ` Marek Vasut
2012-10-17 10:43 ` [U-Boot] [PATCH 1/6 V4] " Marek Vasut
2012-10-18 19:40 ` Wolfgang Denk
2012-10-19 13:19 ` Marek Vasut [this message]
2012-10-20 17:46 ` Tom Rini
2012-10-20 18:56 ` Marek Vasut
2012-10-21 5:20 ` Tom Rini
2012-10-19 15:00 ` [U-Boot] [PATCH 1/6 V5] " Marek Vasut
2012-10-19 15:45 ` Joe Hershberger
2012-10-19 15:48 ` Joe Hershberger
2012-10-22 21:04 ` Tom Rini
2012-10-12 20:27 ` [U-Boot] [PATCH 2/6 V3] common: Implement support for linker-generated arrays Marek Vasut
2012-10-12 20:27 ` [U-Boot] [PATCH 3/6 V3] common: Add .u_boot_list into all linker files Marek Vasut
2012-10-12 20:27 ` [U-Boot] [PATCH 4/6 V3] common: Convert the U-Boot commands to LG-arrays Marek Vasut
2012-10-12 20:27 ` [U-Boot] [PATCH 5/6 V3] common: Discard the __u_boot_cmd section Marek Vasut
2012-10-17 10:45 ` [U-Boot] [PATCH 5/6 V4] " Marek Vasut
2012-10-12 20:27 ` [U-Boot] [PATCH 6/6 V3] kerneldoc: tmpl: Implement template for LG-arrays Marek Vasut
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201210191519.19153.marex@denx.de \
--to=marex@denx.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox