From: Segher Boessenkool <segher@kernel.crashing.org>
To: Scott Wood <scottwood@freescale.com>
Cc: linuxppc-dev@ozlabs.org,
Yoder Stuart-B08248 <stuart.yoder@freescale.com>
Subject: Re: [PATCH 06/17] Document the linux,network-index property.
Date: Fri, 23 Mar 2007 17:42:14 +0100 [thread overview]
Message-ID: <29089968dc071f4b6917428a439f735e@kernel.crashing.org> (raw)
In-Reply-To: <20070323150058.GA6060@ld0162-tx32.am.freescale.net>
>>> Segher, what is the the 'other alias' mechanism you are referring
>>> to that should be dropped? Is it this proposed linux,network-index
>>> property? or something else?
>>
>> Just the
>>
>> pic0: pic@700 {
>> ...
>> }
>>
>> labeling thing -- it becomes redundant when the flat tree
>> stuff would support OF-style aliases, so it can be phased
>> out then.
>
> How? You still need integer-encoded phandles, as that's how various
> properties are defined.
Of course. But DTC could derive them automatically.
> Why should every node that needs a phandle
> reference have to go in /aliases?
They don't. They don't need a label now, either. It's just
handy for many/most things to go in /aliases.
> They seem to be solving two different
> problems to me.
Yeah, one a superset of the other. Except for labels
on / into properties as David mentioned, which is a
flat-tree specific solution for a flat-tree specific
problem.
> As for using aliases for network indices, it could be done, but it'd be
> more complicated to implement due to string manipulation.
It's more complex because it's more flexible, yes. I think
it'll pay off -- and how complex is using a sprintf() really.
Segher
next prev parent reply other threads:[~2007-03-23 16:42 UTC|newest]
Thread overview: 92+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-03-16 17:26 [PATCH 00/17] bootwrapper/cuImage patches Scott Wood
2007-03-16 17:27 ` [PATCH 01/17] bootwrapper: Make ft_create_node() pay attention to the parent parameter Scott Wood
2007-03-17 1:23 ` David Gibson
2007-03-16 17:27 ` [PATCH 02/17] bootwrapper: Add dt_ops methods Scott Wood
2007-03-17 1:24 ` David Gibson
2007-03-16 17:27 ` [PATCH 03/17] bootwrapper: Add xlate_reg(), and use it to find serial registers Scott Wood
2007-03-20 3:50 ` David Gibson
2007-03-20 16:34 ` Scott Wood
2007-03-21 3:12 ` David Gibson
2007-03-21 13:49 ` Segher Boessenkool
2007-03-21 16:01 ` Scott Wood
2007-03-21 19:11 ` Segher Boessenkool
2007-03-16 17:27 ` [PATCH 04/17] bootwrapper: Make compression of the kernel image optional Scott Wood
2007-03-17 1:25 ` David Gibson
2007-03-21 15:03 ` Patch: " Milton Miller
2007-03-16 17:28 ` [PATCH 05/17] bootwrapper: misc device tree fixes Scott Wood
2007-03-21 1:51 ` David Gibson
2007-03-22 16:22 ` Scott Wood
2007-03-23 3:26 ` David Gibson
2007-03-21 13:22 ` Segher Boessenkool
2007-03-16 17:28 ` [PATCH 06/17] Document the linux,network-index property Scott Wood
2007-03-17 1:26 ` David Gibson
2007-03-21 13:30 ` Segher Boessenkool
2007-03-21 14:48 ` Yoder Stuart-B08248
2007-03-21 18:57 ` Segher Boessenkool
2007-03-22 0:29 ` David Gibson
2007-03-22 11:11 ` Segher Boessenkool
2007-03-23 3:19 ` David Gibson
2007-03-23 11:36 ` Segher Boessenkool
2007-03-23 23:59 ` David Gibson
2007-03-24 0:23 ` Segher Boessenkool
2007-03-23 15:00 ` Scott Wood
2007-03-23 16:42 ` Segher Boessenkool [this message]
2007-03-16 17:28 ` [PATCH 07/17] bootwrapper: Add dt_set_memory(), to fill in the /memory node Scott Wood
2007-03-17 1:26 ` David Gibson
2007-03-21 13:32 ` Segher Boessenkool
2007-03-21 23:42 ` David Gibson
2007-03-22 11:02 ` Segher Boessenkool
2007-03-23 3:24 ` David Gibson
2007-03-23 6:49 ` Stefan Roese
2007-03-23 7:40 ` David Gibson
2007-03-23 12:34 ` Segher Boessenkool
2007-03-16 17:28 ` [PATCH 08/17] bootwrapper: Make setprop accept a const buffer Scott Wood
2007-03-17 1:27 ` David Gibson
2007-03-16 17:28 ` [PATCH 09/17] bootwrapper: Add dt_set_cpu_clocks() Scott Wood
2007-03-17 1:28 ` David Gibson
2007-03-16 17:28 ` [PATCH 10/17] bootwrapper: Add dt_set_mac_addresses() Scott Wood
2007-03-17 1:31 ` David Gibson
2007-03-18 0:22 ` Timur Tabi
2007-03-18 11:56 ` David Gibson
2007-03-19 15:09 ` Timur Tabi
2007-03-20 3:59 ` David Gibson
2007-03-20 14:00 ` Timur Tabi
2007-03-21 2:54 ` David Gibson
2007-03-21 15:01 ` Timur Tabi
2007-03-21 15:25 ` Jerry Van Baren
2007-03-21 15:55 ` Timur Tabi
2007-03-22 0:06 ` David Gibson
2007-03-22 15:13 ` Timur Tabi
2007-03-22 15:15 ` Jon Loeliger
2007-03-23 3:22 ` David Gibson
2007-03-23 14:38 ` Timur Tabi
2007-03-23 16:37 ` Segher Boessenkool
2007-03-23 16:42 ` Timur Tabi
2007-03-23 16:51 ` Segher Boessenkool
2007-03-23 16:54 ` Timur Tabi
2007-03-23 23:17 ` David Gibson
2007-03-20 18:17 ` Jon Loeliger
2007-03-21 13:45 ` Segher Boessenkool
2007-03-21 15:15 ` Timur Tabi
2007-03-21 19:07 ` Segher Boessenkool
2007-03-21 19:10 ` Timur Tabi
2007-03-21 19:34 ` Segher Boessenkool
2007-03-21 19:39 ` Timur Tabi
2007-03-21 19:48 ` Segher Boessenkool
2007-03-21 20:03 ` Timur Tabi
2007-03-21 20:22 ` Olof Johansson
2007-03-21 21:58 ` Segher Boessenkool
2007-03-21 21:54 ` Segher Boessenkool
2007-03-21 20:34 ` Doug Maxey
2007-03-21 22:01 ` Segher Boessenkool
2007-03-21 13:38 ` Segher Boessenkool
2007-03-21 15:09 ` Timur Tabi
2007-03-16 17:28 ` [PATCH 11/17] bootwrapper: Make set_cmdline non-static, and accept a const buffer Scott Wood
2007-03-17 1:36 ` David Gibson
2007-03-16 17:29 ` [PATCH 12/17] bootwrapper: Make set_cmdline() create /chosen if it doesn't exist Scott Wood
2007-03-21 13:32 ` Segher Boessenkool
2007-03-16 17:29 ` [PATCH 13/17] bootwrapper: Add ppcboot.h Scott Wood
2007-03-16 17:29 ` [PATCH 14/17] bootwrapper: Add support for cuboot platforms Scott Wood
2007-03-16 17:29 ` [PATCH 15/17] bootwrapper: Add cmd_wrap_dt Scott Wood
2007-03-16 17:29 ` [PATCH 16/17] bootwrapper: Add a cuImage target Scott Wood
2007-03-16 17:29 ` [PATCH 17/17] bootwrapper: cuboot for 83xx Scott Wood
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=29089968dc071f4b6917428a439f735e@kernel.crashing.org \
--to=segher@kernel.crashing.org \
--cc=linuxppc-dev@ozlabs.org \
--cc=scottwood@freescale.com \
--cc=stuart.yoder@freescale.com \
/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).