From: Segher Boessenkool <segher@kernel.crashing.org>
To: "Yoder Stuart-B08248" <stuart.yoder@freescale.com>
Cc: linuxppc-dev@ozlabs.org
Subject: Re: purpose of /chosen node (was RE: [PATCH] powerpc: delete boot-cpu and chosen nodes from all DTSfiles)
Date: Fri, 16 Feb 2007 08:38:08 +0100 [thread overview]
Message-ID: <87521bc5e5a48f417e1f92b5342a5b60@kernel.crashing.org> (raw)
In-Reply-To: <9696D7A991D0824DBA8DFAC74A9C5FA302A1BC91@az33exm25.fsl.freescale.net>
> I understand that point, but mixing the types of properties in
> /chosen doesn't buy us anything. We shouldn't be shackled by
> the 1275 spec,
Sure, you have some special needs, and some ways in which
the flat tree just _is_ different from an OF device tree;
for example, you don't have instances and ihandles.
Still it doesn't help anyone (and will in fact hurt
everyone) if people start to introduce random incompatibilities
for no good reason at all.
> but having nodes like /chosen with designated
> purpose architecturally make sense.
Yes. And since this node already has a well-defined
purpose in OF, you better use it for that purpose only
and make a /flattree/grabbag node for your random stuff.
> /chosen shouldn't be a catch-all or a convenient place to stick
> stuff. I think in this case following 1275 is a good thing.
Segher
next prev parent reply other threads:[~2007-02-16 7:38 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-02-13 16:47 [PATCH] powerpc: delete boot-cpu and chosen nodes from all DTS files Timur Tabi
2007-02-13 17:16 ` Kumar Gala
2007-02-13 17:20 ` Timur Tabi
2007-02-13 19:29 ` Timur Tabi
2007-02-15 1:42 ` Segher Boessenkool
2007-02-14 0:31 ` David Gibson
2007-02-14 5:18 ` Timur Tabi
2007-02-14 5:24 ` David Gibson
2007-02-14 14:05 ` Timur Tabi
2007-02-14 23:33 ` David Gibson
2007-02-14 23:47 ` Timur Tabi
2007-02-15 1:40 ` Segher Boessenkool
2007-02-15 21:26 ` purpose of /chosen node (was RE: [PATCH] powerpc: delete boot-cpu and chosen nodes from all DTSfiles) Yoder Stuart-B08248
2007-02-15 21:39 ` Kumar Gala
2007-02-15 21:45 ` Yoder Stuart-B08248
2007-02-16 7:38 ` Segher Boessenkool [this message]
2007-02-16 7:34 ` Segher Boessenkool
2007-02-15 23:01 ` David Gibson
2007-02-16 0:08 ` Benjamin Herrenschmidt
2007-02-16 16:58 ` purpose of /chosen node (was RE: [PATCH] powerpc: deleteboot-cpu " Yoder Stuart-B08248
2007-02-16 17:00 ` Kumar Gala
2007-02-16 7:30 ` purpose of /chosen node (was RE: [PATCH] powerpc: delete boot-cpu " Segher Boessenkool
2007-02-16 9:54 ` Benjamin Herrenschmidt
2007-02-15 16:18 ` [PATCH] powerpc: delete boot-cpu and chosen nodes from all DTS files Timur Tabi
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=87521bc5e5a48f417e1f92b5342a5b60@kernel.crashing.org \
--to=segher@kernel.crashing.org \
--cc=linuxppc-dev@ozlabs.org \
--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 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.