From: Scott Wood <scottwood@freescale.com>
To: David Gibson <david@gibson.dropbear.id.au>
Cc: linuxppc-dev@ozlabs.org, paulus@samba.org
Subject: Re: [PATCH 19/19] bootwrapper: compatibility layer for old U-Boots (a.k.a. cuImage, cuboot)
Date: Mon, 12 Feb 2007 10:52:59 -0600 [thread overview]
Message-ID: <45D09B6B.8040309@freescale.com> (raw)
In-Reply-To: <20070210010237.GF8276@localhost.localdomain>
David Gibson wrote:
> On Fri, Feb 09, 2007 at 11:11:29AM -0600, Scott Wood wrote:
>>They come from platform files such as cuboot-83xx.c, not (directly) from
>>the kernel config.
>
>
> /me looks again...
>
> .c files containing only #defines? My brain hurts.
So does mine. It wasn't my idea...
> Better to pass these in as -D flags from the gcc invocation within the
> wrapper script, depending on the given platform (for the platform .o
> *only*).
I was explicitly told by Paul that it had to be link-time, not compile-time.
See "http://ozlabs.org/pipermail/linuxppc-dev/2007-January/030532.html".
Unless you're saying that the "wrapper" script should do the compilation
itself, rather than just linking as it currently does... in which case
I'd really like to know what real benefit we're getting out of all this
silliness.
> Oh, and please don't call them CONFIG_* or everyone will
> assume, like me, that they come from the kernel's overall config.h.
Well, they are related, and I'd have to make ppcboot.h diverge even more
from u-boot's version to call it something else.
But I can rename it TARGET_* or something if it'd make y'all happier.
-Scott
next prev parent reply other threads:[~2007-02-12 16:52 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-02-07 23:00 [PATCH 00/19] cuboot bootwrapper patchset Scott Wood
2007-02-07 23:01 ` [PATCH 01/19] bootwrapper: Add stddef.h to ops.h Scott Wood
2007-02-07 23:01 ` [PATCH 02/19] bootwrapper: Set -msoft-float and assembler target options Scott Wood
2007-02-07 23:01 ` [PATCH 03/19] bootwrapper: Remove OF-isms Scott Wood
2007-02-07 23:01 ` [PATCH 04/19] bootwrapper: Add ft_root_node() Scott Wood
2007-02-07 23:01 ` [PATCH 05/19] bootwrapper: Rename ft_node_add() to ft_get_phandle() Scott Wood
2007-02-07 23:01 ` [PATCH 06/19] bootwrapper: Make ft_get_phandle() accept and return NULL Scott Wood
2007-02-07 23:01 ` [PATCH 07/19] bootwrapper: Preserve the pp pointer in ft_make_space() when calling ft_reorder() Scott Wood
2007-02-07 23:01 ` [PATCH 08/19] bootwrapper: Modify *pp, not *p, in ft_shuffle() Scott Wood
2007-02-07 23:01 ` [PATCH 09/19] bootwrapper: Rename p and pp to anchor and anchorptr Scott Wood
2007-02-07 23:01 ` [PATCH 10/19] bootwrapper: Use map_string() instead of lookup_string() in ft_prop() Scott Wood
2007-02-07 23:01 ` [PATCH 11/19] bootwrapper: Add ft_find_device_rel() Scott Wood
2007-02-08 1:11 ` David Gibson
2007-02-08 17:14 ` Scott Wood
2007-02-07 23:01 ` [PATCH 12/19] bootwrapper: Refactor ft_get_prop() into internal and external functions Scott Wood
2007-02-07 23:01 ` [PATCH 13/19] bootwrapper: Make ft_get_parent() return a phandle, and NULL if already top-level Scott Wood
2007-02-07 23:01 ` [PATCH 14/19] bootwrapper: Add ft_find_node_by_prop_value() Scott Wood
2007-02-07 23:01 ` [PATCH 15/19] bootwrapper: Add initrd information to the device tree in ft_finalize() Scott Wood
2007-02-09 23:02 ` Mark A. Greer
2007-02-10 0:37 ` David Gibson
2007-02-12 16:42 ` Scott Wood
2007-02-13 4:29 ` David Gibson
2007-02-13 16:07 ` Scott Wood
2007-02-14 4:43 ` David Gibson
2007-02-07 23:01 ` [PATCH 16/19] bootwrapper: Make ft_create_node() pay attention to the parent parameter Scott Wood
2007-02-07 23:01 ` [PATCH 17/19] bootwrapper: Add dt_ops methods Scott Wood
2007-02-09 23:05 ` Mark A. Greer
2007-02-07 23:01 ` [PATCH 18/19] bootwrapper: Add xlate_reg(), and use it to find serial registers Scott Wood
2007-02-09 23:07 ` Mark A. Greer
2007-02-12 16:45 ` Scott Wood
2007-02-12 21:10 ` Josh Boyer
2007-02-13 4:30 ` David Gibson
2007-02-13 16:00 ` Scott Wood
2007-02-07 23:01 ` [PATCH 19/19] bootwrapper: compatibility layer for old U-Boots (a.k.a. cuImage, cuboot) Scott Wood
2007-02-08 21:15 ` David Gibson
2007-02-09 17:11 ` Scott Wood
2007-02-10 1:02 ` David Gibson
2007-02-12 16:52 ` Scott Wood [this message]
-- strict thread matches above, loose matches on Subject: below --
2007-02-12 20:54 Scott Wood
2007-02-12 21:38 ` Geoff Levand
2007-02-13 16:13 ` Scott Wood
2007-02-13 17:32 ` Geoff Levand
2007-02-18 0:22 ` David Gibson
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=45D09B6B.8040309@freescale.com \
--to=scottwood@freescale.com \
--cc=david@gibson.dropbear.id.au \
--cc=linuxppc-dev@ozlabs.org \
--cc=paulus@samba.org \
/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;
as well as URLs for NNTP newsgroup(s).