linux-kernel - Re: [PATCH v3 0/9] irqchip: stm32: add stm32h7 support

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 7 Nov 2017 11:23:09 +0000
From:   Marc Zyngier <marc.zyngier@....com>
To:     Ludovic Barre <ludovic.Barre@...com>,
        Thomas Gleixner <tglx@...utronix.de>,
        Jason Cooper <jason@...edaemon.net>,
        Rob Herring <robh+dt@...nel.org>
Cc:     Maxime Coquelin <mcoquelin.stm32@...il.com>,
        Alexandre Torgue <alexandre.torgue@...com>,
        linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        devicetree@...r.kernel.org
Subject: Re: [PATCH v3 0/9] irqchip: stm32: add stm32h7 support

Hi Ludovic,

On 06/11/17 17:03, Ludovic Barre wrote:
> From: Ludovic Barre <ludovic.barre@...com>
> 
> This series adds:
> -Management of multi-bank of external interrupts
> stm32h7 has up to 96 inputs (3 banks of  32 inputs).
> -Fix initial value after cold/hot boot (wakeup issue).
> 
> Changes v3:
> -remove chip.name and handler, already done by
> irq_alloc_domain_generic_chips
> -add Rob ack in dt-bindings commit
> 
> Changes v2:
> -Remove irq_mask and adds const on struct stm32_exti_bank
> -Add wrapper functions pending and ack
> -Replace BITS_PER_LONG by IRQS_PER_BANK
> -Fill commit message on
> "ARM: dts: stm32: add support of exti on stm32h743"
> -Add system config bank for stm32h7
> 
> Ludovic Barre (9):
>   irqchip: stm32: select GENERIC_IRQ_CHIP
>   irqchip: stm32: add multi-bank management
>   dt-bindings: interrupt-controllers: add compatible string for stm32h7
>   irqchip: stm32: add stm32h7 support
>   irqchip: stm32: fix initial values
>   irqchip: stm32: move the wakeup on interrupt mask
>   ARM: dts: stm32: add exti support for stm32h743
>   ARM: dts: stm32: add system config bank node for stm32h743
>   ARM: dts: stm32: add support of exti on stm32h743 pinctrl
> 
>  .../interrupt-controller/st,stm32-exti.txt         |   4 +-
>  arch/arm/boot/dts/stm32h743-pinctrl.dtsi           |  24 +++
>  arch/arm/boot/dts/stm32h743.dtsi                   |  13 ++
>  drivers/irqchip/Kconfig                            |   1 +
>  drivers/irqchip/irq-stm32-exti.c                   | 206 ++++++++++++++++-----
>  5 files changed, 198 insertions(+), 50 deletions(-)
> 

I'm happy to take the first 6 patches through the irqchip tree. How
about the last 3? I'd rather see them routed via armsoc if there is any
risk of merge conflicts.

How do you want to proceed?

Thanks,

	M.
-- 
Jazz is not dead. It just smells funny...