From mboxrd@z Thu Jan 1 00:00:00 1970 From: Linus Walleij Subject: Re: [PATCH 2/2] gpio: davinci: Do not assume continuous IRQ numbering Date: Thu, 14 Jun 2018 13:18:36 +0200 Message-ID: References: <1528276734-1483-1-git-send-email-j-keerthy@ti.com> <1528276734-1483-2-git-send-email-j-keerthy@ti.com> <711dc0a5-9099-dbc9-6f8f-f37caf69105f@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: In-Reply-To: <711dc0a5-9099-dbc9-6f8f-f37caf69105f@ti.com> Sender: linux-kernel-owner@vger.kernel.org To: Keerthy Cc: "thierry.reding@gmail.com" , "linux-kernel@vger.kernel.org" , "open list:GPIO SUBSYSTEM" , Tero Kristo List-Id: linux-gpio@vger.kernel.org On Thu, Jun 14, 2018 at 10:51 AM, Keerthy wrote: >> I think the idea is to make it possible to use GPIOLIB_IRQCHIP >> for banked IRQs but the infrastructure is not yet inside the gpiolib >> so it is a bit taped on the side right now. > > Okay. I will take a look at that. The key issue that this patch > addresses is that currently driver only calls platform_get_irq once and > assumes the rest are continuous which is wrong hence the key issue > addressed with this patch is to call platform_get_irq for each interrupt. > > The version 4 is here: > > https://patchwork.kernel.org/patch/10461537/ > > I will try to follow up with generic gpiochip_irq_map/unmap once i get > that working. Hope that is okay? Oh that's fine, I'll merge it. Just wanted you to have a look at Thierry's stuff so we can share more code, you're a core developer so I wanted some smart people to look at this. Yours, Linus Walleij