of: overlay: Make node skipping in init_overlay_changeset() clearer

Make it more clear that nodes without "__overlay__" subnodes are
skipped, by reverting the logic and using continue.
This also reduces indentation level.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Rob Herring <robh@kernel.org>

1 files changed, 11 insertions, 10 deletions

diff --git a/drivers/of/overlay.c b/drivers/of/overlay.c
index 83bb2edfc65c..3981b7da4fa9 100644
--- a/drivers/of/overlay.c
+++ b/drivers/of/overlay.c
@@ -573,18 +573,19 @@ static int init_overlay_changeset(struct overlay_changeset *ovcs,
         cnt = 0;
         for_each_child_of_node(tree, node) {
                 overlay_node = of_get_child_by_name(node, "__overlay__");
-                if (overlay_node) {
-                        fragment = &fragments[cnt];
-                        fragment->overlay = overlay_node;
-                        fragment->target = find_target_node(node);
-                        if (!fragment->target) {
-                                of_node_put(fragment->overlay);
-                                ret = -EINVAL;
-                                goto err_free_fragments;
-                        }
+                if (!overlay_node)
+                        continue;
 
-                        cnt++;
+                fragment = &fragments[cnt];
+                fragment->overlay = overlay_node;
+                fragment->target = find_target_node(node);
+                if (!fragment->target) {
+                        of_node_put(fragment->overlay);
+                        ret = -EINVAL;
+                        goto err_free_fragments;
                 }
+
+                cnt++;
         }
 
         /*