From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marc Zyngier Subject: Re: [PATCH v6 00/15] qcom: spmi: add support for hierarchical IRQ chip Date: Sun, 20 Jan 2019 11:36:30 +0000 Message-ID: <86h8e3a7cx.wl-marc.zyngier@arm.com> References: <20190119204252.18370-1-masneyb@onstation.org> Mime-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=US-ASCII Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Linus Walleij Cc: Brian Masney , Stephen Boyd , Bjorn Andersson , Andy Gross , Shawn Guo , Doug Anderson , "open list:GPIO SUBSYSTEM" , Nicolas Dechesne , Niklas Cassel , David Brown , Rob Herring , Mark Rutland , "thierry.reding@gmail.com" , linux-arm-msm@vger.kernel.org, "linux-kernel@vger.kernel.org" , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" List-Id: linux-arm-msm@vger.kernel.org On Sat, 19 Jan 2019 23:13:45 +0000, Linus Walleij wrote: > > On Sat, Jan 19, 2019 at 9:43 PM Brian Masney wrote: > > > This patch series adds hierarchical IRQ chip support to spmi-gpio so > > that device tree consumers can request an IRQ directly from the GPIO > > block rather than having to request an IRQ from the underlying PMIC. > > > > For more background information, see the email thread with Linus > > Walleij's excellent description of the problem at > > https://www.spinics.net/lists/linux-gpio/msg34655.html. > > > > This work was tested on a LG Nexus 5 (hammerhead) phone. My status page > > at https://masneyb.github.io/nexus-5-upstream/ describes what is working > > so far with the upstream kernel. > > > > Changes since v5: > > - Patch 4: Set handler to edge or level when the IRQ is mapped. > > - Patch 7: Change IRQ_TYPE_NONE to IRQ_TYPE_EDGE_RISING > > - Patch 14: New patch to validate type when mapping IRQ > > If Marc Z is happy I think I will apply all patches on an immutable branch in As a matter of fact, I am! > the pin control tree, so that ARM SoC and GPIO can pull it in later if need > be. (E.g. if they get conflicts.) > > I was thinking to also include the DTS changes as it all is so neatly > coupled, then offer the branch to ARM SoC. > > Anyone against? No objection from me whatsoever. Thanks, M. -- Jazz is not dead, it just smell funny.