From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Thu, 29 Mar 2007 11:56:56 +1000 From: David Gibson To: Segher Boessenkool Subject: Re: [RFC 3/3] powerpc: Add DTS file for the Motorola PrPMC2800 platform Message-ID: <20070329015656.GA16270@localhost.localdomain> References: <20070328011924.GA1586@mag.az.mvista.com> <20070328012206.GD1586@mag.az.mvista.com> <9696D7A991D0824DBA8DFAC74A9C5FA302BDAB3E@az33exm25.fsl.freescale.net> <20070329003550.GB25652@localhost.localdomain> <2cd6c2dd69700dd162e90f8b7a83a99d@kernel.crashing.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <2cd6c2dd69700dd162e90f8b7a83a99d@kernel.crashing.org> Cc: linuxppc-dev , Dale Farnsworth , Yoder Stuart-B08248 List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Thu, Mar 29, 2007 at 02:48:47AM +0200, Segher Boessenkool wrote: > > That does seem like a lot. The only thing that should need > > virtual-reg properties is serial ports. In fact, specifically, only > > serial ports that the zImage is actually expected to use. > > > > In fact, with Scott's addition of xlate_reg() to the bootwrapper, > > virtual-reg should only be needed on systems that enter the zImage > > with MMU on. > > Even then, you can just query the MMU structures to > find out the currently set up translations for that > physical address. Way more robust. > > The only valid reason to use "virtual-reg" is as a > hack to map some device to get some debug output out. That's exactly what it's for. Getting serial output as early as possible. It shouldn't be used for anything else. > If for normal usage you can't be bothered to parse > the current translations, you probably shouldn't be > doing anything as "advanced" as device I/O either -- > just boot the kernel and let it handle it ;-) > > That said, I sure hope the kernel isn't using this > property as well... -- 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