From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexander Holler Subject: Re: [PATCH] RFC: interrupt consistency check for OF GPIO IRQs Date: Wed, 11 Sep 2013 09:30:27 +0200 Message-ID: <52301C13.2030103@ahsoftware.de> References: <1375101368-17645-1-git-send-email-linus.walleij@linaro.org> <51F8CC35.1070704@collabora.co.uk> <522EC392.8070002@ti.com> <522F1BF0.8080608@collabora.co.uk> <522F3408.6050608@ti.com> <52301655.6080801@ahsoftware.de> <523018DB.7080004@ahsoftware.de> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <523018DB.7080004@ahsoftware.de> Sender: linux-kernel-owner@vger.kernel.org To: joelf@ti.com Cc: Javier Martinez Canillas , Linus Walleij , Grant Likely , Linux Kernel Mailing List , "linux-arm-kernel@lists.infradead.org" , Linux-OMAP , "devicetree@vger.kernel.org" , Enric Balletbo i Serra , Jean-Christophe PLAGNIOL-VILLARD , Santosh Shilimkar , Kevin Hilman , Balaji T K , Tony Lindgren , Jon Hunter List-Id: devicetree@vger.kernel.org And another small update. ;) Am 11.09.2013 09:16, schrieb Alexander Holler: >> To summarize what happens if a driver uses a gpio as irq: >> >> gpio_request() // This works only if the gpio was not requested before >> gpio_direction_input() >> gpio_to_irq() // This needs an irq-mapping >> request_threaded_irq() >> >> So I would suggest multiple steps to change that: >> >> 1. Create a mapping for every gpio found in DT (or all gpios if no DT is >> used). I think that is what Linus patch does (sorry, I haven't really >> followed this thread and didn't look in deep at the patch). >> >> 2. Implement gpio_request_for_irq() >> This would just be a small macro for gpio_request(); >> gpio_direction_input() >> I would add gpio_to_irq() to that macro, so that it returns the irq number or zero if something failed. Regards, Alexander Holler