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

On Thu, Feb 15, 2007 at 02:26:41PM -0700, Yoder Stuart-B08248 wrote:

quoted

-----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

quoted

My point is that the interrupt-controller property in 

/chosen *should*

quoted

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.

You're right.  Question is, can we change it now.  It's out there in
the documentation, on the other hand, I don't think the kernel
actually uses it anywhere yet.  BenH, opinion?

-- 
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