From patchwork Tue Jun 28 10:46:10 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jon Hunter X-Patchwork-Id: 641475 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3rf2YQ5Q61z9sdb for ; Tue, 28 Jun 2016 20:46:50 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752464AbcF1KqS (ORCPT ); Tue, 28 Jun 2016 06:46:18 -0400 Received: from hqemgate15.nvidia.com ([216.228.121.64]:15300 "EHLO hqemgate15.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932162AbcF1KqR (ORCPT ); Tue, 28 Jun 2016 06:46:17 -0400 Received: from hqpgpgate101.nvidia.com (Not Verified[216.228.121.13]) by hqemgate15.nvidia.com id ; Tue, 28 Jun 2016 03:45:43 -0700 Received: from HQMAIL104.nvidia.com ([172.20.12.94]) by hqpgpgate101.nvidia.com (PGP Universal service); Tue, 28 Jun 2016 03:46:16 -0700 X-PGP-Universal: processed; by hqpgpgate101.nvidia.com on Tue, 28 Jun 2016 03:46:16 -0700 Received: from UKMAIL102.nvidia.com (10.26.138.15) by HQMAIL104.nvidia.com (172.18.146.11) with Microsoft SMTP Server (TLS) id 15.0.1130.7; Tue, 28 Jun 2016 10:46:16 +0000 Received: from [10.21.132.149] (10.21.132.149) by UKMAIL102.nvidia.com (10.26.138.15) with Microsoft SMTP Server (TLS) id 15.0.1130.7; Tue, 28 Jun 2016 10:46:12 +0000 Subject: Re: [PATCH 5/6] soc/tegra: pmc: Add missing of_node_put To: Stephen Warren , Thierry Reding , Alexandre Courbot References: <1467110308-22126-1-git-send-email-jonathanh@nvidia.com> <1467110308-22126-6-git-send-email-jonathanh@nvidia.com> CC: , From: Jon Hunter Message-ID: <57725572.7090802@nvidia.com> Date: Tue, 28 Jun 2016 11:46:10 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.8.0 MIME-Version: 1.0 In-Reply-To: <1467110308-22126-6-git-send-email-jonathanh@nvidia.com> X-Originating-IP: [10.21.132.149] X-ClientProxiedBy: UKMAIL102.nvidia.com (10.26.138.15) To UKMAIL102.nvidia.com (10.26.138.15) Sender: linux-tegra-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-tegra@vger.kernel.org On 28/06/16 11:38, Jon Hunter wrote: > Add missing of_node_put() in PMC early initialisation function. > > Signed-off-by: Jon Hunter > --- > drivers/soc/tegra/pmc.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c > index 483d54623ec5..e62acaef140a 100644 > --- a/drivers/soc/tegra/pmc.c > +++ b/drivers/soc/tegra/pmc.c > @@ -1572,6 +1572,8 @@ static int __init tegra_pmc_early_init(void) > value &= ~PMC_CNTRL_INTR_POLARITY; > > tegra_pmc_writel(value, PMC_CNTRL); > + > + of_node_put(np); > } > > return 0; Actually there are a couple other places we should put the node ... diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c index 483d54623ec5..48e1de2f7aeb 100644 --- a/drivers/soc/tegra/pmc.c +++ b/drivers/soc/tegra/pmc.c @@ -1540,6 +1540,7 @@ static int __init tegra_pmc_early_init(void) */ if (of_address_to_resource(np, 0, ®s) < 0) { pr_err("failed to get PMC registers\n"); + of_node_put(np); return -ENXIO; } } @@ -1547,6 +1548,7 @@ static int __init tegra_pmc_early_init(void) pmc->base = ioremap_nocache(regs.start, resource_size(®s)); if (!pmc->base) { pr_err("failed to map PMC registers\n"); + of_node_put(np); return -ENXIO; } @@ -1572,6 +1574,8 @@ static int __init tegra_pmc_early_init(void) value &= ~PMC_CNTRL_INTR_POLARITY; tegra_pmc_writel(value, PMC_CNTRL); + + of_node_put(np); } return 0;