linux-kernel - Re: [PATCH v3 0/2] irqchip/mchp-eic: add driver for Microchip EIC

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] [day] [month] [year] [list]

Message-ID: <68755c28-946a-a526-21b1-e40dd4fc5722@microchip.com>
Date:   Fri, 24 Sep 2021 16:32:32 +0000
From:   <Claudiu.Beznea@...rochip.com>
To:     <tglx@...utronix.de>, <maz@...nel.org>, <robh+dt@...nel.org>
CC:     <linux-kernel@...r.kernel.org>, <devicetree@...r.kernel.org>,
        <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH v3 0/2] irqchip/mchp-eic: add driver for Microchip EIC

Sorry, this had to be v4. I'll fix the versioning and re-post.

Thank you,
Claudiu Beznea

On 24.09.2021 19:28, Claudiu Beznea wrote:
> Hi,
> 
> This series adds support for Microchip External Interrupt Controller
> present on SAMA7G5. The controller supports for 2 external interrupt
> lines and is connected to GIC as follows:
> 
> pinX   +------+ EXT_IRQ0 +------+ int 153 (for pinX) +------+
> ------>|      |--------->|      |------------------->|      |
> pinY   | PIO  | EXT_IRQ1 | EIC  | int 154 (for pinY) | GIC  |
> ------>|      |--------->|      |------------------->|      |
>        +------+          +------+                    +------+
> 
> where PIO is the pin controller.
> 
> Thank you,
> Claudiu Beznea
> 
> Changes in v3:
> - use irq_domain_translate_twocell() instead of mchp_eic_domain_translate()
>   from v2
> - use IRQCHIP_PLATFORM_DRIVER_BEGIN(), IRQCHIP_MATCH(),
>   IRQCHIP_PLATFORM_DRIVER_END() instead of builtin_platform_driver()
> - register suspend/resume functins via syscore_ops; along with this
>   driver private data structure (of type struct mchp_eic has been declared
>   as static: struct mchp_eic *eic;)
> - collected tags
> 
> Changes in v2:
> - fix compilation warning on bindings
> 
> Changes in v2:
> - s/mchp/microchip in bindings
> - get rid of glitch filter settings
> - use s/eic@...terrupt-controller@ in bindings
> - removed '' around interrupts in bindings as proposed by Rob
> - removed glitch filter settings
> - switched to hierarchical irqchip driver
> - addressed review comments
> - use builtin_platform_driver instead of module_platform_driver
> 
> Claudiu Beznea (2):
>   dt-bindings: microchip,eic: add bindings
>   irqchip/mchp-eic: add support
> 
>  .../interrupt-controller/microchip,eic.yaml   |  73 +++++
>  MAINTAINERS                                   |   6 +
>  drivers/irqchip/Kconfig                       |   8 +
>  drivers/irqchip/Makefile                      |   1 +
>  drivers/irqchip/irq-mchp-eic.c                | 280 ++++++++++++++++++
>  5 files changed, 368 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/interrupt-controller/microchip,eic.yaml
>  create mode 100644 drivers/irqchip/irq-mchp-eic.c
>