From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from an-out-0708.google.com (an-out-0708.google.com [209.85.132.245]) by ozlabs.org (Postfix) with ESMTP id DFB16DDEDF for ; Sat, 10 Nov 2007 04:49:28 +1100 (EST) Received: by an-out-0708.google.com with SMTP id b2so93312ana for ; Fri, 09 Nov 2007 09:49:27 -0800 (PST) Message-ID: Date: Fri, 9 Nov 2007 10:49:27 -0700 From: "Grant Likely" Sender: glikely@secretlab.ca To: "Marian Balakowicz" , linuxppc-dev@ozlabs.org Subject: Re: [PATCH v3 12/13] [POWERPC] Promess Motion-PRO DTS In-Reply-To: <20071106224200.GE31367@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 References: <20071106200446.10913.29338.stgit@hekate.izotz.org> <20071106200634.10913.6248.stgit@hekate.izotz.org> <20071106224200.GE31367@localhost.localdomain> List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 11/6/07, David Gibson wrote: > On Tue, Nov 06, 2007 at 09:06:34PM +0100, Marian Balakowicz wrote: > [snip] > > + // PSC2 in spi master mode > > + spi@2200 { // PSC2 > > + compatible = "mpc5200b-psc-spi","mpc5200-psc-spi"; > > + cell-index = <1>; > > From your description, this is an incorrect usage of cell-index - it > should *only* be used to index into SoC shared registers; never for > logical numbering. Actually, this one is correct. PSCs are numbered in the data sheet from PSC1 to PSC6; so cell-index = <0> for PSC1, cell-index = <1> for PSC2, etc. Cheers, g. -- Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. grant.likely@secretlab.ca (403) 399-0195