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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.