Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-02 Thread Wolfgang Denk
Dear "Premi, Sanjeev", In message you wrote: > > > Do you still need to remove the $(sort) call for the LIBS? That > > should NOT be done. ... > I know it shouldn't be done - but just trying to take problem one by > one. Unless you want me to report the problems onl after I have a fix > for the

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-02 Thread Premi, Sanjeev
> -Original Message- > From: Wolfgang Denk [mailto:w...@denx.de] > Sent: Thursday, December 02, 2010 5:09 PM > To: Premi, Sanjeev > Cc: Albert ARIBAUD; u-boot@lists.denx.de > Subject: Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across > toolchain versions >

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-02 Thread Wolfgang Denk
Dear "Premi, Sanjeev", In message you wrote: > > Just posted the patch. The u-boot comes up on the EVM - only after > sort related change in the Makefile. Haven't debuged it yet. Could you ***please*** be a bit more precise? What EXACTLY was that "sort related change"? Do you still need to re

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-02 Thread Premi, Sanjeev
> -Original Message- > From: Albert ARIBAUD [mailto:albert.arib...@free.fr] > Sent: Thursday, December 02, 2010 2:12 PM > To: Premi, Sanjeev > Cc: Wolfgang Denk; u-boot@lists.denx.de > Subject: Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across > toolchain ver

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-02 Thread Wolfgang Denk
Dear Albert ARIBAUD, In message <4cf7583c.7000...@free.fr> you wrote: > > > Does it help when you change the "*(.bss)" in > > "arch/arm/cpu/armv7/u-boot.lds" into "*(.*bss)" > > (so it also includes any .sbss objects) ? > > No change. Hm... Maybe it is indeed not a good idea to mix short object

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-02 Thread Albert ARIBAUD
Hi Sanjeev, Le 02/12/2010 09:30, Premi, Sanjeev a écrit : >> -Original Message- >> From: Albert ARIBAUD [mailto:albert.arib...@free.fr] >> Sent: Thursday, December 02, 2010 1:57 PM >> To: Wolfgang Denk >> Cc: u-boot@lists.denx.de; Premi, Sanjeev >> Subj

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-02 Thread Premi, Sanjeev
> -Original Message- > From: Albert ARIBAUD [mailto:albert.arib...@free.fr] > Sent: Thursday, December 02, 2010 1:57 PM > To: Wolfgang Denk > Cc: u-boot@lists.denx.de; Premi, Sanjeev > Subject: Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across > toolchain vers

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-02 Thread Albert ARIBAUD
Le 02/12/2010 09:13, Wolfgang Denk a écrit : > Dear Albert ARIBAUD, > > In message<4cf74fed.2030...@free.fr> you wrote: >> >> BTW... Why on Earth is it an uint8? Probably a space saving measure, but >> one I think is not really fruitful, because of probable paddings and >> alignments; making it an

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-02 Thread Premi, Sanjeev
> -Original Message- > From: Albert ARIBAUD [mailto:albert.arib...@free.fr] > Sent: Thursday, December 02, 2010 12:30 PM > To: u-boot@lists.denx.de > Cc: Premi, Sanjeev; Wolfgang Denk > Subject: Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across > toolchain versio

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-02 Thread Premi, Sanjeev
> -Original Message- > From: Albert ARIBAUD [mailto:albert.arib...@free.fr] > Sent: Thursday, December 02, 2010 1:21 PM > To: Wolfgang Denk > Cc: u-boot@lists.denx.de; Premi, Sanjeev > Subject: Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across > toolchain vers

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-02 Thread Wolfgang Denk
Dear Albert ARIBAUD, In message <4cf74fed.2030...@free.fr> you wrote: > > BTW... Why on Earth is it an uint8? Probably a space saving measure, but > one I think is not really fruitful, because of probable paddings and > alignments; making it an int would allow using smsc_id directly as > value

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-01 Thread Albert ARIBAUD
Hi Wolfgang, Le 02/12/2010 08:34, Wolfgang Denk a écrit : > Dear Albert ARIBAUD, > > In message<4cf743e6.60...@free.fr> you wrote: >> >> Starting with the fact that the linker issue is only for one board, >> omap3_evm, I looked up the board-specific code. First thing that I >> noticed was >> >>

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-01 Thread Wolfgang Denk
Dear Albert ARIBAUD, In message <4cf743e6.60...@free.fr> you wrote: > > Starting with the fact that the linker issue is only for one board, > omap3_evm, I looked up the board-specific code. First thing that I > noticed was > > static u8 omap3_evm_version; > > I changed this to > >

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-01 Thread Albert ARIBAUD
Le 01/12/2010 22:39, Albert ARIBAUD a écrit : > This one is a conundrum. > > Using 2010q1, building omap3_evm causes a linker warning > "arm-none-linux-gnueabi-ld: u-boot: section .bss vma 0x8003e8f0 overlaps > previous sections" while building omap3_beagle does not cause any linker > warning. > >

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-01 Thread Albert ARIBAUD
This one is a conundrum. Using 2010q1, building omap3_evm causes a linker warning "arm-none-linux-gnueabi-ld: u-boot: section .bss vma 0x8003e8f0 overlaps previous sections" while building omap3_beagle does not cause any linker warning. Both boards use the same armv7 u-boot.lds and have a .bss

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-01 Thread Wolfgang Denk
Dear "Premi, Sanjeev", In message you wrote: > > 1) why the current metod produces different errors across > different toolchain versions. Different tool chains may prvide different quality of optimizations (and bugs), resulting in differnt variable locations. With one memory map y

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-01 Thread Wolfgang Denk
Dear "Premi, Sanjeev", In message you wrote: > > The u-boot built after this change alone gets stuck somewhere after > dram_init(). However, removing the "sort" from LIBS (as suggested > by Wolfgang), the u-boot comes up fine on the omap3_evm. I did NOT suggest that. This was just intended as d

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-01 Thread Premi, Sanjeev
> -Original Message- > From: Albert ARIBAUD [mailto:albert.arib...@free.fr] > Sent: Thursday, December 02, 2010 12:07 AM > To: Premi, Sanjeev > Cc: u-boot@lists.denx.de > Subject: Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across > toolchain versions [snip]..

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-01 Thread Albert ARIBAUD
Le 01/12/2010 19:19, Premi, Sanjeev a écrit : >> If you find another way to do this overlay yet end up producing a >> different binary, I'll be interested in the result, but I >> honestly don't >> think you will find any. > > [sp] I am trying on this. Getting back to linker scripts after many >

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-01 Thread Albert ARIBAUD
Le 01/12/2010 19:23, Premi, Sanjeev a écrit : > [sp] Had a quick question - hence separate mail. > > Do we really need to preserve section ".dynsym" in the final > binary. OR are we okay with single section that contains > contents from both? A single section should be OK. Amic

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-01 Thread Premi, Sanjeev
> -Original Message- > From: Albert ARIBAUD [mailto:albert.arib...@free.fr] > Sent: Wednesday, December 01, 2010 11:02 PM > To: Premi, Sanjeev > Cc: u-boot@lists.denx.de > Subject: Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across > toolchain versions > >

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-01 Thread Premi, Sanjeev
> -Original Message- > From: Albert ARIBAUD [mailto:albert.arib...@free.fr] > Sent: Wednesday, December 01, 2010 11:02 PM > To: Premi, Sanjeev > Cc: u-boot@lists.denx.de > Subject: Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across > toolchain versions > >

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-01 Thread Albert ARIBAUD
Le 01/12/2010 18:19, Premi, Sanjeev a écrit : >> Nak -- what we want to overlay is .bss on one hand, and >> .rel.dyn *plus* >> .dynsym on the other hand; OVERLAY { ... } does not allow this. > > [sp] From the earlier discussion, I inferred the overlay was supposed > to be .rel.dyn and .bss.

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-01 Thread Premi, Sanjeev
> -Original Message- > From: u-boot-boun...@lists.denx.de > [mailto:u-boot-boun...@lists.denx.de] On Behalf Of Albert ARIBAUD > Sent: Wednesday, December 01, 2010 10:43 PM > To: u-boot@lists.denx.de > Subject: Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across &g

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-01 Thread Albert ARIBAUD
Le 01/12/2010 16:17, Sanjeev Premi a écrit : > This patch fixes the linker problems noticed while > building the omap3_evm with Codesourcery toolchains > 2009q1, 2009q3 and 2010q1. > > The compilation was tested as success for both > omap3_evm and omap3_beagle with toolchain versions > 2009q1 and 2

Re: [U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-01 Thread Premi, Sanjeev
> -Original Message- > From: Premi, Sanjeev > Sent: Wednesday, December 01, 2010 8:47 PM > To: u-boot@lists.denx.de > Cc: Premi, Sanjeev > Subject: [PATCH] ARMv7: Fix linker errors across toolchain versions > > This patch fixes the linker problems noticed while > building the omap3_evm wi

[U-Boot] [PATCH] ARMv7: Fix linker errors across toolchain versions

2010-12-01 Thread Sanjeev Premi
This patch fixes the linker problems noticed while building the omap3_evm with Codesourcery toolchains 2009q1, 2009q3 and 2010q1. The compilation was tested as success for both omap3_evm and omap3_beagle with toolchain versions 2009q1 and 2010q1. [1] http://marc.info/?l=u-boot&m=129104332808386&