From nobody Mon Apr 27 04:44:19 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id B62BDC43334 for ; Fri, 17 Jun 2022 06:09:24 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1380256AbiFQGJX (ORCPT ); Fri, 17 Jun 2022 02:09:23 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59386 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233127AbiFQGJW (ORCPT ); Fri, 17 Jun 2022 02:09:22 -0400 Received: from m15111.mail.126.com (m15111.mail.126.com [220.181.15.111]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 1F8A326FB for ; Thu, 16 Jun 2022 23:09:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=126.com; s=s110527; h=From:Subject:Date:Message-Id:MIME-Version; bh=o6mVs PYDGXwvBm364qkC63oDBpWTCRDQYS+fGz/FerM=; b=qF2NH9P40QL7D3uxt8Num kp2OyLY2Kos7GZx6G3+0FLw/aD70kT9VbiuanzylIq7VWeVTVT2mhMBAU0Ts8c4x B8Z/Ym68bl7SMJH3yQDHhDfqE/B2vvYScHY3G4BwDH3//NgLETtJkYE8FDulus/0 4t1nN447Fk2GJ0EPMJt1MM= Received: from localhost.localdomain (unknown [124.16.139.61]) by smtp1 (Coremail) with SMTP id C8mowABnHt9cGqxiJHj2Eg--.23720S2; Fri, 17 Jun 2022 14:08:29 +0800 (CST) From: Liang He To: oss@buserror.net, mpe@ellerman.id.au, benh@kernel.crashing.org, paulus@samba.org Cc: windhl@126.com, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: [PATCH v4] powerpc:85xx: Add missing of_node_put() in sgy_cst1000 Date: Fri, 17 Jun 2022 14:08:26 +0800 Message-Id: <20220617060827.4004795-1-windhl@126.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-CM-TRANSID: C8mowABnHt9cGqxiJHj2Eg--.23720S2 X-Coremail-Antispam: 1Uf129KBjvJXoWxZFWDtF4DCFW5Jw4fKw18AFb_yoW5uryfpF Z8CrZakrWkGw17Ga4SyayDZFy2yw18K3yxJw1fCan7C34UX34qqr10yF15WrnxWrW8C3yr Jr1aya40kFZrAaDanT9S1TB71UUUUUUqnTZGkaVYY2UrUUUUjbIjqfuFe4nvWSU5nxnvy2 9KBjDUYxBIdaVFxhVjvjDU0xZFpf9x0zRT7KfUUUUU= X-Originating-IP: [124.16.139.61] X-CM-SenderInfo: hzlqvxbo6rjloofrz/1tbi3B0jF1pEDvP0zQAAsv Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" In gpio_halt_probe(), of_find_matching_node() will return a node pointer with refcount incremented. We should use of_node_put() in fail path or when it is not used anymore. Signed-off-by: Liang He --- changelog: v4: reuse exist 'err' and use a simple code style, advised by CJ v3: use local 'child_node' advised by Michael. v2: use goto-label patch style advised by Christophe Leroy. v1: add of_node_put() before each exit. arch/powerpc/platforms/85xx/sgy_cts1000.c | 35 ++++++++++++++--------- 1 file changed, 22 insertions(+), 13 deletions(-) diff --git a/arch/powerpc/platforms/85xx/sgy_cts1000.c b/arch/powerpc/platf= orms/85xx/sgy_cts1000.c index 98ae64075193..e4588943fe7e 100644 --- a/arch/powerpc/platforms/85xx/sgy_cts1000.c +++ b/arch/powerpc/platforms/85xx/sgy_cts1000.c @@ -71,6 +71,7 @@ static int gpio_halt_probe(struct platform_device *pdev) { enum of_gpio_flags flags; struct device_node *node =3D pdev->dev.of_node; + struct device_node *child_node; int gpio, err, irq; int trigger; =20 @@ -78,26 +79,29 @@ static int gpio_halt_probe(struct platform_device *pdev) return -ENODEV; =20 /* If there's no matching child, this isn't really an error */ - halt_node =3D of_find_matching_node(node, child_match); - if (!halt_node) + child_node =3D of_find_matching_node(node, child_match); + if (!child_node) return 0; =20 /* Technically we could just read the first one, but punish * DT writers for invalid form. */ - if (of_gpio_count(halt_node) !=3D 1) - return -EINVAL; + if (of_gpio_count(child_node) !=3D 1) { + err =3D -EINVAL; + goto err_put; + } =20 /* Get the gpio number relative to the dynamic base. */ - gpio =3D of_get_gpio_flags(halt_node, 0, &flags); - if (!gpio_is_valid(gpio)) - return -EINVAL; + gpio =3D of_get_gpio_flags(child_node, 0, &flags); + if (!gpio_is_valid(gpio)) { + err =3D -EINVAL; + gotot err_put; + } =20 err =3D gpio_request(gpio, "gpio-halt"); if (err) { printk(KERN_ERR "gpio-halt: error requesting GPIO %d.\n", gpio); - halt_node =3D NULL; - return err; + goto err_put; } =20 trigger =3D (flags =3D=3D OF_GPIO_ACTIVE_LOW); @@ -105,15 +109,14 @@ static int gpio_halt_probe(struct platform_device *pd= ev) gpio_direction_output(gpio, !trigger); =20 /* Now get the IRQ which tells us when the power button is hit */ - irq =3D irq_of_parse_and_map(halt_node, 0); + irq =3D irq_of_parse_and_map(child_node, 0); err =3D request_irq(irq, gpio_halt_irq, IRQF_TRIGGER_RISING | - IRQF_TRIGGER_FALLING, "gpio-halt", halt_node); + IRQF_TRIGGER_FALLING, "gpio-halt", child_node); if (err) { printk(KERN_ERR "gpio-halt: error requesting IRQ %d for " "GPIO %d.\n", irq, gpio); gpio_free(gpio); - halt_node =3D NULL; - return err; + goto err_put; } =20 /* Register our halt function */ @@ -123,7 +126,12 @@ static int gpio_halt_probe(struct platform_device *pde= v) printk(KERN_INFO "gpio-halt: registered GPIO %d (%d trigger, %d" " irq).\n", gpio, trigger, irq); =20 + halt_node =3D child_node; return 0; + +err_put: + of_node_put(child_node); + return err; } =20 static int gpio_halt_remove(struct platform_device *pdev) @@ -139,6 +147,7 @@ static int gpio_halt_remove(struct platform_device *pde= v) =20 gpio_free(gpio); =20 + of_node_put(halt_node); halt_node =3D NULL; } =20 --=20 2.25.1