linux-kernel - Re: [PATCH 1/1] irqchip: Add config MTK_SYSIRQ and MTK

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]

Message-ID: <3dc953265ed4cd4300bd9600bf7e33d6@kernel.org>
Date:   Tue, 23 Jun 2020 09:37:49 +0100
From:   Marc Zyngier <maz@...nel.org>
To:     Hanks Chen <hanks.chen@...iatek.com>
Cc:     CC Hwang <cc.hwang@...iatek.com>,
        Jason Cooper <jason@...edaemon.net>, wsd_upstream@...iatek.com,
        Loda Chou <loda.chou@...iatek.com>,
        linux-kernel@...r.kernel.org, linux-mediatek@...ts.infradead.org,
        Matthias Brugger <matthias.bgg@...il.com>,
        Thomas Gleixner <tglx@...utronix.de>,
        linux-arm-kernel@...ts.infradead.org, saravanak@...gle.com
Subject: Re: [PATCH 1/1] irqchip: Add config MTK_SYSIRQ and MTK_CIRQ

+ Saravana

On 2020-06-23 07:36, Hanks Chen wrote:

[...]

> Hi Marc,
> 
> I want to break the dependency between ARCH_MEDIATEK and CIRQ/SYSIRQ,
> because we need to follow the GKI.

This isn't what GKI mandates. GKI requires that the SoC code is compiled
as modules, not that it is dissociated from its platform (which would
be completely pointless).

> Could I add the MTK_CIRQ and MTK_SYSIRQ into defconfig?
> (arch/arm64/config/defconfig)

I don't deal with defconfig at all. That's a patch for the arm-soc 
people.

> 
> It would ensures all platform are executable.
> 
> e.g.
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> 
>  CONFIG_QCOM_PDC=y
> +CONFIG_MTK_SYSIRQ=m
> +CONFIG_MTK_CIRQ=y
>  CONFIG_RESET_QCOM_AOSS=y
> 
> --- a/drivers/irqchip/Kconfig
> +++ b/drivers/irqchip/Kconfig
> @@ -572,4 +572,18 @@ config LOONGSON_PCH_MSI
>         help
>           Support for the Loongson PCH MSI Controller.
> 
> +config MTK_SYSIRQ
> +       tristate "Mediatek interrupt polarity controller"
> +       depends on ARCH_MEDIATEK || COMPILE_TEST
> +       help
> +         Interrupt polarity controller driver to swap polarity for
> +         interrupts for Mediatek mobile chips.
> +
> +config MTK_CIRQ
> +       bool "Mediatek low-power interrupt controller"
> +       depends on ARCH_MEDIATEK || COMPILE_TEST
> +       help
> +         Low-power interrupt controller driver to monitor IRQS
> +         in the sleep mode for Mediatek mobile chips.
> +
> 
> 
> P.S I'll make the irq-mtk-sysirq driver as a loadable kernel module for
> GKI

You might as well turn both drivers into modules. Saravana was working
on a set of patches to ease this transition.

Thanks,

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