[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <33fb6b04-9986-35ee-f4e2-0ebeb79c5e23@users.sourceforge.net>
Date: Wed, 20 Jul 2016 15:20:59 +0200
From: SF Markus Elfring <elfring@...rs.sourceforge.net>
To: linuxppc-dev@...ts.ozlabs.org,
Christophe Lombard <clombard@...ux.vnet.ibm.com>,
Ian Munsie <imunsie@....ibm.com>
Cc: LKML <linux-kernel@...r.kernel.org>,
kernel-janitors@...r.kernel.org,
Julia Lawall <julia.lawall@...6.fr>,
Michael Ellerman <mpe@...erman.id.au>
Subject: [PATCH] cxl: Delete an unnecessary check before the function call
"of_node_put"
From: Markus Elfring <elfring@...rs.sourceforge.net>
Date: Wed, 20 Jul 2016 15:10:32 +0200
The of_node_put() function tests whether its argument is NULL
and then returns immediately.
Thus the test around the call is not needed.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring <elfring@...rs.sourceforge.net>
---
drivers/misc/cxl/of.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/misc/cxl/of.c b/drivers/misc/cxl/of.c
index edc4583..333256a 100644
--- a/drivers/misc/cxl/of.c
+++ b/drivers/misc/cxl/of.c
@@ -490,8 +490,7 @@ int cxl_of_probe(struct platform_device *pdev)
adapter->slices = 0;
}
- if (afu_np)
- of_node_put(afu_np);
+ of_node_put(afu_np);
return 0;
}
--
2.9.2
Powered by blists - more mailing lists