Re: [U-Boot] [PATCH v3] examples: select libgcc for non-default architecture

2014-05-18 Thread Masahiro Yamada
Hi Alexey,

On Fri, 16 May 2014 12:54:17 +0400
Alexey Brodkin  wrote:

> In case of multilib-enabled toolchains if default architecture differ from
> the one examples are being built for linker will fail to link example object
> files with libgcc of another (non-compatible) architecture.
> 
> Interesting enough for years in main Makefile we used CFLAGS/c_flags for this
> but not for examples.
> 
> So fixing it now.
> 
> Signed-off-by: Alexey Brodkin 
> 
> Cc: Masahiro Yamada 
> Cc: Tom Rini 
> Cc: Wolfgang Denx 


I did build test on some boards with/without CONFIG_USE_PRIVATE_LIBGCC
and I worked as expected.

So, you can have my

Acked-by: Masahiro Yamada 

Thanks!


Best Regards
Masahiro Yamada

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v3] examples: select libgcc for non-default architecture

2014-05-16 Thread Wolfgang Denk
Dear Alexey Brodkin,

In message <1400230457-11075-1-git-send-email-abrod...@synopsys.com> you wrote:
> In case of multilib-enabled toolchains if default architecture differ from
> the one examples are being built for linker will fail to link example object
> files with libgcc of another (non-compatible) architecture.
> 
> Interesting enough for years in main Makefile we used CFLAGS/c_flags for this
> but not for examples.
> 
> So fixing it now.
> 
> Signed-off-by: Alexey Brodkin 
> 
> Cc: Masahiro Yamada 
> Cc: Tom Rini 
> Cc: Wolfgang Denx 
> 
> ---
> In v3 we export PLATFORM_LIBGCC from the main Makefile and use it for examples

Thanks!

Acked-by: WOlfgang Denk 

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: w...@denx.de
"Pull the wool over your own eyes!"- J.R. "Bob" Dobbs
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v3] examples: select libgcc for non-default architecture

2014-05-16 Thread Alexey Brodkin
In case of multilib-enabled toolchains if default architecture differ from
the one examples are being built for linker will fail to link example object
files with libgcc of another (non-compatible) architecture.

Interesting enough for years in main Makefile we used CFLAGS/c_flags for this
but not for examples.

So fixing it now.

Signed-off-by: Alexey Brodkin 

Cc: Masahiro Yamada 
Cc: Tom Rini 
Cc: Wolfgang Denx 

---
In v3 we export PLATFORM_LIBGCC from the main Makefile and use it for examples
---
 Makefile | 1 +
 examples/standalone/Makefile | 4 +---
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/Makefile b/Makefile
index ff38a43..fd4e176 100644
--- a/Makefile
+++ b/Makefile
@@ -696,6 +696,7 @@ PLATFORM_LIBGCC := -L $(shell dirname `$(CC) $(c_flags) 
-print-libgcc-file-name`
 endif
 PLATFORM_LIBS += $(PLATFORM_LIBGCC)
 export PLATFORM_LIBS
+export PLATFORM_LIBGCC
 
 # Special flags for CPP when processing the linker script.
 # Pass the version down so we can handle backwards compatibility
diff --git a/examples/standalone/Makefile b/examples/standalone/Makefile
index 9ab5446..2dacba2 100644
--- a/examples/standalone/Makefile
+++ b/examples/standalone/Makefile
@@ -38,8 +38,6 @@ targets += $(patsubst $(obj)/%,%,$(LIB)) $(COBJS) $(LIBOBJS-y)
 LIBOBJS:= $(addprefix $(obj)/,$(LIBOBJS-y))
 ELF:= $(addprefix $(obj)/,$(ELF))
 
-gcclibdir := $(shell dirname `$(CC) -print-libgcc-file-name`)
-
 # For PowerPC there's no need to compile standalone applications as a
 # relocatable executable.  The relocation data is not needed, and
 # also causes the entry point of the standalone application to be
@@ -63,7 +61,7 @@ $(LIB):   $(LIBOBJS) FORCE
 
 quiet_cmd_link_elf = LD  $@
   cmd_link_elf = $(LD) $(LDFLAGS) -g -Ttext $(CONFIG_STANDALONE_LOAD_ADDR) 
\
--o $@ -e $(SYM_PREFIX)$(@F) $< $(LIB) -L$(gcclibdir) -lgcc
+-o $@ -e $(SYM_PREFIX)$(@F) $< $(LIB) $(PLATFORM_LIBGCC)
 
 $(ELF): $(obj)/%: $(obj)/%.o $(LIB) FORCE
$(call if_changed,link_elf)
-- 
1.9.0

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot