of_node_put and put_device has taken the null pointer check into account.
So it is safe to remove the duplicated check.

Signed-off-by: zhong jiang <zhongji...@huawei.com>
---
 drivers/hwtracing/coresight/of_coresight.c | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/drivers/hwtracing/coresight/of_coresight.c 
b/drivers/hwtracing/coresight/of_coresight.c
index da71c97..89092f8 100644
--- a/drivers/hwtracing/coresight/of_coresight.c
+++ b/drivers/hwtracing/coresight/of_coresight.c
@@ -219,12 +219,9 @@ static int of_coresight_parse_endpoint(struct device *dev,
                ret = 1;
        } while (0);
 
-       if (rparent)
-               of_node_put(rparent);
-       if (rep)
-               of_node_put(rep);
-       if (rdev)
-               put_device(rdev);
+       of_node_put(rparent);
+       of_node_put(rep);
+       put_device(rdev);
 
        return ret;
 }
-- 
1.7.12.4

Reply via email to