linux-kernel - Re: [PATCH] irqchip/irq-imx-gpcv2: add IRQCHIP

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: <3e287cee-f10f-7233-ee40-fd736497d3e6@arm.com>
Date:   Tue, 18 Dec 2018 12:59:42 +0000
From:   Marc Zyngier <marc.zyngier@....com>
To:     Lucas Stach <l.stach@...gutronix.de>,
        Andrey Smirnov <andrew.smirnov@...il.com>
Cc:     Thomas Gleixner <tglx@...utronix.de>, patchwork-lst@...gutronix.de,
        Jason Cooper <jason@...edaemon.net>, kernel@...gutronix.de,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] irqchip/irq-imx-gpcv2: add IRQCHIP_DECLARE for i.MX8MQ
 compatible

On 17/12/2018 15:51, Lucas Stach wrote:
> Hi Marc,
> 
> Am Freitag, den 14.12.2018, 13:15 +0100 schrieb Lucas Stach:
>> The GPC node on i.MX8MQ can not claim to be compatible with the i.MX7D
>> GPC, as the power gating part has some significant differences. Thus we
>> can not rely on the irqchip being probed with the old compatible.
>>
>>> Signed-off-by: Lucas Stach <l.stach@...gutronix.de>
>> ---
>>  drivers/irqchip/irq-imx-gpcv2.c | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/irqchip/irq-imx-gpcv2.c b/drivers/irqchip/irq-imx-gpcv2.c
>> index 17a2dad2d4c2..68b9cca24044 100644
>> --- a/drivers/irqchip/irq-imx-gpcv2.c
>> +++ b/drivers/irqchip/irq-imx-gpcv2.c
>> @@ -289,4 +289,5 @@ static int __init imx_gpcv2_irqchip_init(struct device_node *node,
>>>  	return 0;
>>  }
>>  
>> -IRQCHIP_DECLARE(imx_gpcv2, "fsl,imx7d-gpc", imx_gpcv2_irqchip_init);
>> +IRQCHIP_DECLARE(imx_gpcv2_imx7d, "fsl,imx7d-gpc", imx_gpcv2_irqchip_init);
>> +IRQCHIP_DECLARE(imx_gpcv2_imx8mq, "fsl,imx8mq-gpc", imx_gpcv2_irqchip_init);
> 
> It would be nice if we could get this into 4.21, as this is the last
> missing bit preventing me from pushing forward on the patches
> describing the i.MX8MQ IRQ hierarchy properly for the next cycle.

Now applied.

Thanks,

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