From mboxrd@z Thu Jan 1 00:00:00 1970 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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 69A14C54F32 for ; Wed, 28 Aug 2024 12:39:41 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:MIME-Version:References:In-Reply-To:Message-ID:Subject:CC:To: From:Date:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=kF0Gb3ftJ/CiO4TRWQvddhSVwuQqwGT5EBj6Zn+FT2s=; b=nkKvZbTgjsVX/ngZI3tk5u/YbT xP1PdkZYyNl4JqBUsJR25kMVgZqVOVnnD+Dn3JSMY5YvUhQRgg92KmWSCceGtRR5u2xMRdfUy+gYo tG523M68fYUITd0e0oCiTNLdtMUQrmki+CicXakuhAesLEnCv4fV7psqKgxFh3O7w/CBZkb6BkenO +jhigs8PvBNsLdRBS866fsMqjchhoSYZWUrPRPxO2N/QyP4SuBg2GJY2MmR5Ii8wPHPB64g94rPKu qWDq4i0CLhqWYhsr+4ax2ybuIp57mjv9fancdS1vPhRTCNOXTCjvgOHXyYJs+l+IMxxZ+cvpkzGX6 UAgkRtgA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sjHxS-0000000FLcl-3gGw; Wed, 28 Aug 2024 12:39:30 +0000 Received: from frasgout.his.huawei.com ([185.176.79.56]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sjHw3-0000000FLAR-1FUr for linux-arm-kernel@lists.infradead.org; Wed, 28 Aug 2024 12:38:04 +0000 Received: from mail.maildlp.com (unknown [172.18.186.216]) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4Wv3l016Drz6H7RN; Wed, 28 Aug 2024 20:34:36 +0800 (CST) Received: from lhrpeml500005.china.huawei.com (unknown [7.191.163.240]) by mail.maildlp.com (Postfix) with ESMTPS id 5A9B7140447; Wed, 28 Aug 2024 20:37:52 +0800 (CST) Received: from localhost (10.203.177.66) by lhrpeml500005.china.huawei.com (7.191.163.240) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.39; Wed, 28 Aug 2024 13:37:51 +0100 Date: Wed, 28 Aug 2024 13:37:50 +0100 From: Jonathan Cameron To: Jinjie Ruan CC: , , , , , , , , , , , , , , , , , , , , , , , , , , , , , Subject: Re: [PATCH net-next v2 04/13] net: dsa: realtek: Use __free() to simplify code Message-ID: <20240828133750.00007963@Huawei.com> In-Reply-To: <20240828032343.1218749-5-ruanjinjie@huawei.com> References: <20240828032343.1218749-1-ruanjinjie@huawei.com> <20240828032343.1218749-5-ruanjinjie@huawei.com> Organization: Huawei Technologies Research and Development (UK) Ltd. X-Mailer: Claws Mail 4.1.0 (GTK 3.24.33; x86_64-w64-mingw32) MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.203.177.66] X-ClientProxiedBy: lhrpeml100001.china.huawei.com (7.191.160.183) To lhrpeml500005.china.huawei.com (7.191.163.240) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240828_053803_528440_D3D33BE1 X-CRM114-Status: GOOD ( 16.72 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, 28 Aug 2024 11:23:34 +0800 Jinjie Ruan wrote: > Avoid need to manually handle of_node_put() by using __free(), which > can simplfy code. > > Signed-off-by: Jinjie Ruan One suggestion inline. > --- > v2 > - Split into 2 patches. > --- > drivers/net/dsa/realtek/rtl8366rb.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/net/dsa/realtek/rtl8366rb.c b/drivers/net/dsa/realtek/rtl8366rb.c > index 7001b8b1c028..0acdcdd93ea2 100644 > --- a/drivers/net/dsa/realtek/rtl8366rb.c > +++ b/drivers/net/dsa/realtek/rtl8366rb.c > @@ -1009,7 +1009,6 @@ static int rtl8366rb_setup_all_leds_off(struct realtek_priv *priv) > > static int rtl8366rb_setup_leds(struct realtek_priv *priv) > { > - struct device_node *leds_np; > struct dsa_switch *ds = &priv->ds; > struct dsa_port *dp; > int ret = 0; > @@ -1018,7 +1017,8 @@ static int rtl8366rb_setup_leds(struct realtek_priv *priv) > if (!dp->dn) > continue; > > - leds_np = of_get_child_by_name(dp->dn, "leds"); > + struct device_node *leds_np __free(device_node) = > + of_get_child_by_name(dp->dn, "leds"); > if (!leds_np) { > dev_dbg(priv->dev, "No leds defined for port %d", > dp->index); > @@ -1032,7 +1032,6 @@ static int rtl8366rb_setup_leds(struct realtek_priv *priv) > break; > } > > - of_node_put(leds_np); > if (ret) > return ret; Move this return up to the only place it can come from which is inside the loop where the break is. You can then avoid initializing ret and indeed could bring it's scope into the loop for_each_child_of_node(leds_np, led_np) { int ret = rtl8366rb_setup_led(priv, dp, of_fwnode_handle(led_np)); if (ret) return ret; } > }