Re: purpose of /chosen node (was RE: [PATCH] powerpc: delete boot-cpu and chosen nodes from all DTSfiles)

From: Kumar Gala <galak@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: Thu, 15 Feb 2007 15:39:08 -0600	[thread overview]
Message-ID: <B8882F02-616E-4E71-BECC-82BDE5D88A4E@kernel.crashing.org> (raw)
In-Reply-To: <9696D7A991D0824DBA8DFAC74A9C5FA302A1BC76@az33exm25.fsl.freescale.net>

On Feb 15, 2007, at 3:26 PM, Yoder Stuart-B08248 wrote:

>
>
>> -----Original Message-----
>> From: linuxppc-dev-bounces+b08248=freescale.com@ozlabs.org
>> [mailto:linuxppc-dev-bounces+b08248=freescale.com@ozlabs.org]
>> On Behalf Of Segher Boessenkool
>> Sent: Wednesday, February 14, 2007 7:40 PM
>> To: David Gibson
>> Cc: linuxppc-dev@ozlabs.org; Tabi Timur-B04825
>> Subject: Re: [PATCH] powerpc: delete boot-cpu and chosen
>> nodes from all DTSfiles
>>
>>> My point is that the interrupt-controller property in
>> /chosen *should*
>>> be there, but it doesn't make a lot of sense for it to be set by the
>>> bootloader (because it's a fixed property of the board).
>>
>> And that means it shouldn't be in /chosen at all.  It's not
>> just a fixed property of the board, it is a physical property
>> of the board.  It doesn't belong in /chosen.
>
> I agree with Segher here.  The interrupt-controller property doesn't
> belong in /chosen.
>
> The 1275 OF spec says about /chosen:
>
>    Has properties describing parameters chosen or specified at
>    runtime.

I want to reiterate something Paul mentioned, any similarity that the  
device tree has to 1275 is strictly coincidental.

- k