From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jamie Iles Subject: Re: [PATCHv2 1/2] gpio: add a driver for the Synopsys DesignWare APB GPIO block Date: Sat, 31 Dec 2011 01:03:46 +0000 Message-ID: <20111231010346.GA3222@page> References: <1324345431-12251-1-git-send-email-jamie@jamieiles.com> <20111230195955.GA3427@page> <4EFE1E35.106@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <4EFE1E35.106-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Sender: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org To: Rob Herring Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org, Linus Walleij , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: devicetree@vger.kernel.org On Fri, Dec 30, 2011 at 02:25:25PM -0600, Rob Herring wrote: > On 12/30/2011 01:59 PM, Jamie Iles wrote: > > Grant, Linus, > > > > On Tue, Dec 20, 2011 at 01:43:50AM +0000, Jamie Iles wrote: > >> The Synopsys DesignWare block is used in some ARM devices (picoxcell) > >> and can be configured to provide multiple banks of GPIO pins. The first > >> bank (A) can also provide IRQ capabilities. > >> > >> v2: - use Rob Herring's irqdomain in generic irq chip patches > >> - use reg property to indicate bank index > >> - support irqs on both edges based on LinusW's u300 driver > >> > >> Cc: Grant Likely > >> Cc: Linus Walleij > >> Cc: Rob Herring > >> Signed-off-by: Jamie Iles > > > > Is this driver okay by you guys? I'd like to get this one in for 3.3 if > > at all possible as it's the last patch that allows picoxcell to boot to > > a semi useful state without out of tree patches and that would make > > testing a lot easier! > > We've got to sort out getting irq domains enabled on x86 and MIPS in > order to merge generic irq chip using domains. I'm not sure that will > get done in time. > > Can you leave out the interrupt portion of the driver? Or that is needed > to get to a useful state. Sure, the GPIO is only needed for the NAND interface so the IRQ portion can easily be dropped. I'll respin the patch just doing basic GPIO but keep the binding details of the interrupt stuff as I can't see that it should change. Thanks, Jamie