From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from yow.seanm.ca (toronto-hs-216-138-233-67.s-ip.magma.ca [216.138.233.67]) by ozlabs.org (Postfix) with SMTP id 445B0DDEDA for ; Tue, 29 Apr 2008 07:37:40 +1000 (EST) Date: Mon, 28 Apr 2008 17:37:38 -0400 From: Sean MacLennan To: "Grant Likely" Subject: Re: [RESEND][PATCH][POWERPC] PIKA Warp: Update platform code to supportRev B boards Message-ID: <20080428173738.6b1a2896@lappy.seanm.ca> In-Reply-To: References: <20080417152251.2bf07219@lappy.seanm.ca> <20080428145310.553b8271@lappy.seanm.ca> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII 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, 28 Apr 2008 13:56:11 -0600 "Grant Likely" wrote: > > You need to add the gpio-controller and #gpio-cells properties to the > GPIO nodes for the LED's gpios property to work correctly. Search for > "2) gpio-controller nodes" in > Documentation/powerpc/booting-without-of.txt for details. #gpio-cells > should probably be '2' for this gpio controller; 1 cell for the gpio > pin and 1 cell for flags. I believe these gpio nodes predate that text, but I added the fields anyway. > > These should not be children of the soc node (they are not part of the > SoC internal bus). However, I think it would be perfectly valid to > make them children of the gpio node since they don't have any > connections to other device on the platform. I put them in gpio. That was where I put them initialy. > Why is this information in the dts *and* the platform file? I haven't > been following the flash partition map binding conventions, but having > it in both places looks wrong.... > > oh, wait... the one in the dts is for NOR and this one is for NAND, > right? And we don't have a binding yet for NAND partitions yet, > correct? Correct. Josh originally asked me to split out the warp-nand.c file so that once the NAND is in the dts, we can just delete the file. NAND is much more complicated that NOR to configure. > When exporting symbols for platform code you should avoid polluting > the global Linux namespace and prefix the functions with your platform > name. I was hoping dtm was good enough. I prefixed them with the company name. We are expecting to have a "family" of Asterisk appliances and I am trying to make educated guesses as to what will be family wide (prefixed with pika) and what will be warp specific. Cheers, Sean