From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate.crashing.org (gate.crashing.org [63.228.1.57]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTP id 51199DDE35 for ; Thu, 15 Feb 2007 08:17:16 +1100 (EST) Subject: Re: [PATCH] powerpc: delete boot-cpu and chosen nodes from all DTS files From: Benjamin Herrenschmidt To: Timur Tabi In-Reply-To: <45D378F4.3040003@freescale.com> References: <1171409687840-git-send-email-timur@freescale.com> <1171411075.20192.116.camel@localhost.localdomain> <45D29AE5.7090004@freescale.com> <1171430877.20192.133.camel@localhost.localdomain> <45D29EB1.5060801@freescale.com> <1171431334.20192.138.camel@localhost.localdomain> <45D316CD.9070901@freescale.com> <1171486473.20192.147.camel@localhost.localdomain> <45D378F4.3040003@freescale.com> Content-Type: text/plain Date: Thu, 15 Feb 2007 08:17:02 +1100 Message-Id: <1171487822.20192.163.camel@localhost.localdomain> Mime-Version: 1.0 Cc: linuxppc-dev@ozlabs.org, paulus@samba.org, Stuart Yoder List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Wed, 2007-02-14 at 15:02 -0600, Timur Tabi wrote: > Benjamin Herrenschmidt wrote: > > >> Obviously, the kernel needs a 'chosen' section, and that's why U-Boot > >> adds one. And since U-Boot adds one, the DTS itself doesn't need one. > >> Does the 8272 never use U-Boot? > > > > I'm not talking specifically about the 8272. I think we should at it > > back to all dts's > > Ugh. Considering that U-Boot overrides whatever is in the 'chosen' section, can > you tell me why you think it should be there? If we put an empty 'chosen' > section in the DTS files, are we actually doing anything meaningful? > > In the interest of saving time, I'm going to respin my patch without the > 'chosen' section removed. In case somebody wants to use something else than uboot ? I'd like the base structure of the mandatory nodes to be there. /chosen is mandatory, so it should be there. Ben.