[{"id":1770177,"web_url":"http://patchwork.ozlabs.org/comment/1770177/","msgid":"<20170918134610.17743-2-jbrunet@baylibre.com>","list_archive_url":null,"date":"2017-09-18T13:46:08","subject":"[PATCH v4 0/2] irqchip: meson: add support for the gpio interrupt\n\tcontroller","submitter":{"id":69839,"url":"http://patchwork.ozlabs.org/api/people/69839/","name":"Jerome Brunet","email":"jbrunet@baylibre.com"},"content":"This patch series adds support for the GPIO interrupt controller found on\nAmlogic's meson SoC families.\n\nUnlike what the name suggests, this controller is not part of the SoC\nGPIO subsystem. It is a separate controller which can watch almost all\ngpio pads of the SoC and generate and interrupt from it. \"Almost\" because\nthere are always exceptions and some specific gpios (TEST_N) lack this\ncapability.\n\nHardware wise, the controller is a 256 to 8 router with a filtering block\nto select edge or level input and the polarity of the signal. We can't\nsetup the filtring to generate a signal on both the high and low polarity\nso, ATM, IRQ_TYPE_EDGE_BOTH is not supported.\n\nThe number of interrupt line routed to the controller depends on the SoC,\nand essentially the total number of GPIO available on the different gpio\ncontrollers of the SoC.\n\nThis series has been tested on Amlogic S905-P200 board with the front\npanel power button. Also tested on the Nanopi-k2 with the ethernet PHY\ninterrupt pin.\n\nThis work is derived from the previous work of Carlo Caione [1].\n\nChanges since RFC : [2]\n* Remove interrupt property in device tree: the controller cannot\n generate interrupts on its own and is merely routing the interrupt to\n the GIC, therefore it should not use the interrupt property. This data\n is now stored directly in the driver, same as the pinctrl data.\n* Improve compatibility checking of meson pinctrl on its interrupt\n parent to activate gpio_to_irq callback\n* Drop IRQ_BOTH hack. Need more work to have an acceptable solution for\n this\n\nChanges since v1 : [3]\n* Correct mistake in patch 4 when no compatible\n controller is found. Sorry for the inconvenience.\n\nChanges since v2: [4]\n* Address Marc's comment on the irqchip driver from v2\n* Drop all gpio subsystem related patches. This will be dealt with\n separately.\n\nChanges since v3: [5]\n* Replace \"upstream\" irq with \"channel\" irq\n* Improve documentation a bit.\n* Drop DT patches: these will be sent later on to the DT maintainer\n if/when the driver is accepted.\n\n[1] : https://lkml.kernel.org/r/1448987062-31225-1-git-send-email-carlo@caione.org\n[2] : https://lkml.kernel.org/r/1475593708-10526-1-git-send-email-jbrunet@baylibre.com\n[3] : https://lkml.kernel.org/r/1476871709-8359-1-git-send-email-jbrunet@baylibre.com\n[4] : https://lkml.kernel.org/r/1476890480-8884-1-git-send-email-jbrunet@baylibre.com\n[5] : https://lkml.kernel.org/r/20170615161804.32658-1-jbrunet@baylibre.com\n\nJerome Brunet (2):\n dt-bindings: interrupt-controller: add DT binding for meson GPIO\n interrupt controller\n irqchip: meson: add support for gpio interrupt controller\n\n .../amlogic,meson-gpio-intc.txt | 35 ++\n drivers/irqchip/Kconfig | 8 +\n drivers/irqchip/Makefile | 1 +\n drivers/irqchip/irq-meson-gpio.c | 414 +++++++++++++++++++++\n 4 files changed, 458 insertions(+)\n create mode 100644 Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt\n create mode 100644 drivers/irqchip/irq-meson-gpio.c","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"EsSYbHq2\"; \n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=baylibre-com.20150623.gappssmtp.com\n\theader.i=@baylibre-com.20150623.gappssmtp.com header.b=\"tmVk3PmK\"; \n\tdkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xwnYc5Dv2z9s7G\n\tfor ;\n\tMon, 18 Sep 2017 23:54:28 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dtwVC-0000vx-Dd; Mon, 18 Sep 2017 13:54:22 +0000","from mail-wr0-x234.google.com ([2a00:1450:400c:c0c::234])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dtwNh-0004SK-2V for linux-arm-kernel@lists.infradead.org;\n\tMon, 18 Sep 2017 13:46:41 +0000","by mail-wr0-x234.google.com with SMTP id v109so550222wrc.1\n\tfor ;\n\tMon, 18 Sep 2017 06:46:16 -0700 (PDT)","from localhost.localdomain ([90.63.244.31])\n\tby smtp.googlemail.com with ESMTPSA id\n\tn29sm6888585wmi.46.2017.09.18.06.46.14\n\t(version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256);\n\tMon, 18 Sep 2017 06:46:14 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:MIME-Version:Cc:List-Subscribe:\n\tList-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:References:\n\tIn-Reply-To:Message-Id:Date:Subject:To:From:Reply-To:Content-ID:\n\tContent-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc\n\t:Resent-Message-ID:List-Owner;\n\tbh=46OXd5pOyrWkWY2VTEXZXlOqz8jo0UbWRcU6ASkkBo0=;\n\tb=EsSYbHq2lglCsthIVrdKfCwJa2\n\tXhehe+oCEySQ0usHN0wGo2bd2x6yp83WVpjMPNxVr0VvGRUunN7NrWGTLN6qsTtXeJWzWxkFR5tlk\n\thjyrxfjBKi25ZTOOpN8QknyY4tth6TyjR5JYlCySJ6y4n6AO5lTSvT+yV8z8VF2J2A/NTEhvTKk1N\n\tzU4Oeu4FSNaJHjIOndBZGCmt68II4hhrGSw3qAZXvwTHNqJn7IbgwKdWT7piGNPAQtyEofh4zVXg5\n\teLmOtR4+J7v0rD2zcNZ+27PnVuCmpnRollZJYBxdn0xB3S0opLALwnd9JbscmPnkylEvlY1CoAUU0\n\tBEG6EX/w==;","v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=baylibre-com.20150623.gappssmtp.com; s=20150623;\n\th=from:to:cc:subject:date:message-id:in-reply-to:references;\n\tbh=vA3XijoA0DcWQnuDzYXPF7k4YAfktZr3K71pjj4ysXM=;\n\tb=tmVk3PmKaXY1vloUhIkZ8ztpNwyKFqcZYC9JPnCwz3+JDT3SwK3iMzz/S3nTL8Cdg9\n\th6EpIrmeD9MZOZklGBJ22HiBCDAHdueIk7lxspNy+Kv/DiQ78r2BKjLUURPtSJoKYj69\n\tYKX0Ek3qSP1qDtlKoajMf0NH6eOoCZSWbeuhY8Bl0d8z8WssG18YZr/3NfjUjK7CK9W4\n\tKsqvSRbgX43/+oWIGyHtTMoAOQozJZnk8fMiTunbe6Q1FxArRH+C+kKVsSYnsKvV33jb\n\tTznJr17XIxT1Zq0HnVA6ibazaP+RgcmiPth5e28ewbBdyltt8MLKNHuKGrBkt9PDpM7m\n\tvpyw=="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to\n\t:references;\n\tbh=vA3XijoA0DcWQnuDzYXPF7k4YAfktZr3K71pjj4ysXM=;\n\tb=Pk1n4X1i34xqvwkkSM2rOGOGqMidpP4AEvaLJuUp7Jdn4SjUUp3+PD5KKhgvBLoQHg\n\tdidxi13k2MiIsamc0B+U1ozo+LmFQBAeaEY2mJvDzaoe5/gpVNHApqzivm9+KcQPQTxt\n\tGvJUSdLAYgAfYfdlka4bBFD+TXvHBOyX519GOmp2fBh6IbUAqeIJimi7B/kn5PY9Rzvv\n\tbb7Xz5o6Mrawrp/8o6QAyKuCiOmoPGfCSClINcdwaOCkbjo8FdpWxhWqUxCCuXMSALdw\n\t4obax2Mowj5ByRzxUK0arem3jR4Lq52RKtJLMj9Je76Ybpw9+ifGVB7ljaBGUV/FekGk\n\t/HpA==","X-Gm-Message-State":"AHPjjUjEnskwB9cEwKSB1ma8oQoD97jQJ2qB9je7NJIXqi39aKkh1f5w\n\tY0L5WrjySjZx4hNXuRU=","X-Google-Smtp-Source":"ADKCNb5R/kEkKf9HCBVqrSP1AZoMz5AufLDYRNIKFTC1NlHvTaCjOJaIVib6IRT9rIo/CVLoGtQj3g==","X-Received":"by 10.223.155.194 with SMTP id e2mr31453232wrc.49.1505742375162; \n\tMon, 18 Sep 2017 06:46:15 -0700 (PDT)","From":"Jerome Brunet ","To":"Marc Zyngier , Thomas Gleixner ,\n\tJason Cooper ,\n\tHeiner Kallweit ","Subject":"[PATCH v4 0/2] irqchip: meson: add support for the gpio interrupt\n\tcontroller","Date":"Mon, 18 Sep 2017 15:46:08 +0200","Message-Id":"<20170918134610.17743-2-jbrunet@baylibre.com>","X-Mailer":"git-send-email 2.13.5","In-Reply-To":"<20170918134610.17743-1-jbrunet@baylibre.com>","References":"<20170918134610.17743-1-jbrunet@baylibre.com>","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170918_064637_306481_82A47DC4 ","X-CRM114-Status":"GOOD ( 16.42 )","X-Spam-Score":"-1.2 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.2 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno\n\ttrust [2a00:1450:400c:c0c:0:0:0:234 listed in] [list.dnswl.org]\n\t0.7 SPF_SOFTFAIL SPF: sender does not match SPF record (softfail)\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"devicetree@vger.kernel.org, Kevin Hilman ,\n\tlinux-kernel@vger.kernel.org, Carlo Caione ,\n\tlinux-amlogic@lists.infradead.org, linux-arm-kernel@lists.infradead.org, \n\tJerome Brunet ","MIME-Version":"1.0","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1789787,"web_url":"http://patchwork.ozlabs.org/comment/1789787/","msgid":"<87shegieqv.fsf@on-the-bus.cambridge.arm.com>","list_archive_url":null,"date":"2017-10-18T16:48:24","subject":"Re: [PATCH v4 0/2] irqchip: meson: add support for the gpio\n\tinterrupt controller","submitter":{"id":7353,"url":"http://patchwork.ozlabs.org/api/people/7353/","name":"Marc Zyngier","email":"marc.zyngier@arm.com"},"content":"On Mon, Sep 18 2017 at 3:46:07 pm BST, Jerome Brunet wrote:\n> This patch series adds support for the GPIO interrupt controller found on\n> Amlogic's meson SoC families.\n>\n> Unlike what the name suggests, this controller is not part of the SoC\n> GPIO subsystem. It is a separate controller which can watch almost all\n> gpio pads of the SoC and generate and interrupt from it. \"Almost\" because\n> there are always exceptions and some specific gpios (TEST_N) lack this\n> capability.\n>\n> Hardware wise, the controller is a 256 to 8 router with a filtering block\n> to select edge or level input and the polarity of the signal. We can't\n> setup the filtring to generate a signal on both the high and low polarity\n> so, ATM, IRQ_TYPE_EDGE_BOTH is not supported.\n>\n> The number of interrupt line routed to the controller depends on the SoC,\n> and essentially the total number of GPIO available on the different gpio\n> controllers of the SoC.\n>\n> This series has been tested on Amlogic S905-P200 board with the front\n> panel power button. Also tested on the Nanopi-k2 with the ethernet PHY\n> interrupt pin.\n>\n> This work is derived from the previous work of Carlo Caione [1].\n\nQueued for 4.15.\n\n\tM.","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"EmFlyBwn\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3yHJ1s4DZ4z9t3h\n\tfor ;\n\tThu, 19 Oct 2017 03:49:37 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e4rXA-00086A-Vw; Wed, 18 Oct 2017 16:49:33 +0000","from usa-sjc-mx-foss1.foss.arm.com ([217.140.101.70]\n\thelo=foss.arm.com)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e4rWS-0007Kd-EJ; Wed, 18 Oct 2017 16:49:02 +0000","from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249])\n\tby usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 36688F;\n\tWed, 18 Oct 2017 09:48:28 -0700 (PDT)","from on-the-bus (unknown [10.3.33.38])\n\tby usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id\n\t691D33F25D; Wed, 18 Oct 2017 09:48:26 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:References\n\t:In-Reply-To:Subject:To:From:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=7Tt5rm9zOd/pMKBMbcMcAIKtM1vyl6f+Bay8K7bMuk4=;\n\tb=EmFlyBwna2Mbvc\n\tMXosZ6LI5eUr3PZxNUSG8OFfpnD5MReb2H5WtYDhvnTaG1qI+UCy4VJnaSRhx/lnjmEYHLIiiPH0t\n\topDPv2f09vOwXrn8zEryOlQJUv7XghkvEh9Yq8LlKKCy8wpYIRk6fuAta2qdGuUYL2ETsfoF66rLX\n\ttEp2WUqbdOZkuH9Ph8HM5ksrNAVcIa+fFXdSDPNhtrHXutZsrp5nbgVm4N77yVVB4jLI6W0RpCgFw\n\t85qI+yYL4KFv0ag5tCfkPJqOgJ2y55AvlIK2vA0XQExdUolskcmsMu5YwwZA7brGCM7+d4oVWlhuL\n\tpCKfuxSBwyDRECn8myvQ==;","From":"Marc Zyngier ","To":"Jerome Brunet ","Subject":"Re: [PATCH v4 0/2] irqchip: meson: add support for the gpio\n\tinterrupt controller","In-Reply-To":"<20170918134610.17743-1-jbrunet@baylibre.com> (Jerome Brunet's\n\tmessage of \"Mon, 18 Sep 2017 15:46:07 +0200\")","Organization":"ARM Ltd","References":"<20170918134610.17743-1-jbrunet@baylibre.com>","User-Agent":"Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)","Date":"Wed, 18 Oct 2017 17:48:24 +0100","Message-ID":"<87shegieqv.fsf@on-the-bus.cambridge.arm.com>","MIME-Version":"1.0","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171018_094848_682338_DBC8DCE8 ","X-CRM114-Status":"GOOD ( 13.01 )","X-Spam-Score":"-6.9 (------)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-6.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-5.0 RCVD_IN_DNSWL_HI RBL: Sender listed at http://www.dnswl.org/,\n\thigh trust [217.140.101.70 listed in list.dnswl.org]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"devicetree@vger.kernel.org, Jason Cooper ,\n\tKevin Hilman , linux-kernel@vger.kernel.org,\n\tCarlo Caione , linux-amlogic@lists.infradead.org,\n\tThomas Gleixner ,\n\tlinux-arm-kernel@lists.infradead.org, \n\tHeiner Kallweit ","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}}]