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 to get

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 compile

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 removes them.

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 yesterday

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 going

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

[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 pty...@xes-inc.com 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 +++

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 pty...@xes-inc.com Date: Wed Sep 29 14:05:56 2010 -0500 85xx: Use gc-sections to reduce image size I'd guess