linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Segher Boessenkool <segher@kernel.crashing.org>
To: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: linuxppc-dev@ozlabs.org, Olaf Hering <olaf@aepfle.de>, paulus@samba.org
Subject: Re: [patch 29/30] PS3: Device tree source.
Date: Thu, 14 Jun 2007 12:25:21 +0200	[thread overview]
Message-ID: <b7282fe60cb00a9ba8756e152d277fc3@kernel.crashing.org> (raw)
In-Reply-To: <1181815870.14818.401.camel@localhost.localdomain>

>> Please review the OF spec, sections 3.4 and the "/" entry
>> of 3.5.
>>
>> I'f the flat device tree doc and/or code says differently
>> here, that is a bug.
>
> Well, I wonder if that's been superceeded by some RFC or never 
> validated
> addition ...

It has been that way since before the OF standard
existed, and never has been changed.

> None of the Apple trees for example has a /name property

Apple's trees have many many problems, you know that
as well as anybody.

> and the idea of having a node with a name that doesn't participate in
> the path resolution in just wrong.

Nonsense, a pathname starting with "/" is handled
specially.  This is true on Unix systems as well.

> We should check but I don't think IBM
> trees have one either.
>
> So I stand by the way I defined the flat DT -> no name in the root. But
> you are welcome to use compatible

You can use "compatible" as an alternate name, sure.
Not allowing a "name" in the root node is simply an
oversight in the flat device tree, and should be fixed
in my opinion.  There is no backwwards-compatibility
problem with that.

> and model.

> "model" is a completely separate thing, so yeah.


Segher

  reply	other threads:[~2007-06-14 10:25 UTC|newest]

Thread overview: 69+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20070612181825.730300780@am.sony.com>
2007-06-12 18:42 ` [patch 01/30] Cell: Add spu shutdown method Geoff Levand
2007-06-12 18:43 ` [patch 02/30] PS3: Rename IPI symbols Geoff Levand
2007-06-12 18:43 ` [patch 03/30] PS3: Use __maybe_unused Geoff Levand
2007-06-12 18:43 ` [patch 04/30] PS3: Compare firmware version Geoff Levand
2007-06-12 18:43 ` [patch 05/30] PS3: Use ioremap_flags Geoff Levand
2007-06-13  9:10   ` Arnd Bergmann
2007-06-14  1:42     ` Geoff Levand
2007-06-14  2:31   ` Takao Shinohara
2007-06-14 19:03   ` [patch 05/30 v2] " Geoff Levand
2007-06-14 23:43     ` Arnd Bergmann
2007-06-12 18:43 ` [patch 06/30] PS3: Fix sparse warnings Geoff Levand
2007-06-12 18:44 ` [patch 07/30] PS3: Add support for HDMI RGB Full Range mode Geoff Levand
2007-06-12 18:44 ` [patch 08/30] PS3: Move chip mask defs up Geoff Levand
2007-06-12 18:44 ` [patch 09/30] PS3: Use inline DBG Geoff Levand
2007-06-12 18:44 ` [patch 10/30] PS3: Kexec support Geoff Levand
2007-06-12 18:52 ` [patch 11/30] PS3: System-bus rework Geoff Levand
2007-06-12 18:52 ` [patch 12/30] PS3: System-bus uevent Geoff Levand
2007-06-12 18:52 ` [patch 13/30] PS3: System-bus modinfo attribute Geoff Levand
2007-06-12 18:53 ` [patch 14/30] PS3: Repository probe cleanups Geoff Levand
2007-06-12 18:53 ` [patch 15/30] PS3: Vuart rework Geoff Levand
2007-06-12 18:53 ` [patch 16/30] PS3: System manager re-work Geoff Levand
2007-06-12 18:53 ` [patch 17/30] PS3: Rework AV settings driver Geoff Levand
2007-06-12 18:53 ` [patch 18/30] PS3: Frame buffer system-bus rework Geoff Levand
2007-06-12 18:53 ` [patch 19/30] PS3: Device registration routines Geoff Levand
2007-06-12 18:53 ` [patch 20/30] PS3: Rename processor id symbols Geoff Levand
2007-06-12 18:54 ` [patch 21/30] PS3: Use clear_bit Geoff Levand
2007-06-12 18:54 ` [patch 22/30] Powerpc: Output params value in early_init_devtree Geoff Levand
2007-06-12 18:54 ` [patch 23/30] powerpc: Localize mmu_off Geoff Levand
2007-06-12 18:54 ` [patch 24/30] powerpc: Correct __secondary_hold comment Geoff Levand
2007-06-12 18:54 ` [patch 25/30] Powerpc: Add signed types to bootwrapper Geoff Levand
2007-06-12 18:54 ` [patch 26/30] Powerpc: Add u64 printf " Geoff Levand
2007-06-12 18:54 ` [patch 27/30] Powerpc: Fix constantness of bootwrapper arg Geoff Levand
2007-06-12 18:54 ` [patch 28/30] powerpc: Bootwrapper global scope kernel_entry_t Geoff Levand
2007-06-12 18:54 ` [patch 29/30] PS3: Device tree source Geoff Levand
2007-06-13  4:52   ` Segher Boessenkool
2007-06-13  8:40     ` Olaf Hering
2007-06-13  9:05       ` Segher Boessenkool
2007-06-13 11:24         ` Olaf Hering
2007-06-13 11:59           ` Segher Boessenkool
2007-06-13 12:28             ` Olaf Hering
2007-06-13 13:51               ` Segher Boessenkool
2007-06-13 16:04                 ` Olaf Hering
2007-06-13 13:43             ` Geert Uytterhoeven
2007-06-13 13:52               ` Segher Boessenkool
2007-06-13 14:09                 ` Geert Uytterhoeven
2007-06-13 14:18                   ` Segher Boessenkool
2007-06-13 14:20                     ` Geert Uytterhoeven
2007-06-13 17:36             ` Geoff Levand
2007-06-13 22:42         ` Benjamin Herrenschmidt
2007-06-14  7:31           ` Segher Boessenkool
2007-06-14 10:11             ` Benjamin Herrenschmidt
2007-06-14 10:25               ` Segher Boessenkool [this message]
2007-06-13 17:35       ` Geoff Levand
2007-06-13 18:18         ` Olaf Hering
2007-06-13 22:46           ` Benjamin Herrenschmidt
2007-06-14  7:32             ` Segher Boessenkool
2007-06-14 14:39               ` Yoder Stuart-B08248
2007-06-14 15:36                 ` Segher Boessenkool
2007-06-14 15:53                   ` Yoder Stuart-B08248
2007-06-15  8:32                     ` Segher Boessenkool
2007-06-15  0:16                   ` Benjamin Herrenschmidt
2007-06-15  8:37                     ` Segher Boessenkool
2007-06-13 22:44         ` Benjamin Herrenschmidt
2007-06-14  1:31     ` Geoff Levand
2007-06-14  7:39       ` Segher Boessenkool
2007-06-14  1:31   ` [patch 29/30 v2] " Geoff Levand
2007-06-14  7:55     ` Geert Uytterhoeven
2007-06-14 17:05       ` Geoff Levand
2007-06-12 18:55 ` [patch 30/30] PS3: Bootwrapper support Geoff Levand

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=b7282fe60cb00a9ba8756e152d277fc3@kernel.crashing.org \
    --to=segher@kernel.crashing.org \
    --cc=benh@kernel.crashing.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=olaf@aepfle.de \
    --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).