Re: [PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-18 Thread Segher Boessenkool
You don't need to mention _all_ compatible devices in the "compatible" property, only the few that matter; typically the oldest one, and sometimes some intermediate device that has extra features over the original one. The oldest one is difficult to find out sometime. Can we only set the self

Re: [PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-18 Thread Segher Boessenkool
You don't need to mention _all_ compatible devices in the compatible property, only the few that matter; typically the oldest one, and sometimes some intermediate device that has extra features over the original one. The oldest one is difficult to find out sometime. Can we only set the self

RE: [PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-17 Thread Zhang Wei-r63237
Hi, Kumar and Segher, > > > "..8641.." "..8641d.." "..8548.." "..8548e.." "..8543.." > "..8543e.." > > "..8572.." "..8572e.." "..8567.." "..8567e.." "..8568.." "..8568e.." > > You don't need to mention _all_ compatible devices in > the "compatible" property, only the few that matter; >

RE: [PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-17 Thread Zhang Wei-r63237
Hi, Kumar and Segher, ..8641.. ..8641d.. ..8548.. ..8548e.. ..8543.. ..8543e.. ..8572.. ..8572e.. ..8567.. ..8567e.. ..8568.. ..8568e.. You don't need to mention _all_ compatible devices in the compatible property, only the few that matter; typically the oldest one, and sometimes

Re: [PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-14 Thread Segher Boessenkool
Not at all. On an 8641 it could be compatible = "fsl,mpc8641-rapidio" "fsl,mpc8548-rapidio"; which states "this is the 8641 thing and it is compatible to the 8548 thing". Perfectly clear. The concern is this isn't just compatible = "..8641.." "..8548.." but something like:

Re: [PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-14 Thread Segher Boessenkool
Not at all. On an 8641 it could be compatible = fsl,mpc8641-rapidio fsl,mpc8548-rapidio; which states this is the 8641 thing and it is compatible to the 8548 thing. Perfectly clear. The concern is this isn't just compatible = ..8641.. ..8548.. but something like: ..8641..

Re: [PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-13 Thread Kumar Gala
Some silicons of Freescale processor are the same RapidIO controller, such as mpc8540/mpc8560 are the same (v0.0), mpc8548/mpc8641 are the same (v1.0). For v1.0 RapidIO controller, should we use mpc8548 or mpc8641? Those will make people confused. Not at all. On an 8641 it could be

Re: [PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-13 Thread Segher Boessenkool
+- device_type : Should be "rapidio" There is no OF binding, so no. So, we need to define it. If you want to. Until that has been done, don't use a "device_type". Linux won't use it, anyway. Do you have another ideas about that? Only remove it? Yeah, remove it. Using IP Block

RE: [PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-13 Thread Zhang Wei-r63237
Hi, Segher, > > >>> +- device_type : Should be "rapidio" > >> > >> There is no OF binding, so no. > > > > So, we need to define it. > > If you want to. Until that has been done, don't use > a "device_type". Linux won't use it, anyway. Do you have another ideas about that? Only remove

Re: [PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-13 Thread Segher Boessenkool
+- device_type : Should be "rapidio" There is no OF binding, so no. So, we need to define it. If you want to. Until that has been done, don't use a "device_type". Linux won't use it, anyway. +- compatible : Should be "fsl,rapidio-v0.0" or "fsl,rapidio-v1.0" + and so on.

RE: [PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-13 Thread Zhang Wei-r63237
Hi, Segher, > -Original Message- > Subject: Re: [PATCH 1/5] Add the explanation and sample of > RapidIO DTS sector to the document of booting-without-of.txt file. > > > + k) RapidIO > > + > > + Required properties: > > + > > +- device_type : Should be "rapidio" > > There is no

RE: [PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-13 Thread Zhang Wei-r63237
Hi, Segher, -Original Message- Subject: Re: [PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file. + k) RapidIO + + Required properties: + +- device_type : Should be rapidio There is no OF binding, so

Re: [PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-13 Thread Segher Boessenkool
+- device_type : Should be rapidio There is no OF binding, so no. So, we need to define it. If you want to. Until that has been done, don't use a device_type. Linux won't use it, anyway. +- compatible : Should be fsl,rapidio-v0.0 or fsl,rapidio-v1.0 + and so on. The

RE: [PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-13 Thread Zhang Wei-r63237
Hi, Segher, +- device_type : Should be rapidio There is no OF binding, so no. So, we need to define it. If you want to. Until that has been done, don't use a device_type. Linux won't use it, anyway. Do you have another ideas about that? Only remove it? +-

Re: [PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-13 Thread Segher Boessenkool
+- device_type : Should be rapidio There is no OF binding, so no. So, we need to define it. If you want to. Until that has been done, don't use a device_type. Linux won't use it, anyway. Do you have another ideas about that? Only remove it? Yeah, remove it. Using IP Block

Re: [PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-13 Thread Kumar Gala
Some silicons of Freescale processor are the same RapidIO controller, such as mpc8540/mpc8560 are the same (v0.0), mpc8548/mpc8641 are the same (v1.0). For v1.0 RapidIO controller, should we use mpc8548 or mpc8641? Those will make people confused. Not at all. On an 8641 it could be

Re: [PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-12 Thread Segher Boessenkool
+ k) RapidIO + + Required properties: + +- device_type : Should be "rapidio" There is no OF binding, so no. +- compatible : Should be "fsl,rapidio-v0.0" or "fsl,rapidio-v1.0" + and so on. The version number is got from IP Block Revision + Register of RapidIO controller.

[PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-12 Thread Zhang Wei
Add the explanation and a sample of RapidIO DTS sector to the document of booting-without-of.txt file. Signed-off-by: Zhang Wei <[EMAIL PROTECTED]> --- Documentation/powerpc/booting-without-of.txt | 41 ++ 1 files changed, 41 insertions(+), 0 deletions(-) diff --git

[PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-12 Thread Zhang Wei
Add the explanation and a sample of RapidIO DTS sector to the document of booting-without-of.txt file. Signed-off-by: Zhang Wei [EMAIL PROTECTED] --- Documentation/powerpc/booting-without-of.txt | 41 ++ 1 files changed, 41 insertions(+), 0 deletions(-) diff --git

Re: [PATCH 1/5] Add the explanation and sample of RapidIO DTS sector to the document of booting-without-of.txt file.

2007-06-12 Thread Segher Boessenkool
+ k) RapidIO + + Required properties: + +- device_type : Should be rapidio There is no OF binding, so no. +- compatible : Should be fsl,rapidio-v0.0 or fsl,rapidio-v1.0 + and so on. The version number is got from IP Block Revision + Register of RapidIO controller. It's