Re: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-10-05 Thread Marek Vasut
On Thursday, October 06, 2011 12:02:17 AM Scott Wood wrote: > On 10/05/2011 04:44 PM, Tom Rini wrote: > > On Thu, Sep 22, 2011 at 1:52 AM, Marek Vasut wrote: > >> On Wednesday, September 21, 2011 01:31:28 AM Scott Wood wrote: > >>> Is it really such a burden to put something like > >>> > >>> #def

Re: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-10-05 Thread Scott Wood
On 10/05/2011 04:44 PM, Tom Rini wrote: > On Thu, Sep 22, 2011 at 1:52 AM, Marek Vasut wrote: >> On Wednesday, September 21, 2011 01:31:28 AM Scott Wood wrote: >>> Is it really such a burden to put something like >>> >>> #define CONFIG_SPL_ARCH_CPU >>> >>> in your board config header? >> >> Yes it

Re: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-10-05 Thread Tom Rini
On Thu, Sep 22, 2011 at 1:52 AM, Marek Vasut wrote: > On Wednesday, September 21, 2011 01:31:28 AM Scott Wood wrote: >> On 09/20/2011 04:30 PM, Marek Vasut wrote: >> > On Tuesday, September 20, 2011 11:23:01 PM Scott Wood wrote: >> >> On 09/20/2011 04:16 PM, Marek Vasut wrote: >> >>> On Tuesday, S

Re: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-09-22 Thread Marek Vasut
On Wednesday, September 21, 2011 01:31:28 AM Scott Wood wrote: > On 09/20/2011 04:30 PM, Marek Vasut wrote: > > On Tuesday, September 20, 2011 11:23:01 PM Scott Wood wrote: > >> On 09/20/2011 04:16 PM, Marek Vasut wrote: > >>> On Tuesday, September 20, 2011 09:12:08 PM Scott Wood wrote: > On 0

Re: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-09-20 Thread Scott Wood
On 09/20/2011 04:30 PM, Marek Vasut wrote: > On Tuesday, September 20, 2011 11:23:01 PM Scott Wood wrote: >> On 09/20/2011 04:16 PM, Marek Vasut wrote: >>> On Tuesday, September 20, 2011 09:12:08 PM Scott Wood wrote: On 09/19/2011 05:31 PM, Marek Vasut wrote: > Then you adjust the makefile

Re: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-09-20 Thread Marek Vasut
On Tuesday, September 20, 2011 11:23:01 PM Scott Wood wrote: > On 09/20/2011 04:16 PM, Marek Vasut wrote: > > On Tuesday, September 20, 2011 09:12:08 PM Scott Wood wrote: > >> On 09/19/2011 05:31 PM, Marek Vasut wrote: > >>> Then you adjust the makefile there by ifdef CONFIG_SPL_BUILD > >> > >> It

Re: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-09-20 Thread Scott Wood
On 09/20/2011 04:16 PM, Marek Vasut wrote: > On Tuesday, September 20, 2011 09:12:08 PM Scott Wood wrote: >> On 09/19/2011 05:31 PM, Marek Vasut wrote: >>> Then you adjust the makefile there by ifdef CONFIG_SPL_BUILD >> >> It's not quite that simple, since different SPLs will have different >> requ

Re: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-09-20 Thread Marek Vasut
On Tuesday, September 20, 2011 09:12:08 PM Scott Wood wrote: > On 09/19/2011 05:31 PM, Marek Vasut wrote: > > On Monday, September 19, 2011 08:13:45 PM Scott Wood wrote: > >> On 09/16/2011 05:48 PM, Marek Vasut wrote: > >>> The stuff in arch/arm/cpu should be exactly what you need, nothing more > >

Re: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-09-20 Thread Scott Wood
On 09/19/2011 05:31 PM, Marek Vasut wrote: > On Monday, September 19, 2011 08:13:45 PM Scott Wood wrote: >> On 09/16/2011 05:48 PM, Marek Vasut wrote: >>> The stuff in arch/arm/cpu should be exactly what you need, nothing more ! >> >> This is "spl/", not "arch/arm/spl/", so let's not reason from on

Re: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-09-19 Thread Marek Vasut
On Monday, September 19, 2011 08:13:45 PM Scott Wood wrote: > On 09/16/2011 05:48 PM, Marek Vasut wrote: > > On Saturday, September 17, 2011 12:07:52 AM Scott Wood wrote: > >> You have no idea why I'd like to be extremely selective with what I > >> include in a 4K binary? > > > > That I do underst

Re: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-09-19 Thread Scott Wood
On 09/16/2011 05:48 PM, Marek Vasut wrote: > On Saturday, September 17, 2011 12:07:52 AM Scott Wood wrote: >> You have no idea why I'd like to be extremely selective with what I >> include in a 4K binary? > > That I do understand -- but that kind of selection is there. >> >> It's not about avoidin

Re: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-09-16 Thread Marek Vasut
On Saturday, September 17, 2011 12:07:52 AM Scott Wood wrote: > On 09/16/2011 04:47 PM, Marek Vasut wrote: > > On Friday, September 16, 2011 11:42:50 PM Scott Wood wrote: > >> On 09/16/2011 04:38 PM, Marek Vasut wrote: > >>> On Friday, September 16, 2011 09:49:28 PM Scott Wood wrote: > Still,

Re: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-09-16 Thread Scott Wood
On 09/16/2011 04:47 PM, Marek Vasut wrote: > On Friday, September 16, 2011 11:42:50 PM Scott Wood wrote: >> On 09/16/2011 04:38 PM, Marek Vasut wrote: >>> On Friday, September 16, 2011 09:49:28 PM Scott Wood wrote: Still, this seems hackish. Shouldn't the control be on specific files tha

Re: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-09-16 Thread Marek Vasut
On Friday, September 16, 2011 11:42:50 PM Scott Wood wrote: > On 09/16/2011 04:38 PM, Marek Vasut wrote: > > On Friday, September 16, 2011 09:49:28 PM Scott Wood wrote: > >> Still, this seems hackish. Shouldn't the control be on specific files > >> that you include, not directories? > > > > I don

Re: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-09-16 Thread Scott Wood
On 09/16/2011 04:38 PM, Marek Vasut wrote: > On Friday, September 16, 2011 09:49:28 PM Scott Wood wrote: >> Still, this seems hackish. Shouldn't the control be on specific files >> that you include, not directories? > > I don't think so ... why ? That's how the main U-Boot build does it... More

Re: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-09-16 Thread Marek Vasut
On Friday, September 16, 2011 09:49:28 PM Scott Wood wrote: > On 09/15/2011 06:17 PM, Marek Vasut wrote: > > On Friday, September 16, 2011 12:57:44 AM Scott Wood wrote: > >> On 09/11/2011 11:03 PM, Marek Vasut wrote: > >>> Introduce CONFIG_SPL_NO_CPU_SUPPORT_CODE to avoid compiling the CPU > >>> su

Re: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-09-16 Thread Scott Wood
On 09/15/2011 06:17 PM, Marek Vasut wrote: > On Friday, September 16, 2011 12:57:44 AM Scott Wood wrote: >> On 09/11/2011 11:03 PM, Marek Vasut wrote: >>> Introduce CONFIG_SPL_NO_CPU_SUPPORT_CODE to avoid compiling the CPU >>> support library. This can be useful on some setups. >>> >>> Signed-off-b

Re: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-09-15 Thread Marek Vasut
On Friday, September 16, 2011 12:57:44 AM Scott Wood wrote: > On 09/11/2011 11:03 PM, Marek Vasut wrote: > > Introduce CONFIG_SPL_NO_CPU_SUPPORT_CODE to avoid compiling the CPU > > support library. This can be useful on some setups. > > > > Signed-off-by: Marek Vasut > > Cc: Stefano Babic > > Cc

Re: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-09-15 Thread Scott Wood
On 09/11/2011 11:03 PM, Marek Vasut wrote: > Introduce CONFIG_SPL_NO_CPU_SUPPORT_CODE to avoid compiling the CPU support > library. This can be useful on some setups. > > Signed-off-by: Marek Vasut > Cc: Stefano Babic > Cc: Wolfgang Denk > Cc: Detlev Zundel > Cc: Chander Kashyap But you didn

[U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library

2011-09-11 Thread Marek Vasut
Introduce CONFIG_SPL_NO_CPU_SUPPORT_CODE to avoid compiling the CPU support library. This can be useful on some setups. Signed-off-by: Marek Vasut Cc: Stefano Babic Cc: Wolfgang Denk Cc: Detlev Zundel Cc: Chander Kashyap --- spl/Makefile |7 ++- 1 files changed, 6 insertions(+), 1 de