linux-kernel - Re: [PATCH 3/4] i.MX: GPC: Use IRQCHIP_DECLARE

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: <1485510840.28751.26.camel@pengutronix.de>
Date:   Fri, 27 Jan 2017 10:54:00 +0100
From:   Lucas Stach <l.stach@...gutronix.de>
To:     Andrey Smirnov <andrew.smirnov@...il.com>
Cc:     linux-kernel@...r.kernel.org, Jason Cooper <jason@...edaemon.net>,
        Marc Zyngier <marc.zyngier@....com>,
        Thomas Gleixner <tglx@...utronix.de>,
        Shawn Guo <shawnguo@...nel.org>,
        linux-arm-kernel@...ts.infradead.org, yurovsky@...il.com
Subject: Re: [PATCH 3/4] i.MX: GPC: Use IRQCHIP_DECLARE_DRIVER

Hi Andrey,

Am Donnerstag, den 26.01.2017, 14:05 -0800 schrieb Andrey Smirnov:
> Use IRQCHIP_DECLARE_DRIVER instead of Use IRQCHIP_DECLARE to allow
> 'imx_gpc_driver' to be probed in addition to irqchip initialized by
> imx_gpc_init().
> 
> Cc: yurovsky@...il.com
> Cc: Shawn Guo <shawnguo@...nel.org>
> Cc: Thomas Gleixner <tglx@...utronix.de>
> Cc: Jason Cooper <jason@...edaemon.net>
> Cc: Marc Zyngier <marc.zyngier@....com>
> Cc: linux-arm-kernel@...ts.infradead.org
> Signed-off-by: Andrey Smirnov <andrew.smirnov@...il.com>
> ---
>  arch/arm/mach-imx/gpc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/mach-imx/gpc.c b/arch/arm/mach-imx/gpc.c
> index 1dc2a34..8f0d19b 100644
> --- a/arch/arm/mach-imx/gpc.c
> +++ b/arch/arm/mach-imx/gpc.c
> @@ -279,7 +279,7 @@ static int __init imx_gpc_init(struct device_node *node,
>  
>  	return 0;
>  }
> -IRQCHIP_DECLARE(imx_gpc, "fsl,imx6q-gpc", imx_gpc_init);
> +IRQCHIP_DECLARE_DRIVER(imx_gpc, "fsl,imx6q-gpc", imx_gpc_init);
>  
>  void __init imx_gpc_check_dt(void)
>  {

I don't see the other patches from the series on the list, so commenting
here: You might want to take a look at my GPC rework series, it has a
much nicer binding for the power domains, that you might want to use for
the GPCv2 also.

Additionally I think the power domain stuff doesn't belong into the
irqchip driver, but rather in a separate driver in drivers/soc like it
is done in my series.

I'll send out an updated version of this series today and you might
consider basing your GPCv2 on top of that.

Regards,
Lucas