From mboxrd@z Thu Jan 1 00:00:00 1970 From: Samuel Ortiz Subject: Re: [PATCH] ARM: OMAP: TWL4030 IRQ Date: Tue, 4 Aug 2009 15:30:44 +0200 Message-ID: <20090804133041.GC8601@sortiz.org> References: <1248674448-17495-1-git-send-email-santosh.shilimkar@ti.com> <20090803163611.GB28606@sortiz.org> <20090803172321.GB32479@flint.arm.linux.org.uk> <20090804105100.GK24334@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mga09.intel.com ([134.134.136.24]:18359 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755546AbZHDN2d (ORCPT ); Tue, 4 Aug 2009 09:28:33 -0400 Content-Disposition: inline In-Reply-To: <20090804105100.GK24334@atomide.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Tony Lindgren Cc: Russell King , Santosh Shilimkar , linux-omap@vger.kernel.org, linux-arm-kernel@lists.arm.linux.org.uk Hi Tony, On Tue, Aug 04, 2009 at 01:51:00PM +0300, Tony Lindgren wrote: > * Russell King [090803 20:26]: > > On Mon, Aug 03, 2009 at 06:36:12PM +0200, Samuel Ortiz wrote: > > > Hi Santosh, > > > > > > On Mon, Jul 27, 2009 at 11:30:48AM +0530, Santosh Shilimkar wrote: > > > > From: Russell King > > > > > > > > (Rebased on 2.6.31-rc4) > > > > > > > > The TWL4030 IRQ handler has a bug which leads to spinlock lock-up. It is > > > > calling the 'unmask' function in a process context. :The mask/unmask/ack > > > > functions are only designed to be called from the IRQ handler code, > > > > or the proper API interfaces found in linux/interrupt.h. > > > > > > > > Also there is no need to have IRQ chaining mechanism. The right way to > > > > handle this is to claim the parent interrupt as a standard interrupt > > > > and arrange for handle_twl4030_pih to take care of the rest of the devices. > > > I'd like this one to be split in 2 different patches as you're addressing 2 > > > different issues here. > > > > You'd like me to remove the IRQ handling entirely from this code as one > > patch, thereby breaking it, and then add the new IRQ handling as a > > separate patch? > > > > Are you sure? > > > > I really don't think so, and I suspect you haven't even read the patch. > > > > It's all _one_ issue, with two explainations of why the current code is > > wrong. > > > > So my reply is: unable to split patch. > > Yeah I guess the description "Also there is no need.." above could be > just "This is fixed by not using IRQ chaining mechanism" if anything. Yep, I was mislead by the description, but I also didnt look at the patch carefully enough. > Anyways would be nice to get this in as a fix. I applied it to my for-next branch for now, and I'll also try to have Linus pulling it. Cheers, Samuel. > Acked-by: Tony Lindgren -- Intel Open Source Technology Centre http://oss.intel.com/