From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v5 4/5] OMAP: networking support for SPL
Date: Mon, 17 Sep 2012 11:07:13 -0700 [thread overview]
Message-ID: <505766D1.2060101@ti.com> (raw)
In-Reply-To: <CAA3CPjWPVPhc72mJnCvSSkk9zv1980SwGv+u0mo05ekFSuX=DA@mail.gmail.com>
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 09/17/12 10:54, Ilya Yanok wrote:
> Hi Tom,
>
> On Mon, Sep 17, 2012 at 9:04 PM, Tom Rini <trini@ti.com
> <mailto:trini@ti.com>> wrote:
>
>> I agree it's not the best place... config_cmd_spl.h sounds a
>> little bit crazy as we don't have any commands at all in SPL...
>
>
> How about config_uncmd_spl.h then and a nice big comment up top
>
>
> Well, it will be at least less confusing...
>
>
> explaining what we're doing. Or can we take another stab at
> seeing why some stuff isn't being garbage collected? If
> garbage_collected_func_a calls never_seen_while_linking_func, we
> still succeed since we garbage collect the first func. There's
> just the gcc issue I've noted before about strings.
>
>
> That's not really about garbage collection in this case (net-spl).
> I want to disable some functionality of generic net code not some
> stuff used only by commands implementation. The confusion comes
> from the fact that this code is protected by CONFIG_CMD_* defines.
So I guess the code construct is roughly:
function_we_need(...) {
#ifdef CONFIG_CMD_A
... stuff_spl_does_not_need();
#endif
...
}
? Otherwise we would end up building files we don't use, but then all
of the un-used code gets garbage collected.
- --
Tom
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://www.enigmail.net/
iQIbBAEBAgAGBQJQV2bQAAoJENk4IS6UOR1WXNAP+MLdJeSu0x5xpbKEyURT6ISc
hKwYFO9esQRTnssp+0efz+RlYgAV1fet5pxhtn+RqMvA5XfWT413yZoEOg7XzsBE
TYcVs63TRuTDI5o0qgOryAQifY6GTFj1hw4BecAvODErJl7D6dtyGJjg4QWVUP4M
C1AFb0132ILlj5MSXCAC3d0ZYVNDqkIGl0BNGUSwOmZ8OT7IHi/x1tOUaKibOin6
wytrPXFwNxjvNeDVXe1Ot89Kx/t+kbNh6LDJLoIPK8SasuKdrtDxiQLCMMGGxT7R
vAS7//AqupWuzyWPdcQvM+YJetDKk/iYCy1FpYeo0yY1nKBHgh85lr6yBqjRnwKV
Tbh9ny41J1xd4AhIbRvEXirReZ/Zu3vEr01Qe+ddqgY+2mol0wucyhY2dgWlAn5G
jRCARRpEd8tIgzWBN5lxosHq+v7iltAOXZT/hHZpv19zzZ2KK3xG4CiqaZZvRS2B
DHWj2dOZW7CJGhCpYapLtCmxhh+M4X6YGflNCkiuQV9NGZjE3PhGh0N9QRQXBQjU
CzmY/cqWXG7QZ6NIlKyEG9nZMuFggSW2miHINGmk4G68rH6QfCpGilucOkN2LDjt
sE60qAIxRmkW2emn12V9dBU5CTyhgVetr0nYBD0PGO366Z+Xoq9sODG2aiQYdNTF
250WBWG235mtco6BNUw=
=v55G
-----END PGP SIGNATURE-----
next prev parent reply other threads:[~2012-09-17 18:07 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-05 21:21 [U-Boot] [PATCH v4 0/5] OMAP: SPL networking support Ilya Yanok
2012-08-05 21:21 ` [U-Boot] [PATCH v4 1/5] net/bootp: add VCI support for BOOTP also Ilya Yanok
2012-08-10 17:57 ` Ilya Yanok
2012-08-29 21:01 ` Joe Hershberger
2012-08-05 21:21 ` [U-Boot] [PATCH v4 2/5] spl: don't mark __u_boot_cmd* as undefined Ilya Yanok
2012-08-05 22:36 ` Tom Rini
2012-08-06 15:10 ` Ilya Yanok
2012-08-06 15:30 ` Tom Rini
2012-08-06 15:31 ` Ilya Yanok
2012-08-06 17:10 ` Tom Rini
2012-08-06 19:15 ` Ilya Yanok
2012-08-06 20:52 ` Tom Rini
2012-08-06 21:11 ` Ilya Yanok
2012-08-07 8:12 ` Ilya Yanok
2012-08-05 21:21 ` [U-Boot] [PATCH v4 3/5] OMAP: spl: call timer_inti() from SPL Ilya Yanok
2012-08-05 22:35 ` Tom Rini
2012-08-06 15:02 ` Ilya Yanok
2012-08-06 15:11 ` Tom Rini
2012-08-06 18:49 ` [U-Boot] [PATCH] am33xx: Remove redundant timer config Tom Rini
2012-08-05 21:21 ` [U-Boot] [PATCH v4 4/5] OMAP: networking support for SPL Ilya Yanok
2012-08-07 8:07 ` [U-Boot] [PATCH v5 " Ilya Yanok
2012-08-29 21:25 ` Joe Hershberger
2012-08-29 22:28 ` Tom Rini
2012-09-17 9:55 ` Ilya Yanok
2012-09-17 17:04 ` Tom Rini
2012-09-17 17:54 ` Ilya Yanok
2012-09-17 18:07 ` Tom Rini [this message]
2012-09-17 18:10 ` Ilya Yanok
2012-09-17 18:16 ` Ilya Yanok
2012-09-17 18:19 ` Tom Rini
2012-09-17 18:36 ` Ilya Yanok
2012-08-05 21:21 ` [U-Boot] [PATCH v4 5/5] am335x_evm: enable networking in SPL Ilya Yanok
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=505766D1.2060101@ti.com \
--to=trini@ti.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