From patchwork Mon Jul 4 06:55:57 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Peter Chen X-Patchwork-Id: 9211691 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 5E06B60752 for ; Mon, 4 Jul 2016 07:05:15 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 4D0AD285B4 for ; Mon, 4 Jul 2016 07:05:15 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 416172860D; Mon, 4 Jul 2016 07:05:15 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.2 required=2.0 tests=BAD_ENC_HEADER,BAYES_00, RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.9]) (using TLSv1.2 with cipher AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 0F12D285B4 for ; Mon, 4 Jul 2016 07:05:13 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.85_2 #1 (Red Hat Linux)) id 1bJxul-0002xX-Vm; Mon, 04 Jul 2016 07:03:32 +0000 Received: from mail-sn1nam01on0061.outbound.protection.outlook.com ([104.47.32.61] helo=NAM01-SN1-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.85_2 #1 (Red Hat Linux)) id 1bJxuj-0002vt-Jj for linux-arm-kernel@lists.infradead.org; Mon, 04 Jul 2016 07:03:30 +0000 Received: from BN3PR0301CA0027.namprd03.prod.outlook.com (10.160.180.165) by BL2PR03MB243.namprd03.prod.outlook.com (10.255.231.23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.528.16; Mon, 4 Jul 2016 07:03:07 +0000 Received: from BY2FFO11FD010.protection.gbl (2a01:111:f400:7c0c::153) by BN3PR0301CA0027.outlook.office365.com (2a01:111:e400:4000::37) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.528.16 via Frontend Transport; Mon, 4 Jul 2016 07:03:07 +0000 Authentication-Results: spf=fail (sender IP is 192.88.158.2) smtp.mailfrom=nxp.com; nxp.com; dkim=none (message not signed) header.d=none; nxp.com; dmarc=fail action=none header.from=nxp.com; nxp.com; dkim=none (message not signed) header.d=none; Received-SPF: Fail (protection.outlook.com: domain of nxp.com does not designate 192.88.158.2 as permitted sender) receiver=protection.outlook.com; client-ip=192.88.158.2; helo=az84smr01.freescale.net; Received: from az84smr01.freescale.net (192.88.158.2) by BY2FFO11FD010.mail.protection.outlook.com (10.1.14.74) with Microsoft SMTP Server (TLS) id 15.1.523.9 via Frontend Transport; Mon, 4 Jul 2016 07:03:07 +0000 Received: from shlinux2.ap.freescale.net (shlinux2.ap.freescale.net [10.192.224.44]) by az84smr01.freescale.net (8.14.3/8.14.0) with ESMTP id u647343x009256; Mon, 4 Jul 2016 00:03:05 -0700 From: Peter Chen To: Subject: [PATCH 1/1] firmware: arm_scpi: add missing of_node_put after calling of_parse_phandle Date: Mon, 4 Jul 2016 14:55:57 +0800 Message-ID: <1467615357-440-1-git-send-email-peter.chen@nxp.com> X-Mailer: git-send-email 1.9.1 X-EOPAttributedMessage: 0 X-Matching-Connectors: 131120893873742394; (91ab9b29-cfa4-454e-5278-08d120cd25b8); () X-Forefront-Antispam-Report: CIP:192.88.158.2; IPV:NLI; CTRY:US; EFV:NLI; SFV:NSPM; SFS:(10009020)(6009001)(7916002)(2980300002)(1109001)(1110001)(339900001)(199003)(189002)(87936001)(11100500001)(50466002)(305945005)(68736007)(50226002)(8666005)(7846002)(48376002)(8936002)(86362001)(104016004)(106466001)(85426001)(33646002)(6806005)(19580405001)(8676002)(69596002)(47776003)(81156014)(97736004)(229853001)(4326007)(586003)(2906002)(2351001)(189998001)(356003)(105606002)(92566002)(77096005)(50986999)(110136002)(5003940100001)(36756003)(81166006)(19580395003)(7059030); DIR:OUT; SFP:1101; SCL:1; SRVR:BL2PR03MB243; H:az84smr01.freescale.net; FPR:; SPF:Fail; PTR:InfoDomainNonexistent; A:1; MX:1; LANG:en; X-Microsoft-Exchange-Diagnostics: 1; BY2FFO11FD010; 1:v3G75uQ8yuH/iZi1Z507jGR17RBusgs6jPo2sLRTes0eMc2Vp31xHAMMj30BI9pwZ5GvEgOla2fYYSXGEWVZXU4KJ3AOastB+Uxbk8UpL53cofJfAwo6xCTWdPtgcdI5gEQM7QgaY1ysZaXx7L4kjLrOFtuupbQ6bgUPe2+xmKAYEy4rkYDngADGYnC1CTqht356pC1Ad1jrNNIhG2vvEuOPC51z4FldUkHa37p7bY1cd49O6KFl6v2BdAMhF6Llwk0rh+FJngIpyG5UhkqD4eZHC0PSdmhD0ecqVgiQnpyuyMf3mvyLykzkg5ImQTjG/9wuHR0kMJo8N9pguk/YimeOwwTXQK10U/8gtg+UeyoMxfp29i6j+JHgWSU9wzk1rpLEi0HrKWTXrAMM4/MbqXIHtwy4ZvneU+DZo+CEBBQzPuw1Ro3dybxAj9u6WFNRPfSU8RxmVnwz9wMshS7roWo/NQoroYggUjvCHDpZ10XDBaU0LV1BxTIsUrq1LoCf2ToWKGWtFdGQZlLUGLm5/bIQOVAahmUjYKHsecSp1jrW7Fu+U3bMzJtS41rqTiFu4u2w8AfHhgS4Gkf7+pEdhV3gBnjIUONBLOlhPWnj8DHU1GykeL+6csqTeJGnCxZ7LUOMuTSnYHSDweYN32DXZSBubZ0t63VDFlFA+xYtddwhRtnb/jDjLgv5LhXqHPE9pw8oV58h0Med0DP1ZcDMl52Ras/2XR2/Ivz9b8NnxI1yzwQJgQ86bvH+weEXaQWu MIME-Version: 1.0 X-MS-Office365-Filtering-Correlation-Id: 97d98f2d-788f-4d44-d73a-08d3a3d9405d X-Microsoft-Exchange-Diagnostics: 1; BL2PR03MB243; 2:6jiWZQsgUZxsDE9MZ7QLEsvfCyMnjkstzoKTDGczYuBm/F4S1sAOEaDZZaIaFdO8biRiQSL40YaaR1sKXcyB2XkPUT3qW4jZv/oJNW7HMWqilMX4tjLYdPi7h8LxGSEnQ4/JFlvYQyZx/joix6BZuOPQv7JI8qiInYwmkhQPYyADR1rOoNvIyZWtSkDOyBXR; 3:CMzXa1RBfmeMbBlrt1lhd5eaBTNbMjehe2nTe3QOamdwiYciQjJH1UrJBxfeC0tH2NCTifzCeXA20L0hCcL6ms3+X8EgvllNdrhAjOA5kxQJQodIDQsBsEJ3ROAvmV7AxwZGSeHVplRWz9atZmVQ/IU7fT0mNKoREH320TXkMGd9VCtXtFSSnvejUjA2jWeVk6Gg3ZiwevDKsH2mVJ0O+hxh2IUoiJsKEjn7Ahd0S+s=; 25:j9sO4MH1UbSvK30KJo6ge3BnFaqFyVgw1WvbOFyCV+eQk3KFZY9YZPW03hAswWpFjlFqwwqjgGUpKhJODuuZkiwn643YHykDX28T+GJdkrGS1hyexu5h04Acl9aDPn+aoDgN0EH6hBj9QKb4OkADi7HM0G+nl+ktIfQv0vkF3yr9H0HKXW8w2Js4egF/VtohfQT96v0W1R/h7JQWK25Ba8BiksoVxIrXNYTm/hFWOxuweCFXFy6megrWzcaNDfYHlm6s7JyOeGRQIfeUwqTzWxy6kULkx3wSw14lbj3OJVL0cbFqHJD6PAdS8aW8agCNoRNmuOk1TUMo2x7Of0ldhwVZOYJV3qhuc95ryhtbMD1+01Jfmr7WjAjP3WfPV7UxeO4hSROtZQTUu2fDfascD7mdsDTNJnhhqzMZ5TZsPss= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BL2PR03MB243; X-Microsoft-Exchange-Diagnostics: 1; BL2PR03MB243; 31:SAeubBbby3IeIptbGea8Ytx8ov3q+BKU6elhKJl2ZsHho0rTyrzt+3hGURiC5DtshKlUPaTXxTGdBN+DZS+GlvD086WA2gMCA6J8xxgk+nz58krv2U7uoDTxpgSKj6mSnBZvHI8RHSthO5v9+nPJZvcsAi1CCNAz0Q1utNPbdPePFg4hEDu/LT/XODQnng8UEF231Rhz72jFxYlDrgYhxQ==; 4:VuAf0T9npoqpvodMDE+n4Yey3TGRX26WcNkgy70CknRO4FU0xzt0TMK5tRYqd4wf0Iqc5fxO1eF71uYb7c7MPFeCwAdH0kHHmLqXnOW+nWy8IZTwGoQ2XmzWS1hqKnz0zmH6oV0Sb2bGDDCr91t7EAKKLXnp0hHWb8/859JDR+xrh1wUFE64g0sGWi/Z7wldFK9qbw9Hr1b+/PBvSm2REhTgmYnCMwhGyudwHTqqixJfQjVo7GXwY2q2AX/trmb+dYWusnOOgfyxkaC/Tr9id7S4yOVAbOho/X+5L/6PNyf1ktAiMQE7SsbTA4M0q0W1N3BXkjEgKK1RJ6105f+E3PFrjYPy6HeEPBGte+QZxjRs9aLJ0ujoZqYr7LUdi+dxIOzFqh5rb9qzluGnWWUF+g7tg4bsbjkaEhy1wcCcnKqc7fxdGzGktePJ0cLL2Dx1bzsEzhIAB1WdoI3BynFVmncO/VGylQfUXqIL/gzgInQgAhDyvOKUrIH6eEI4i0Maz0Js7EXTkAzpNHEGcxvuTQ== X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(185117386973197); X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(601004)(2401047)(13018025)(13015025)(13017025)(13024025)(13023025)(5005006)(8121501046)(3002001)(10201501046)(6055026); SRVR:BL2PR03MB243; BCL:0; PCL:0; RULEID:(400006); SRVR:BL2PR03MB243; X-Forefront-PRVS: 0993689CD1 X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; BL2PR03MB243; 23:urbprTSkWwDeAszQcCq5xxwxBlu9BspT92bTTASDbU?= =?us-ascii?Q?p65bFwWJRxl9/hNEDW75OAeZsLXEWMT2ZwD/KUeV9zNLtqjtIhi+1sycuRXU?= =?us-ascii?Q?MOrhWBHFRarXK+Tiogj64c4exp5ZxdVeN5vLhLyeO0gxFXiOJsHv++BkjHf+?= =?us-ascii?Q?oko1EawXgjOtSfDID5WLiQxwmCCsbNW5jUQWyQsgS+fk7YcQzSCaQQFpIAFk?= =?us-ascii?Q?A5jj1jNf0ngIE/P/e9VyKcb3xZQnEhBe5hwpDEWAyZnr02tz3pxnZBpfGv0S?= =?us-ascii?Q?D+b3O20/zqCP8S6gJPFVZZgBIj+jWKo4NVKUDALNvZe/KzmYhfj2XNsm3oeX?= =?us-ascii?Q?GOJbBgccLBh+ZjeDFmd5xcgdtW6TTQeuHCqKDd0YWlmX8pyhukSJ2tz/+10M?= =?us-ascii?Q?+IL6o4Xv/x/yw4Nnnk0HivqMiv+zh9StlSrGLnsjVWmTSLLnTJi2CrzsGlGl?= =?us-ascii?Q?rII3SaTFZAhdzcui/5rs6CuvSU33Gxfnmuk1C1Qvk11sKi0ItMAsyiBMQoHR?= =?us-ascii?Q?9yD5CrEykZqkps9UU9S/IFIZa4B6/MoALTfxzO364NLZOOExnMLPzHJtLQPl?= =?us-ascii?Q?Pcvzo4lX79Tsu+bo2ZsOKvhhmWpB5pv8+dz26SOovGJt8TU6SfVIrFirhAPB?= =?us-ascii?Q?Y0sMhiZispHI9gnaXwawUIUOBCqPMkeHPTA3lAhB0UHXsduySCHYqQIv37Re?= =?us-ascii?Q?fXrLaafbezQGIUlCYd0lnX9fPJwGiytxaKwxnaG0DfELN5VhCdP9K+qnnLxO?= =?us-ascii?Q?HAhrTCAK3OS1H655QOojof7dQU/YMwtLXgrvvin2KN5lZJU49YQI/4/yD17O?= =?us-ascii?Q?Rwzw5betYt5XtZFef5vKm3awBF6x9szxCvZN8+tcdhtw+EzmEjQAph3p9X2A?= =?us-ascii?Q?ehwjka/pJe/M5HjME2WL4vLoUG1VUFmS5sdPJcCLD1kIPFD9t7p6VvCPYrFw?= =?us-ascii?Q?J5IIrsLYcQSVez00TAUWHJm8fb+gxwP5ehWX86Z3KAoqVFhS2aJ+2zM9hNN+?= =?us-ascii?Q?YyYO/OZ1ZNaMJKHDc5zfyGjRAqXDz7j2YpZNSe9XaQDxgn0U3Kp0SYjyHlCT?= =?us-ascii?Q?KfE1g2s6Vh32XSFNnlRm75hTOiALfXzBvMZlmJgvL8zahf/oOmSlnNDidV/Z?= =?us-ascii?Q?jL1GsTTU8=3D?= X-Microsoft-Exchange-Diagnostics: 1; BL2PR03MB243; 6:/97JadAZR89DjGw+/thKFDYwkPtO8rImXxYttYdupr9slIQ2g2bWQAcSQLvepDqc4pOKWF7FS3PI+V2uIhSaBk9ZhQmViPkqC8NgvmgkDsICns3DEUEnFohEzxIushfMIrNAhWY2NME0gL7pYqGDbhqCG/hvsWtoWo4pmuKbV0yry7ktgkcdD3SCi3nNoy51I+lW+aXRvkfGAWYxCf+Q2rJTnjXHuxP211tB84DJOyMh+QQy+eADNbwluwLrjdI4K5Xzi+THK6fBZJLX2K/JLxe5ojneHWdP5JvNHabJ7yM=; 5:6VAFrWENce9JzFnLDeTKcbyw2RZKwv7HpYCPgrxKAYNVV5aok81RREI/S+PPQ0TSPi4sc/89fittft9wDFkxzPbcc+lXa+wOiNycpqxfD6SuAC5qUYl3l9qGU0Snmai2taLf+I+14QC4g2XWKBRqLdGEhDriCGRbLPkMGqANhpo=; 24:TcFXED+tL098TQ9Vos13erMo9H3/iQJc7XgR110831iBWbygPXt+iRJ10pFUt3WuEFa9BHRssm0769Xr9xbqFed98P9z8q9XWwdMBJybk8k=; 7:lqwEh2NwzytOCQKk0aJFY9535nla0G1UlS6sUdGqQC7mcwEU0eURjNkG/ykKOnRxJKR5Nbq7JR6fu8F9rCcSNhMSQd+IRhUU/xLKCjT56h6fwip+qybRn1+dPbV82yO/23HSducoRAEsPPgigp6SUMqTIMbhAHdWaMtshrJasUj4EPLZgS3dxGc5VtwJyTw8lL1OBNv9cX7MPBg3q+rduqQokS9fggphRoNKYNoPOnxYD9BKmEhiHnDrd9obbjoJ SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Jul 2016 07:03:07.1402 (UTC) X-MS-Exchange-CrossTenant-Id: 5afe0b00-7697-4969-b663-5eab37d5f47e X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=5afe0b00-7697-4969-b663-5eab37d5f47e; Ip=[192.88.158.2]; Helo=[az84smr01.freescale.net] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL2PR03MB243 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20160704_000329_781003_F11A5586 X-CRM114-Status: GOOD ( 11.99 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Chen , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP of_node_put needs to be called when the device node which is got from of_parse_phandle has finished using. Besides, of_address_to_resource always returns -EINVAL for error, delete the assignment for ret. Signed-off-by: Peter Chen --- drivers/firmware/arm_scpi.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/firmware/arm_scpi.c b/drivers/firmware/arm_scpi.c index 7e3e595..fbf10d1 100644 --- a/drivers/firmware/arm_scpi.c +++ b/drivers/firmware/arm_scpi.c @@ -683,9 +683,10 @@ static int scpi_probe(struct platform_device *pdev) struct mbox_client *cl = &pchan->cl; struct device_node *shmem = of_parse_phandle(np, "shmem", idx); - if (of_address_to_resource(shmem, 0, &res)) { + ret = of_address_to_resource(shmem, 0, &res); + of_node_put(shmem); + if (ret) { dev_err(dev, "failed to get SCPI payload mem resource\n"); - ret = -EINVAL; goto err; }