Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753154Ab3HUXAR (ORCPT ); Wed, 21 Aug 2013 19:00:17 -0400 Received: from mail-ob0-f170.google.com ([209.85.214.170]:55023 "EHLO mail-ob0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753118Ab3HUXAO (ORCPT ); Wed, 21 Aug 2013 19:00:14 -0400 MIME-Version: 1.0 In-Reply-To: <201308211521.21129.poeschel@lemonage.de> References: <1376387195-27469-1-git-send-email-larsi@wh2.tu-dresden.de> <9241298.URG9CVQFmF@flatron> <5212737A.1030904@wwwdotorg.org> <201308211521.21129.poeschel@lemonage.de> Date: Thu, 22 Aug 2013 01:00:14 +0200 Message-ID: Subject: Re: [PATCH v2] RFC: interrupt consistency check for OF GPIO IRQs From: Linus Walleij To: Lars Poeschel Cc: Stephen Warren , Tomasz Figa , Lars Poeschel , Grant Likely , "linux-gpio@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , Javier Martinez Canillas , Enric Balletbo i Serra , Jean-Christophe PLAGNIOL-VILLARD , Santosh Shilimkar , Kevin Hilman , Balaji T K , Tony Lindgren , Jon Hunter , Mark Rutland , Ian Campbell , galak@codeaurora.org, Pawel Moll Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1750 Lines: 38 On Wed, Aug 21, 2013 at 3:21 PM, Lars Poeschel wrote: > On Monday 19 August 2013 at 21:35:22, Stephen Warren wrote: >> On 08/17/2013 03:59 AM, Tomasz Figa wrote: >> > Then two _example_ formats follow, preceded by following statement: >> > The following two variants are commonly used: >> > I already know a variant which uses three (Exynos combiner) and four >> > (S3C24xx interrupt controller) cells. They are not pin controllers, >> > but you can't stop anyone from adopting similar or even more complex >> > specifiers formats for their hardware, especially when it matches >> > more closely the interrupt/pin layout used in their hardware. >> >> Yes, the binding doc interrupts.txt mentioned above does not specify >> *the* one-/two-cell format, but *a* common/possible one- and two-cell >> format. There's no strict reason that all interrupt controllers have to >> use those exact formats. The only way to parse interrupt specifiers is >> to ask the driver for the the interrupt controller code to parse the >> property. > > I agree with you. I also understand the interrupts.txt binding doc as it > lists only a common/possible cell format, but it's purpose is not to > restrict it to this. > > I send an updated patch, that uses the drivers xlate function to parse the > interrupt property in a few minutes. Thanks that you take this on a spin, I've been busy... (I also agree with the other comments here, let's see if we can figure this out!) Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/