| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <1414163209.14830.614.camel@mtksdaap41> Date: Fri, 24 Oct 2014 23:06:49 +0800 From: Yingjoe Chen <yingjoe.chen@...iatek.com> To: Matthias Brugger <matthias.bgg@...il.com> CC: Rob Herring <robh+dt@...nel.org>, Jiang Liu <jiang.liu@...ux.intel.com>, Marc Zyngier <marc.zyngier@....com>, <arm@...nel.org>, Pawel Moll <pawel.moll@....com>, Mark Rutland <mark.rutland@....com>, Ian Campbell <ijc+devicetree@...lion.org.uk>, Kumar Gala <galak@...eaurora.org>, Russell King <linux@....linux.org.uk>, Thomas Gleixner <tglx@...utronix.de>, Jason Cooper <jason@...edaemon.net>, Benjamin Herrenschmidt <benh@...nel.crashing.org>, Grant Likely <grant.likely@...aro.org>, Hans de Goede <hdegoede@...hat.com>, Boris BREZILLON <boris.brezillon@...e-electrons.com>, "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, "linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org>, Olof Johansson <olof@...om.net>, Arnd Bergmann <arnd@...db.de>, srv_heupstream <srv_heupstream@...iatek.com>, Yingjoe Chen <yingjoe.chen@...il.com>, Hsien-Chun Yen <hc.yen@...iatek.com>, huang eddie <eddie.huang@...iatek.com>, Nathan Chung <nathan.chung@...iatek.com>, Yuhau Chen <yh.chen@...iatek.com> Subject: Re: [PATCH v4 0/7] ARM: mediatek: Add support for interrupt polarity On Fri, 2014-10-24 at 16:59 +0200, Matthias Brugger wrote: > Hi Joe, > > 2014-10-23 17:53 GMT+02:00 Yingjoe Chen <yingjoe.chen@...iatek.com>: > > This series is 4th version of interrupt polarity support for MediaTek SoCs. > > This is based on Jiang's hierarchy irqdomain v2 [1] and my mediatek SoC > > basic support [2]. > > > > This version addressed comments from previous discussion[3]. I removed > > arm,hierarchy-irq-domain property. When GIC is probed by DT, it will > > support hierarchy irqdomain. I still keep the first patch because this is > > still base on hierarchy irqdomain v2. > > Is this set based on v3.17-rc1? > Is it based on one of Jiangs trees [0]? > > I tried to use your patch set with v3.18-rc1 using Jiangs tree but the > kernel crashed on mt6589 with a null-pointer exception [1]. A test > with mt6592 didn't work either. > > It would be convenient if you rebase your patches to v3.18-rc1. It is based on Jiang's tree[0], which is based on v3.17-rc1. So yes, this series is still on v3.17-rc1. All I need from Jiang's series are the first 2 patches, I'll bring them to v3.18-rc1 to test next week. Joe.C [0] https://github.com/jiangliu/linux.git irqdomain/p2v2 > > Thanks, > Matthias > > [0] https://github.com/jiangliu/linux > [1] http://pastebin.com/3F1pVaM9 > > > > > In Jiang's version of irq_create_of_mapping, if irqdomain is hierarchy, it > > will not perform irq_find_mapping check and set_type. The outermost > > irqdomain need to take care of that. Because we will have several different > > outermost irqdomain in different ARM SoCs, this cause code duplication. I > > moved them back to irq_create_of_mapping. Since Jiang's changes are not > > merged yet, maybe that patch should be merged there. > > > > Simplified block diagram for interrupt on my system: > > > > +-------+ +-------+ > > ---| SYSIRQ|------|ARM GIC| > > ---| |------| | > > ---| |------| | > > ---| |------| | > > ---| |------| | > > +-------+ +-------+ > > > > In device tree, interrupt-parent for other devices is sysirq, child of gic. > > This describe HW better and allow device to specify polarity as it is sent > > by the device. > > > > When using hierarchy irq domain, gic will use irq_domain_add_linear to > > create irqdomain and all interrupt numbers must come from device tree. My > > /proc/interrupts looks like this now: > > > > # cat /proc/interrupts > > CPU0 > > 16: 1862 MT_SYSIRQ 113 mtk_timer > > 17: 67 MT_SYSIRQ 54 mtk-uart > > > > Changes in v3: > > - Discussion in [3] > > - First implementation using hierarchy irqdomain > > > > [1] http://lists.infradead.org/pipermail/linux-arm-kernel/2014-September/290832.html > > [2] http://lists.infradead.org/pipermail/linux-arm-kernel/2014-September/284553.html > > [3] http://lists.infradead.org/pipermail/linux-arm-kernel/2014-October/293766.html > > > > Yingjoe Chen (7): > > irqdomain: Fix irq_domain_alloc_irqs return check. > > irqdomain: Add back xlate and set_type for hierarchy irq domain > > genirq: Add more helper functions to support stacked irq_chip > > irqchip: gic: Support hierarchy irq domain. > > ARM: mediatek: Add sysirq interrupt polarity support > > ARM: mediatek: Add sysirq in mt6589/mt8135/mt8127 dtsi > > dt-bindings: add bindings for mediatek sysirq > > > > .../bindings/arm/mediatek/mediatek,sysirq.txt | 26 ++++ > > arch/arm/boot/dts/mt6589.dtsi | 14 +- > > arch/arm/boot/dts/mt8127.dtsi | 14 +- > > arch/arm/boot/dts/mt8135.dtsi | 14 +- > > drivers/irqchip/Kconfig | 1 + > > drivers/irqchip/Makefile | 1 + > > drivers/irqchip/irq-gic.c | 54 +++++-- > > drivers/irqchip/irq-mtk-sysirq.c | 168 +++++++++++++++++++++ > > include/linux/irq.h | 6 + > > kernel/irq/chip.c | 28 ++++ > > kernel/irq/irqdomain.c | 31 ++-- > > 11 files changed, 332 insertions(+), 25 deletions(-) > > create mode 100644 Documentation/devicetree/bindings/arm/mediatek/mediatek,sysirq.txt > > create mode 100644 drivers/irqchip/irq-mtk-sysirq.c > > > > -- > > 1.8.1.1.dirty > > > > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists