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

[David Gibson <david@gibson.dropbear.id.au>]

On Thu, Feb 15, 2007 at 02:26:41PM -0700, 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.

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