Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758259Ab3JKPyW (ORCPT ); Fri, 11 Oct 2013 11:54:22 -0400 Received: from mail-ie0-f182.google.com ([209.85.223.182]:58167 "EHLO mail-ie0-f182.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757207Ab3JKPyT (ORCPT ); Fri, 11 Oct 2013 11:54:19 -0400 MIME-Version: 1.0 In-Reply-To: References: <1380902596-4693-1-git-send-email-sujithkv@ti.com> <1380902596-4693-2-git-send-email-sujithkv@ti.com> Date: Fri, 11 Oct 2013 17:54:18 +0200 Message-ID: Subject: Re: [PATCH v3 1/3] gpio: davinci: add OF support From: Linus Walleij To: Prabhakar Lad Cc: "devicetree@vger.kernel.org" , DLOS , "linux-gpio@vger.kernel.org" , LDOC , LKML , LAK , Sekhar Nori , Rob Herring , Pawel Moll , Mark Rutland , Stephen Warren , Ian Campbell , Rob Landley 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: 1947 Lines: 53 On Fri, Oct 11, 2013 at 4:59 PM, Prabhakar Lad wrote: > On 10/11/13, Linus Walleij wrote: >> On Fri, Oct 4, 2013 at 6:03 PM, Prabhakar Lad >> wrote: >>> +- ti,davinci-gpio-irq-base: Base from where GPIO interrupt numbering >>> starts. >> >> What is this? >> >> If I have ever ACKed this I have been drunk. I take it back. >> > here is the ACK https://patchwork.kernel.org/patch/2721181/ And as suspected that version of the patch did not contain this strange node property. Don't keep my ACK on patches if you change basic stuff like that, they need to be re-acked, this runs the risk of abusing my trust amongst other subsystem maintainers who might go and merge this because "aha the GPIO maintainer thinks that this is OK". >> This "base" is a Linux-specific thing and has no place in the >> device tree, and shall not be there. You have to find some way to >> avoid this, what do you think some other OS should do with >> this value... >> >> All IRQs in Linux are assumed to be dynamically assigned numbers >> nowadays, with a property like this you can never switch on >> SPARSE_IRQ for the DaVinci. >> > Can you point to any alternative solution if you have any ? First convert this GPIO driver to use an irqdomain to map HW IRQs to Linux IRQs, and grab a few IRQ descriptors dynamically off the irq descriptor heap. Example: commit a6c45b99a658521291cfb66ecf035cc58b38f206 "pinctrl/coh901: use irqdomain, allocate irqdescs" Then on a longer term convert DaVinci to use dynamically allocated IRQs for all interrupt controllers, and move it over to SPARSE_IRQ so you know this works. 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/