for_each_child_of_node can help to iterate through the device_node,
and we don't need to use while loop. No functional change with this
conversion.

Signed-off-by: Zhang Zekun <zhangzeku...@huawei.com>
---
 arch/powerpc/platforms/pseries/dlpar.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/arch/powerpc/platforms/pseries/dlpar.c 
b/arch/powerpc/platforms/pseries/dlpar.c
index 47f8eabd1bee..ee47ed21b99d 100644
--- a/arch/powerpc/platforms/pseries/dlpar.c
+++ b/arch/powerpc/platforms/pseries/dlpar.c
@@ -250,11 +250,8 @@ int dlpar_detach_node(struct device_node *dn)
        struct device_node *child;
        int rc;
 
-       child = of_get_next_child(dn, NULL);
-       while (child) {
+       for_each_child_of_node(dn, child)
                dlpar_detach_node(child);
-               child = of_get_next_child(dn, child);
-       }
 
        rc = of_detach_node(dn);
        if (rc)
-- 
2.17.1


Reply via email to