From patchwork Thu Apr 10 20:17:44 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Javier Martinez Canillas X-Patchwork-Id: 3964651 Return-Path: X-Original-To: patchwork-linux-omap@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 47E169F3D5 for ; Thu, 10 Apr 2014 20:17:52 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 62BFB20826 for ; Thu, 10 Apr 2014 20:17:51 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 41BE220824 for ; Thu, 10 Apr 2014 20:17:50 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965603AbaDJURt (ORCPT ); Thu, 10 Apr 2014 16:17:49 -0400 Received: from mail-wi0-f176.google.com ([209.85.212.176]:50892 "EHLO mail-wi0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1030339AbaDJURp (ORCPT ); Thu, 10 Apr 2014 16:17:45 -0400 Received: by mail-wi0-f176.google.com with SMTP id r20so11372354wiv.3 for ; Thu, 10 Apr 2014 13:17:44 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc:content-type; bh=cJ/dBokB81bsvQbjjYPzBdQnmRW9c2sN6kW7zhAeaTE=; b=TVIOsMkISjJtYpSOkRFXp0NpH5X7A5LrujWBhFT3J4QklG/ZZlkaOaXNgGoyvGPkRT waxh7Ect5YvwVgtMYeYkIrwni+sBmB3qGSGm6ibtBocd40Uoi64kWoEb1p0srubQe2ic SC7ExLhmJ98KeGXWo1WKkAVel2znZsLRV3mL1R10ZV/04XR3w/p4A1u5BzBUd8jJSaAq mS0NOjN88P06FhEKJvXqIz6ahzWKfmKd1PkLABtR7o+oR+HpuG+PmcxCBQ3Mz1DZ7VZO h7fE5sUSQ+iS4CLJl7FBuUubeMkB56BvDWyZro8Ca0ACC/GogMEeuUZaF7OJhmru7fWf gsjQ== X-Gm-Message-State: ALoCoQmpndGw89nHVTTyCOEgQtPIkyqFpY9CVMyavlgwBUVVBrJqK4GWJINRQRu7/SRNFYSoD1+p MIME-Version: 1.0 X-Received: by 10.194.191.133 with SMTP id gy5mr17122899wjc.34.1397161064703; Thu, 10 Apr 2014 13:17:44 -0700 (PDT) Received: by 10.180.87.36 with HTTP; Thu, 10 Apr 2014 13:17:44 -0700 (PDT) X-Originating-IP: [95.21.193.102] In-Reply-To: <20140410193048.GE15736@drone.musicnaut.iki.fi> References: <1396796297-13002-1-git-send-email-javier.martinez@collabora.co.uk> <20140410193048.GE15736@drone.musicnaut.iki.fi> Date: Thu, 10 Apr 2014 22:17:44 +0200 Message-ID: Subject: Re: [PATCH 0/5] GPIO OMAP driver changes for v3.16 From: Javier Martinez Canillas To: Aaro Koskinen Cc: Linus Walleij , Javier Martinez Canillas , Santosh Shilimkar , Kevin Hilman , Tony Lindgren , Paul Walmsley , Nishanth Menon , "linux-gpio@vger.kernel.org" , Linux-OMAP , "linux-arm-kernel@lists.infradead.org" Sender: linux-omap-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-omap@vger.kernel.org X-Spam-Status: No, score=-7.5 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_HI, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=ham version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Hello Aaro, Thanks a lot for testing the series! On Thu, Apr 10, 2014 at 9:30 PM, Aaro Koskinen wrote: > Hi, > > On Thu, Apr 10, 2014 at 07:29:26PM +0200, Linus Walleij wrote: >> On Sun, Apr 6, 2014 at 4:58 PM, Javier Martinez Canillas >> wrote: >> >> > Now that you have sent your changes for v3.15 to Torvalds, here are some >> > changes for the OMAP GPIO driver targeted to v3.16. Mostly improvements >> > so nothing here is -rc material. >> >> I like this series so I have applied them for v3.16, pending some ACK >> from Kevin &| Santosh. > > I tried these patches on OMAP1 on top of today's Torvalds git > (4ba85265790ba3681deeaf73f018c0eb829a7341). > > On Amstrad E3 I'm getting the following logs: > > [ 0.156491] omap_gpio omap_gpio.0: Runtime PM disabled, clock forced on. > [ 0.164604] genirq: Setting trigger mode 0 for irq 64 failed (gpio_irq_type+0x0/0x1f0) > [ 0.165418] genirq: Setting trigger mode 0 for irq 65 failed (gpio_irq_type+0x0/0x1f0) > [ 0.166133] genirq: Setting trigger mode 0 for irq 66 failed (gpio_irq_type+0x0/0x1f0) > [ 0.166838] genirq: Setting trigger mode 0 for irq 67 failed (gpio_irq_type+0x0/0x1f0) > [...] > [ 0.182856] genirq: Setting trigger mode 0 for irq 79 failed (gpio_irq_type+0x0/0x1f0) > [ 0.186887] omap_gpio omap_gpio.1: Could not get gpio dbck > [ 0.189308] genirq: Setting trigger mode 0 for irq 95 failed (gpio_irq_type+0x0/0x1f0) > [...] > [ 0.203121] genirq: Setting trigger mode 0 for irq 110 failed (gpio_irq_type+0x0/0x1f0) > > However it still seems to work. The serio is only GPIO IRQ and it > triggers when I press the external keyboard. > > The same happens also on Nokia 770: > > [ 0.118896] genirq: Setting trigger mode 0 for irq 128 failed (gpio_irq_type+0x0/0x220) > [ 0.119201] genirq: Setting trigger mode 0 for irq 129 failed (gpio_irq_type+0x0/0x220) > [...] > [ 0.124999] genirq: Setting trigger mode 0 for irq 143 failed (gpio_irq_type > +0x0/0x220) > [ 0.126831] omap_gpio omap_gpio.1: Could not get gpio dbck > [ 0.127258] OMAP GPIO hardware version 1.1 > [ 0.127624] omap_gpio omap_gpio.2: Could not get gpio dbck > [ 0.128204] omap_gpio omap_gpio.3: Could not get gpio dbck > [ 0.128753] omap_gpio omap_gpio.4: Could not get gpio dbck > > Here also GPIO IRQs (touchscreen, Retu) still work. > > A. I don't have those errors when booting on my DM3730 IGEPv2 board but it seems that for some reason on omap1 __irq_set_trigger() complains when IRQ_TYPE_NONE is used as a default flag when calling gpiochip_irqchip_add() Could you please test the following patch and tell me if your board still works and makes the errors go away? %d\n", ret); Best regards, Javier --- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/drivers/gpio/gpio-omap.c b/drivers/gpio/gpio-omap.c index 8cc9e91..5bc8aec 100644 --- a/drivers/gpio/gpio-omap.c +++ b/drivers/gpio/gpio-omap.c @@ -1122,7 +1122,7 @@ static int omap_gpio_chip_init(struct gpio_bank *bank) ret = gpiochip_irqchip_add(&bank->chip, &gpio_irq_chip, irq_base, gpio_irq_handler, - IRQ_TYPE_NONE); + IRQ_TYPE_LEVEL_LOW); if (ret) { dev_err(bank->dev, "Couldn't add irqchip to gpiochip