[PATCH] powerpc: #address-cells & #size-cells properties not inherited

Subsystems: documentation, the rest

STALE6808d

6 messages, 3 authors, 2008-01-03 · open the first message on its own page

[PATCH] powerpc: #address-cells & #size-cells properties not inherited

From: Mark A. Greer <hidden>
Date: 2008-01-03 00:07:50

From: Mark A. Greer <redacted>

Fix error in booting-without-of.txt that indicates that a node can inherit
its #address-cells and #size-cells definitions from its parent's parent.
This is not correct and the latest dtc enforces it.

Signed-off-by: Mark A. Greer <redacted>
---
 Documentation/powerpc/booting-without-of.txt |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/Documentation/powerpc/booting-without-of.txt b/Documentation/powerpc/booting-without-of.txt
index ee0209a..58db5ea 100644
--- a/Documentation/powerpc/booting-without-of.txt
+++ b/Documentation/powerpc/booting-without-of.txt
@@ -671,10 +671,10 @@ device or bus to be described by the device tree.
 
 In general, the format of an address for a device is defined by the
 parent bus type, based on the #address-cells and #size-cells
-property. In the absence of such a property, the parent's parent
-values are used, etc... The kernel requires the root node to have
-those properties defining addresses format for devices directly mapped
-on the processor bus.
+properties.  Note that the parent's parent definitions of #address-cells
+and #size-cells are not inhereted so every node with children must specify
+them.  The kernel requires the root node to have those properties defining
+addresses format for devices directly mapped on the processor bus.
 
 Those 2 properties define 'cells' for representing an address and a
 size. A "cell" is a 32-bit number. For example, if both contain 2

Re: [PATCH] powerpc: #address-cells & #size-cells properties not inherited

From: Josh Boyer <hidden>
Date: 2008-01-03 01:46:40

On Wed, 2 Jan 2008 17:07:50 -0700
"Mark A. Greer" [off-list ref] wrote:
From: Mark A. Greer <redacted>

Fix error in booting-without-of.txt that indicates that a node can inherit
its #address-cells and #size-cells definitions from its parent's parent.
This is not correct and the latest dtc enforces it.
I'm lazy so I haven't checked myself, but by "latest dtc" does that
mean the DTC that's in the kernel now?

Things are going to be fun when we start saying generic things like
"latest DTC".  And when upstream DTC gets new features or enforcements,
we'll have to have all the in-kernel DTS files patched up when that
version of DTC gets merged in-kernel.  Just something to keep in mind
as we move along.

josh

Re: [PATCH] powerpc: #address-cells & #size-cells properties not inherited

From: David Gibson <hidden>
Date: 2008-01-03 09:46:49

On Wed, Jan 02, 2008 at 05:07:50PM -0700, Mark A. Greer wrote:
From: Mark A. Greer <redacted>

Fix error in booting-without-of.txt that indicates that a node can inherit
its #address-cells and #size-cells definitions from its parent's parent.
This is not correct and the latest dtc enforces it.
For small values of "enforce"; it's a warning only.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

Re: [PATCH] powerpc: #address-cells & #size-cells properties not inherited

From: Mark A. Greer <hidden>
Date: 2008-01-03 15:32:56

On Wed, Jan 02, 2008 at 07:46:40PM -0600, Josh Boyer wrote:
On Wed, 2 Jan 2008 17:07:50 -0700
"Mark A. Greer" [off-list ref] wrote:
quoted
From: Mark A. Greer <redacted>

Fix error in booting-without-of.txt that indicates that a node can inherit
its #address-cells and #size-cells definitions from its parent's parent.
This is not correct and the latest dtc enforces it.
I'm lazy so I haven't checked myself, but by "latest dtc" does that
mean the DTC that's in the kernel now?

Things are going to be fun when we start saying generic things like
"latest DTC".  And when upstream DTC gets new features or enforcements,
we'll have to have all the in-kernel DTS files patched up when that
version of DTC gets merged in-kernel.  Just something to keep in mind
as we move along.
Yeah, I thought about that last night, actually.
I think I'll just change that sentence to "This is not correct."

Avoid the issue :)

Mark

[PATCH v2] powerpc: #address-cells & #size-cells properties not inherited

From: Mark A. Greer <hidden>
Date: 2008-01-03 15:40:47

From: Mark A. Greer <redacted>

Fix error in booting-without-of.txt that indicates that a node can inherit
its #address-cells and #size-cells definitions from its parent's parent.
This is not correct.

Signed-off-by: Mark A. Greer <redacted>
---
 Documentation/powerpc/booting-without-of.txt |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/Documentation/powerpc/booting-without-of.txt b/Documentation/powerpc/booting-without-of.txt
index ee0209a..58db5ea 100644
--- a/Documentation/powerpc/booting-without-of.txt
+++ b/Documentation/powerpc/booting-without-of.txt
@@ -671,10 +671,10 @@ device or bus to be described by the device tree.
 
 In general, the format of an address for a device is defined by the
 parent bus type, based on the #address-cells and #size-cells
-property. In the absence of such a property, the parent's parent
-values are used, etc... The kernel requires the root node to have
-those properties defining addresses format for devices directly mapped
-on the processor bus.
+properties.  Note that the parent's parent definitions of #address-cells
+and #size-cells are not inhereted so every node with children must specify
+them.  The kernel requires the root node to have those properties defining
+addresses format for devices directly mapped on the processor bus.
 
 Those 2 properties define 'cells' for representing an address and a
 size. A "cell" is a 32-bit number. For example, if both contain 2

Re: [PATCH] powerpc: #address-cells & #size-cells properties not inherited

From: Josh Boyer <hidden>
Date: 2008-01-03 16:01:31

On Thu, 3 Jan 2008 08:32:56 -0700
"Mark A. Greer" [off-list ref] wrote:
On Wed, Jan 02, 2008 at 07:46:40PM -0600, Josh Boyer wrote:
quoted
On Wed, 2 Jan 2008 17:07:50 -0700
"Mark A. Greer" [off-list ref] wrote:
quoted
From: Mark A. Greer <redacted>

Fix error in booting-without-of.txt that indicates that a node can inherit
its #address-cells and #size-cells definitions from its parent's parent.
This is not correct and the latest dtc enforces it.
I'm lazy so I haven't checked myself, but by "latest dtc" does that
mean the DTC that's in the kernel now?

Things are going to be fun when we start saying generic things like
"latest DTC".  And when upstream DTC gets new features or enforcements,
we'll have to have all the in-kernel DTS files patched up when that
version of DTC gets merged in-kernel.  Just something to keep in mind
as we move along.
Yeah, I thought about that last night, actually.
I think I'll just change that sentence to "This is not correct."

Avoid the issue :)
Lazy!  I like it ;)

josh
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help