Re: [U-Boot] patch for gc-sections

2010-11-10 Thread Haiying Wang
On Thu, 2010-11-04 at 14:22 -0400, Haiying Wang wrote: > On Thu, 2010-11-04 at 09:36 -0700, Peter Tyser wrote: > > Glad to hear. I'll submit an official patch shortly. Just to make > > sure, have you tried running one of the nand-spl images after the > > patch > > I sent yesterday? It'd be good

Re: [U-Boot] patch for gc-sections

2010-11-04 Thread Haiying Wang
On Thu, 2010-11-04 at 09:36 -0700, Peter Tyser wrote: > Glad to hear. I'll submit an official patch shortly. Just to make > sure, have you tried running one of the nand-spl images after the > patch > I sent yesterday? It'd be good to get confirmation that the > -gc-sections doesn't have any acci

Re: [U-Boot] patch for gc-sections

2010-11-04 Thread Peter Tyser
> Yes, you are right, the size is down. I only noticed the u-boot-spl size > by using ls -l, not via size. Now I get the same result as yours. > > It is very good to see this trim-down size because I know some board > developers are fighting with the 4k limitation of the nand-spl size. Are > you

Re: [U-Boot] patch for gc-sections

2010-11-04 Thread Haiying Wang
On Thu, 2010-11-04 at 08:45 -0700, Peter Tyser wrote: > Can you explain what you mean? The binary needs to be 4K, right? So > it > can't be trimmed down. But there should be more available space in > that > 4K region, eg (all tests on MPC8536DS_NAND_config): > > /* After apply my patch sent yes

Re: [U-Boot] patch for gc-sections

2010-11-04 Thread Peter Tyser
On Thu, 2010-11-04 at 10:19 -0400, Haiying Wang wrote: > On Wed, 2010-11-03 at 13:38 -0700, Peter Tyser wrote: > > I'd guess none of the functions in the SPL binary are referenced in > > the > > linker script or linker command line, so the linker thinks none of > > them > > are necessary and remove

Re: [U-Boot] patch for gc-sections

2010-11-04 Thread Haiying Wang
On Wed, 2010-11-03 at 13:38 -0700, Peter Tyser wrote: > I'd guess none of the functions in the SPL binary are referenced in > the > linker script or linker command line, so the linker thinks none of > them > are necessary and removes them. > > Can you try the following patch: > I did a quick compi

Re: [U-Boot] patch for gc-sections

2010-11-03 Thread Peter Tyser
On Wed, 2010-11-03 at 15:07 -0400, Haiying Wang wrote: > Peter, > > Do you have any idea on why your commit: > " > commit fbe53f59bd40b3b1ab66dc98859e26589d64d1b7 > Author: Peter Tyser > Date: Wed Sep 29 14:05:56 2010 -0500 > > 85xx: Use gc-sections to reduce image size I'd guess none of

[U-Boot] patch for gc-sections

2010-11-03 Thread Haiying Wang
Peter, Do you have any idea on why your commit: " commit fbe53f59bd40b3b1ab66dc98859e26589d64d1b7 Author: Peter Tyser Date: Wed Sep 29 14:05:56 2010 -0500 85xx: Use gc-sections to reduce image size which made change to : " --- a/arch/powerpc/cpu/mpc85xx/config.mk +++ b/arch/powerpc/c