From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Tue, 7 Aug 2007 13:35:38 +1000 From: David Gibson To: Segher Boessenkool Subject: Re: [PATCH 2/6] PowerPC 440EPx: Sequoia DTS Message-ID: <20070807033538.GB13522@localhost.localdomain> References: <20070730150648.GA5005@ru.mvista.com> <20070801020836.GB31391@localhost.localdomain> <65ff446478a9fd0a48061079d5f04f8f@kernel.crashing.org> <20070801050422.GI31391@localhost.localdomain> <20070801054751.GM31391@localhost.localdomain> <46B1F6D4.3070707@ru.mvista.com> <20070803031349.GD6418@localhost.localdomain> <3831d2304a027a1790e6d3cd666c2494@kernel.crashing.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <3831d2304a027a1790e6d3cd666c2494@kernel.crashing.org> Cc: linuxppc-dev@ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Mon, Aug 06, 2007 at 10:20:01PM +0200, Segher Boessenkool wrote: > >>> + - compatible : should contain the specific model of flash > >>> chip(s) used > >>> + followed by either "cfi-flash" or "jedec-flash" > >> > >> This "compatible" prop (and the node in whole) doesn't say a > >> thing about how the flash is mapped into the CPU address space. I > >> strongly disagree that this node provides enough info to select a > >> driver. :-/ > > > > To be honest, I'm not sure that describing the mapping is really the > > job of the compatible property. That the flash is mapped into the > > address space is kind of implicit in the way the reg interacts with > > the parents' ranges property. > > Exactly. > > > Can you describe some of the options for *not* direct mapped flash > > chips - I can't reasonably come up with a way of describing the > > distinction when I've never seen NOR flash other than direct mapped. > > What, you've never seen SPI flash or IIC flash? :-) Nope. Colour me ignorant :-p. -- 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