From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Gleixner Subject: Re: [PATCH v5 06/27] irq_domain/powerpc: eliminate irq_map; use irq_alloc_desc() instead Date: Tue, 3 Apr 2012 10:23:56 +0200 (CEST) Message-ID: References: <1329383368-12122-1-git-send-email-grant.likely@secretlab.ca> <1329383368-12122-7-git-send-email-grant.likely@secretlab.ca> <20120402202848.1EB833E02A7@localhost> <20120402215524.GM24211@n2100.arm.linux.org.uk> <1333406005.30734.61.camel@pasglop> <20120402225223.GN24211@n2100.arm.linux.org.uk> Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Return-path: In-Reply-To: <20120402225223.GN24211@n2100.arm.linux.org.uk> Sender: linux-kernel-owner@vger.kernel.org To: Russell King - ARM Linux Cc: Benjamin Herrenschmidt , Grant Likely , Andreas Schwab , devicetree-discuss@lists.ozlabs.org, linux-kernel@vger.kernel.org, Milton Miller , Rob Herring , linuxppc-dev@lists.ozlabs.org, linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org On Mon, 2 Apr 2012, Russell King - ARM Linux wrote: > If we want to fix it a better way, then sure, that'll be good. But what > we shouldn't do is re-introduce one regression to fix a different > regression. > > So, Thomas, what do you think about providing a way that a disabled > interrupt could have its pending status cleared such that when it's > enabled, any queued events are ignored? Maybe an enable_and_clear_irq() ? We can make it a flag, which you can either add to the irq itself or hand it in on request_irq(). Does that require a special chip->callback() function as well ? Thanks, tglx