Received: by 2002:a25:d7c1:0:0:0:0:0 with SMTP id o184csp2258190ybg; Sun, 27 Oct 2019 14:34:21 -0700 (PDT) X-Google-Smtp-Source: APXvYqwqKX/Q+gZv7Iuxx01kWs5LvE9XI2YAEIjfTVJLbbfitJLdpJ2oB7GYeonl8Rwc4QV7HQ0s X-Received: by 2002:a50:875e:: with SMTP id 30mr16447101edv.45.1572212061827; Sun, 27 Oct 2019 14:34:21 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1572212061; cv=none; d=google.com; s=arc-20160816; b=WT4Kws5FvWKTXgd1OSTnDr/MABkVA8/hV7fAd/fi4rPJ/9+P1Y94DIUE/YokcIH5h4 Adnj381OfANzpRPdUd3Kvj3uK+Dipr1F45UQNsVBUvFVMUosAv/rrOXYu4uaj4mcYAdJ IWT7Vj8lDN78Enf3yKsaL6cOX4fUK0YWQ0kjDQYBpnWM8CtOrHbMJ/8w8JLYWRMMtwsV WSMR0dNiaHmMF22gNyzbvQcm2YqDLiWf09JgRKAja0FWB4bQsd0o+ymtiNx3RTINNE7E 62Ta3vL0leGbshuDBCW8QmDv40kYMHVh+Wocg64Mccu+vABqGlK6iWmriNgTwQgZ+KYz 66dw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :user-agent:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=yTXPN0McLE5CtHQiFaX22CZ3UvgpLFmMW7XigmH+uNs=; b=NpyC4BGI2vB/AZZm5rVku/EpXHjISpdieqKKkmj0WO8TW0HuJL7ZUCM8Bz+kcfsRnL GSwPZmu7U+UgiZEA/OCybhvjnf/fAAuu/zToxG9en7yT+QNVMPQXF8ug2zOwA8KSdiIc haOaMfSYLeYeWwFUwLa1H00BwI+/IPE4nEM+GMG1ynmfbUMm1heM7yrHyia57c3Jc/hh +8astaWZEnaBztdAIZnJnQpRqHMj7YYfgGecLkh925gtE30MLIq5nk6c0caoDpdy2yI7 FdoZefiaJpUfIzeU77zQhjJJkeclUZiph2UMspZqDuBSEPkyg42quIhrnExJ8WgEbNCT 83GA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=iiOiojoZ; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id w57si6169813eda.78.2019.10.27.14.33.58; Sun, 27 Oct 2019 14:34:21 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=iiOiojoZ; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1732776AbfJ0Vai (ORCPT + 99 others); Sun, 27 Oct 2019 17:30:38 -0400 Received: from mail.kernel.org ([198.145.29.99]:38822 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1731310AbfJ0VSi (ORCPT ); Sun, 27 Oct 2019 17:18:38 -0400 Received: from localhost (100.50.158.77.rev.sfr.net [77.158.50.100]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id CA16520717; Sun, 27 Oct 2019 21:18:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1572211117; bh=pE0yT9Ol68fBMk/rzKDUq3s4Y8QKqjuEq7VMxFMPklc=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=iiOiojoZAv6DR+EihEm9o5W/zql8Fq6LvGeEOsrxK9yCLI4Ay7umTWsOz5e5mPdfc W3u/y8hCktsDHRbQRaF+knFCAyJ5D/4zvONjRzkH2SFKPfMwkWq+2H4WvVTfsLPhzF b6P6f/lBu+5pjkmhNtUnLqY4ykEsC1u6gShByNNo= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Wen Yang , Linus Walleij , Andrew Lunn , Vivien Didelot , Florian Fainelli , "David S. Miller" , netdev@vger.kernel.org, Sasha Levin Subject: [PATCH 5.3 037/197] net: dsa: rtl8366rb: add missing of_node_put after calling of_get_child_by_name Date: Sun, 27 Oct 2019 21:59:15 +0100 Message-Id: <20191027203353.725873412@linuxfoundation.org> X-Mailer: git-send-email 2.23.0 In-Reply-To: <20191027203351.684916567@linuxfoundation.org> References: <20191027203351.684916567@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Wen Yang [ Upstream commit f32eb9d80470dab05df26b6efd02d653c72e6a11 ] of_node_put needs to be called when the device node which is got from of_get_child_by_name finished using. irq_domain_add_linear() also calls of_node_get() to increase refcount, so irq_domain will not be affected when it is released. Fixes: d8652956cf37 ("net: dsa: realtek-smi: Add Realtek SMI driver") Signed-off-by: Wen Yang Cc: Linus Walleij Cc: Andrew Lunn Cc: Vivien Didelot Cc: Florian Fainelli Cc: "David S. Miller" Cc: netdev@vger.kernel.org Cc: linux-kernel@vger.kernel.org Reviewed-by: Linus Walleij Signed-off-by: David S. Miller Signed-off-by: Sasha Levin --- drivers/net/dsa/rtl8366rb.c | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/drivers/net/dsa/rtl8366rb.c b/drivers/net/dsa/rtl8366rb.c index a268085ffad28..f5cc8b0a7c74c 100644 --- a/drivers/net/dsa/rtl8366rb.c +++ b/drivers/net/dsa/rtl8366rb.c @@ -507,7 +507,8 @@ static int rtl8366rb_setup_cascaded_irq(struct realtek_smi *smi) irq = of_irq_get(intc, 0); if (irq <= 0) { dev_err(smi->dev, "failed to get parent IRQ\n"); - return irq ? irq : -EINVAL; + ret = irq ? irq : -EINVAL; + goto out_put_node; } /* This clears the IRQ status register */ @@ -515,7 +516,7 @@ static int rtl8366rb_setup_cascaded_irq(struct realtek_smi *smi) &val); if (ret) { dev_err(smi->dev, "can't read interrupt status\n"); - return ret; + goto out_put_node; } /* Fetch IRQ edge information from the descriptor */ @@ -537,7 +538,7 @@ static int rtl8366rb_setup_cascaded_irq(struct realtek_smi *smi) val); if (ret) { dev_err(smi->dev, "could not configure IRQ polarity\n"); - return ret; + goto out_put_node; } ret = devm_request_threaded_irq(smi->dev, irq, NULL, @@ -545,7 +546,7 @@ static int rtl8366rb_setup_cascaded_irq(struct realtek_smi *smi) "RTL8366RB", smi); if (ret) { dev_err(smi->dev, "unable to request irq: %d\n", ret); - return ret; + goto out_put_node; } smi->irqdomain = irq_domain_add_linear(intc, RTL8366RB_NUM_INTERRUPT, @@ -553,12 +554,15 @@ static int rtl8366rb_setup_cascaded_irq(struct realtek_smi *smi) smi); if (!smi->irqdomain) { dev_err(smi->dev, "failed to create IRQ domain\n"); - return -EINVAL; + ret = -EINVAL; + goto out_put_node; } for (i = 0; i < smi->num_ports; i++) irq_set_parent(irq_create_mapping(smi->irqdomain, i), irq); - return 0; +out_put_node: + of_node_put(intc); + return ret; } static int rtl8366rb_set_addr(struct realtek_smi *smi) -- 2.20.1