[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <163914278213.23020.634415261822719928.tip-bot2@tip-bot2>
Date: Fri, 10 Dec 2021 13:26:22 -0000
From: "irqchip-bot for Ye Guojin" <tip-bot2@...utronix.de>
To: linux-kernel@...r.kernel.org
Cc: Zeal Robot <zealci@....com.cn>, Ye Guojin <ye.guojin@....com.cn>,
Florian Fainelli <f.fainelli@...il.com>,
Marc Zyngier <maz@...nel.org>, tglx@...utronix.de
Subject: [irqchip: irq/irqchip-fixes] irqchip/irq-bcm7120-l2: Add put_device()
after of_find_device_by_node()
The following commit has been merged into the irq/irqchip-fixes branch of irqchip:
Commit-ID: c3fbab7767c53397d7b849799474f5a27cf306e6
Gitweb: https://git.kernel.org/pub/scm/linux/kernel/git/maz/arm-platforms/c3fbab7767c53397d7b849799474f5a27cf306e6
Author: Ye Guojin <ye.guojin@....com.cn>
AuthorDate: Tue, 09 Nov 2021 05:59:58
Committer: Marc Zyngier <maz@...nel.org>
CommitterDate: Fri, 10 Dec 2021 13:23:13
irqchip/irq-bcm7120-l2: Add put_device() after of_find_device_by_node()
This was found by coccicheck:
./drivers/irqchip/irq-bcm7120-l2.c,328,1-7,ERROR missing put_device;
call of_find_device_by_node on line 234, but without a corresponding
object release within this function.
./drivers/irqchip/irq-bcm7120-l2.c,341,1-7,ERROR missing put_device;
call of_find_device_by_node on line 234, but without a corresponding
object release within this function.
Reported-by: Zeal Robot <zealci@....com.cn>
Signed-off-by: Ye Guojin <ye.guojin@....com.cn>
Reviewed-by: Florian Fainelli <f.fainelli@...il.com>
Signed-off-by: Marc Zyngier <maz@...nel.org>
Link: https://lore.kernel.org/r/20211109055958.130287-1-ye.guojin@zte.com.cn
---
drivers/irqchip/irq-bcm7120-l2.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/irqchip/irq-bcm7120-l2.c b/drivers/irqchip/irq-bcm7120-l2.c
index d80e67a..bb6609c 100644
--- a/drivers/irqchip/irq-bcm7120-l2.c
+++ b/drivers/irqchip/irq-bcm7120-l2.c
@@ -238,6 +238,7 @@ static int __init bcm7120_l2_intc_probe(struct device_node *dn,
}
data->num_parent_irqs = platform_irq_count(pdev);
+ put_device(&pdev->dev);
if (data->num_parent_irqs <= 0) {
pr_err("invalid number of parent interrupts\n");
ret = -ENOMEM;
Powered by blists - more mailing lists