From: Marek Vasut <marek.vasut@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library
Date: Sat, 17 Sep 2011 00:48:36 +0200 [thread overview]
Message-ID: <201109170048.36433.marek.vasut@gmail.com> (raw)
In-Reply-To: <4E73C8B8.3040505@freescale.com>
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, 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 specifically, the
> >> config.h controls should be on features, and the makefiles should decide
> >> which (if any) files are required for that feature. If there are no
> >> files from arch/$(ARCH)/cpu/$(CPU) needed, then we get an empty
> >> lib$(CPU).o -- nothing special needed to avoid it.
> >
> > Yes, but you basically _always_ want that CPU support code in ... and I
> > have no idea why you'd like to avoid particular files.
>
> 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 avoiding particular files. It's about including
> *nothing* but what is explicitly asked for through some SPL-specific
> CONFIG symbol. Maybe that includes everything in arch/$(ARCH)/cpu.
> Maybe it includes nothing in there. More likely, it includes just a
> fraction of it.
The stuff in arch/arm/cpu should be exactly what you need, nothing more !
>
> If your answer is gc-sections, why do you need to drop the whole
> directory? And why waste time building entire files that we know we
> don't want? Why waste time debugging where the sudden bloat came from
> instead of getting a simple and clear undefined-symbol error?
>
> -Scott
next prev parent reply other threads:[~2011-09-16 22:48 UTC|newest]
Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-09-12 4:03 [U-Boot] [PATCH 0/2] SPL improvements Marek Vasut
2011-09-12 4:03 ` [U-Boot] [PATCH 1/2 RESEND] SPL: Make path to start.S configurable Marek Vasut
2011-10-05 19:08 ` Wolfgang Denk
2011-10-05 20:07 ` Wolfgang Denk
2011-10-05 20:15 ` Wolfgang Denk
2011-09-12 4:03 ` [U-Boot] [PATCH 2/2 RESEND] SPL: Allow user to disable CPU support library Marek Vasut
2011-09-15 22:57 ` Scott Wood
2011-09-15 23:17 ` Marek Vasut
2011-09-16 19:49 ` Scott Wood
2011-09-16 21:38 ` Marek Vasut
2011-09-16 21:42 ` Scott Wood
2011-09-16 21:47 ` Marek Vasut
2011-09-16 22:07 ` Scott Wood
2011-09-16 22:48 ` Marek Vasut [this message]
2011-09-19 18:13 ` Scott Wood
2011-09-19 22:31 ` Marek Vasut
2011-09-20 19:12 ` Scott Wood
2011-09-20 21:16 ` Marek Vasut
2011-09-20 21:23 ` Scott Wood
2011-09-20 21:30 ` Marek Vasut
2011-09-20 23:31 ` Scott Wood
2011-09-22 8:52 ` Marek Vasut
2011-10-05 21:44 ` Tom Rini
2011-10-05 22:02 ` Scott Wood
2011-10-05 22:20 ` Marek Vasut
2011-10-06 0:13 ` [U-Boot] [PATCH 1/2] " Marek Vasut
2011-10-06 0:13 ` [U-Boot] [PATCH 2/2] SPL: Allow ARM926EJS to avoid compiling in the CPU support code Marek Vasut
2011-10-18 21:33 ` Albert ARIBAUD
2011-10-18 22:30 ` Marek Vasut
2011-10-21 20:44 ` Marek Vasut
2011-10-21 21:52 ` Albert ARIBAUD
2011-10-21 22:00 ` Marek Vasut
2011-10-21 22:44 ` Albert ARIBAUD
2011-10-21 22:46 ` Marek Vasut
2011-10-21 23:08 ` Albert ARIBAUD
2011-10-21 23:45 ` Marek Vasut
2011-10-22 0:04 ` Tom Rini
2011-10-22 0:19 ` Marek Vasut
2011-10-22 0:41 ` Albert ARIBAUD
2011-10-22 1:20 ` Marek Vasut
2011-10-22 7:05 ` Albert ARIBAUD
2011-10-24 10:14 ` [U-Boot] [PATCH 2/2 V2] " Marek Vasut
2011-11-03 0:05 ` Marek Vasut
2011-11-04 13:59 ` Marek Vasut
2011-11-08 21:15 ` Albert ARIBAUD
2011-10-06 15:54 ` [U-Boot] [PATCH 1/2] SPL: Allow user to disable CPU support library Scott Wood
2011-10-06 23:35 ` Marek Vasut
2011-09-12 4:12 ` [U-Boot] [PATCH 0/2] SPL improvements Marek Vasut
2011-10-05 11:04 ` Marek Vasut
2011-10-05 19:14 ` Wolfgang Denk
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201109170048.36433.marek.vasut@gmail.com \
--to=marek.vasut@gmail.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox