linux-kernel - RE: [PATCH 1/1] irqchip: gpcv2: make config option visible

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: <AM0PR04MB421158B2E042D623792A296E809C0@AM0PR04MB4211.eurprd04.prod.outlook.com>
Date:   Fri, 18 Jan 2019 09:49:43 +0000
From:   Aisheng Dong <aisheng.dong@....com>
To:     Marc Zyngier <marc.zyngier@....com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
CC:     "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "shawnguo@...nel.org" <shawnguo@...nel.org>,
        dl-linux-imx <linux-imx@....com>,
        "l.stach@...gutronix.de" <l.stach@...gutronix.de>,
        Thomas Gleixner <tglx@...utronix.de>,
        Jason Cooper <jason@...edaemon.net>,
        Andrey Smirnov <andrew.smirnov@...il.com>
Subject: RE: [PATCH 1/1] irqchip: gpcv2: make config option visible

> From: Marc Zyngier [mailto:marc.zyngier@....com]
> Sent: Friday, January 18, 2019 5:13 PM
> On 18/01/2019 07:41, Aisheng Dong wrote:
> > Make GPCv2 config option visible, then user can select it in defconfig
> > for ARMv8 platforms.
> >
> > Cc: Thomas Gleixner <tglx@...utronix.de>
> > Cc: Jason Cooper <jason@...edaemon.net>
> > Cc: Marc Zyngier <marc.zyngier@....com>
> > Cc: Shawn Guo <shawnguo@...nel.org>
> > Cc: Lucas Stach <l.stach@...gutronix.de>
> > Cc: Andrey Smirnov <andrew.smirnov@...il.com>
> > Signed-off-by: Dong Aisheng <aisheng.dong@....com>
> > ---
> >  drivers/irqchip/Kconfig | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/irqchip/Kconfig b/drivers/irqchip/Kconfig index
> > 3d1e607..ed04d47 100644
> > --- a/drivers/irqchip/Kconfig
> > +++ b/drivers/irqchip/Kconfig
> > @@ -289,7 +289,8 @@ config RENESAS_H8S_INTC
> >  	select IRQ_DOMAIN
> >
> >  config IMX_GPCV2
> > -	bool
> > +	bool "i.MX GPCv2 IRQ chip"
> > +	depends on ARCH_MXC || (COMPILE_TEST && OF)
> >  	select IRQ_DOMAIN
> >  	help
> >  	  Enables the wakeup IRQs for IMX platforms with GPCv2 block
> >
> 
> How does it help exactly? It is pretty difficult for a user to know exactly what
> they need. I'd rather see it selected by ARCH_MXC, which makes it

ARM64 SoC maintainers suggest not add more driver specific options except an
Generic ARCH option.

As GPCv2 is also used in MX8MQ. So we may select it in armv8 defconfig.
If this option is invisible, we can't select it.

Regards
Dong Aisheng

> unambiguous.
> 
> Thanks,
> 
> 	M.
> --
> Jazz is not dead. It just smells funny...