* [U-Boot] [PATCH] helper: Add architecture specific symbol prefix in the uboot list link script.
@ 2013-02-06 9:21 Sonic Zhang
2013-02-06 13:36 ` Albert ARIBAUD
2013-02-06 23:15 ` Wolfgang Denk
0 siblings, 2 replies; 3+ messages in thread
From: Sonic Zhang @ 2013-02-06 9:21 UTC (permalink / raw)
To: u-boot
From: Sonic Zhang <sonic.zhang@analog.com>
Blackfin compiler add prefix '_' to each symbols. All symbols defined in the
link script should have this prefix. Add it to the uboot list helper function
make_u_boot_list.
Signed-off-by: Sonic Zhang <sonic.zhang@analog.com>
---
helper.mk | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/helper.mk b/helper.mk
index 79a1da0..16e4cfd 100644
--- a/helper.mk
+++ b/helper.mk
@@ -58,7 +58,7 @@ $(1): $(2)
-e ':s /^.\+$$$$/ { p;s/^\(.*\)\.[^\.]*$$$$/\1/;b s }' | \
sed -n -e 'h;s/$$$$/\a/p;g;s/$$$$/@/p;g;s/$$$$/~/p;' | \
LC_COLLATE=C sort -u | \
- sed -n -e '/\a$$$$/ { s/\./_/g;s/\a$$$$/__start = .;/p; }'\
- -e '/~$$$$/ { s/\./_/g;s/~$$$$/__end = .;/p; }'\
+ sed -n -e '/\a$$$$/ { s/\./_/g;s/\_u_boot_list/$(SYM_PREFIX)_u_boot_list/g;s/\a$$$$/__start = .;/p; }'\
+ -e '/~$$$$/ { s/\./_/g;s/\_u_boot_list/$(SYM_PREFIX)_u_boot_list/g;s/~$$$$/__end = .;/p; }'\
-e '/@$$$$/ { s/\(.*\)@$$$$/*(SORT(\1.*));/p }' > $(1)
endef
--
1.7.0.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH] helper: Add architecture specific symbol prefix in the uboot list link script.
2013-02-06 9:21 [U-Boot] [PATCH] helper: Add architecture specific symbol prefix in the uboot list link script Sonic Zhang
@ 2013-02-06 13:36 ` Albert ARIBAUD
2013-02-06 23:15 ` Wolfgang Denk
1 sibling, 0 replies; 3+ messages in thread
From: Albert ARIBAUD @ 2013-02-06 13:36 UTC (permalink / raw)
To: u-boot
Hi Sonic,
On Wed, 6 Feb 2013 17:21:03 +0800, Sonic Zhang <sonic.adi@gmail.com>
wrote:
> From: Sonic Zhang <sonic.zhang@analog.com>
>
> Blackfin compiler add prefix '_' to each symbols. All symbols defined in the
> link script should have this prefix. Add it to the uboot list helper function
> make_u_boot_list.
>
> Signed-off-by: Sonic Zhang <sonic.zhang@analog.com>
> ---
> helper.mk | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/helper.mk b/helper.mk
> index 79a1da0..16e4cfd 100644
> --- a/helper.mk
> +++ b/helper.mk
> @@ -58,7 +58,7 @@ $(1): $(2)
> -e ':s /^.\+$$$$/ { p;s/^\(.*\)\.[^\.]*$$$$/\1/;b s }' | \
> sed -n -e 'h;s/$$$$/\a/p;g;s/$$$$/@/p;g;s/$$$$/~/p;' | \
> LC_COLLATE=C sort -u | \
> - sed -n -e '/\a$$$$/ { s/\./_/g;s/\a$$$$/__start = .;/p; }'\
> - -e '/~$$$$/ { s/\./_/g;s/~$$$$/__end = .;/p; }'\
> + sed -n -e '/\a$$$$/ { s/\./_/g;s/\_u_boot_list/$(SYM_PREFIX)_u_boot_list/g;s/\a$$$$/__start = .;/p; }'\
> + -e '/~$$$$/ { s/\./_/g;s/\_u_boot_list/$(SYM_PREFIX)_u_boot_list/g;s/~$$$$/__end = .;/p; }'\
> -e '/@$$$$/ { s/\(.*\)@$$$$/*(SORT(\1.*));/p }' > $(1)
> endef
The patch I submitted to fix relocation of linker lists [1] removes file
helper.mk. There is thus no need to change it.
[1] http://patchwork.ozlabs.org/patch/217695/
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH] helper: Add architecture specific symbol prefix in the uboot list link script.
2013-02-06 9:21 [U-Boot] [PATCH] helper: Add architecture specific symbol prefix in the uboot list link script Sonic Zhang
2013-02-06 13:36 ` Albert ARIBAUD
@ 2013-02-06 23:15 ` Wolfgang Denk
1 sibling, 0 replies; 3+ messages in thread
From: Wolfgang Denk @ 2013-02-06 23:15 UTC (permalink / raw)
To: u-boot
Dear Sonic Zhang,
In message <1360142463-5035-1-git-send-email-sonic.adi@gmail.com> you wrote:
> From: Sonic Zhang <sonic.zhang@analog.com>
>
> Blackfin compiler add prefix '_' to each symbols. All symbols defined in the
> link script should have this prefix. Add it to the uboot list helper function
> make_u_boot_list.
>
> Signed-off-by: Sonic Zhang <sonic.zhang@analog.com>
Maybe the BF compiler does so - but this script is used by all
architectures: did you verify that this change is actually working on
those?
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"Nobody will ever need more than 640k RAM!" -- Bill Gates, 1981
"Windows 95 needs at least 8 MB RAM." -- Bill Gates, 1996
"Nobody will ever need Windows 95." -- logical conclusion
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-02-06 23:15 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-02-06 9:21 [U-Boot] [PATCH] helper: Add architecture specific symbol prefix in the uboot list link script Sonic Zhang
2013-02-06 13:36 ` Albert ARIBAUD
2013-02-06 23:15 ` Wolfgang Denk
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.