From mboxrd@z Thu Jan 1 00:00:00 1970 From: Charles Keepax Subject: Re: [PATCH v2] mfd: arizona: Add DT binding for irq-gpios property Date: Tue, 3 Nov 2015 16:52:47 +0000 Message-ID: <20151103165247.GC2549@ck-lbox> References: <1446559806-3315-1-git-send-email-sjan@baylibre.com> <20151103150450.GP20228@sirena.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20151103150450.GP20228@sirena.org.uk> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Mark Brown Cc: devicetree@vger.kernel.org, alsa-devel@alsa-project.org, Samuel Ortiz , Sebastien Jan , patches@opensource.wolfsonmicro.com, Lee Jones List-Id: devicetree@vger.kernel.org On Tue, Nov 03, 2015 at 03:04:50PM +0000, Mark Brown wrote: > On Tue, Nov 03, 2015 at 03:10:06PM +0100, Sebastien Jan wrote: > > > + - irq-gpios : GPIO specifier for the GPIO connected to /IRQ > > The platform data this is providing is essentially a workaround for the > lack of an irq_to_gpio() function and the use of it is a workaround for > the lack of level triggered IRQ emulation in genirq... do we really > want this in the ABI (especially as a generic thing)? Personally I would rather not add this to the DT unless we really had to. Also are you sure this is necessary on your system? Specifically you only need this entry if the CODEC is connected to an IRQ line that only supports edge triggered IRQs. Thanks, Charles