Re: [U-Boot] [PATCH 7/7] add gc-sections to TPL boot

2011-01-28 Thread Scott Wood
On Thu, 27 Jan 2011 23:58:10 -0500 haiying.w...@freescale.com wrote: From: Haiying Wang haiying.w...@freescale.com Signed-off-by: Haiying Wang haiying.w...@freescale.com --- arch/powerpc/config.mk |4 config.mk |7 ++- 2 files changed, 10 insertions(+), 1

Re: [U-Boot] [PATCH 7/7] add gc-sections to TPL boot

2011-01-28 Thread Haiying Wang
On Fri, 2011-01-28 at 11:36 -0600, Scott Wood wrote: On Thu, 27 Jan 2011 23:58:10 -0500 haiying.w...@freescale.com wrote: From: Haiying Wang haiying.w...@freescale.com Signed-off-by: Haiying Wang haiying.w...@freescale.com --- arch/powerpc/config.mk |4 config.mk

Re: [U-Boot] [PATCH 7/7] add gc-sections to TPL boot

2011-01-28 Thread Albert ARIBAUD
Le 28/01/2011 19:08, Haiying Wang a écrit : I see patch 3/8, 4/8, 5/8, and 7/7. Where are the rest? Sorry, patch 7/7 is a wrong number here. I kept the patch # as 3/8/, 4/8, 5/8, 8/8 to be consistent with the order in the patchset(v2) I submitted in last December. I thought it would be

Re: [U-Boot] [PATCH 7/7] add gc-sections to TPL boot

2011-01-28 Thread Scott Wood
On Fri, 28 Jan 2011 13:08:30 -0500 Haiying Wang haiying.w...@freescale.com wrote: On Fri, 2011-01-28 at 11:36 -0600, Scott Wood wrote: On Thu, 27 Jan 2011 23:58:10 -0500 haiying.w...@freescale.com wrote: From: Haiying Wang haiying.w...@freescale.com Signed-off-by: Haiying Wang

Re: [U-Boot] [PATCH 7/7] add gc-sections to TPL boot

2011-01-28 Thread Haiying Wang
On Fri, 2011-01-28 at 12:30 -0600, Scott Wood wrote: diff --git a/config.mk b/config.mk index 5147c35..d7bb07f 100644 --- a/config.mk +++ b/config.mk @@ -260,8 +260,13 @@ $(obj)%.s: %.c #

Re: [U-Boot] [PATCH 7/7] add gc-sections to TPL boot

2011-01-28 Thread Scott Wood
On Fri, 28 Jan 2011 13:46:06 -0500 Haiying Wang haiying.w...@freescale.com wrote: On Fri, 2011-01-28 at 12:30 -0600, Scott Wood wrote: I think --gc-sections should go in LDFLAGS_u-boot instead. LDFLAGS_u-boot has --gc-sections already, I did not change it. It looks like LDFLAGS_u-boot may

Re: [U-Boot] [PATCH 7/7] add gc-sections to TPL boot

2011-01-28 Thread Haiying Wang
On Fri, 2011-01-28 at 13:46 -0500, Haiying Wang wrote: In any case, I don't think we want different behavior here based on whether we have TPL. Either LDFLAGS is used in partial linking, or it's not. I don't understand why LDFLAGS was added here in patch

Re: [U-Boot] [PATCH 7/7] add gc-sections to TPL boot

2011-01-28 Thread Scott Wood
On Fri, 28 Jan 2011 14:07:09 -0500 Haiying Wang haiying.w...@freescale.com wrote: On Fri, 2011-01-28 at 13:46 -0500, Haiying Wang wrote: In any case, I don't think we want different behavior here based on whether we have TPL. Either LDFLAGS is used in partial linking, or it's not. I

[U-Boot] [PATCH 7/7] add gc-sections to TPL boot

2011-01-27 Thread Haiying.Wang
From: Haiying Wang haiying.w...@freescale.com Signed-off-by: Haiying Wang haiying.w...@freescale.com --- arch/powerpc/config.mk |4 config.mk |7 ++- 2 files changed, 10 insertions(+), 1 deletions(-) diff --git a/arch/powerpc/config.mk b/arch/powerpc/config.mk