arm64: make dt_scan_depth1_nodes more readable
authorMark Rutland <mark.rutland@arm.com>
Mon, 25 Apr 2016 10:25:11 +0000 (11:25 +0100)
committerWill Deacon <will.deacon@arm.com>
Mon, 25 Apr 2016 12:54:15 +0000 (13:54 +0100)
Improve the readability of dt_scan_depth1_nodes by removing the nested
conditionals.

Signed-off-by: Mark Rutland <mark.rutland@arm.com>
Signed-off-by: Stefano Stabellini <sstabellini@kernel.org>
Signed-off-by: Will Deacon <will.deacon@arm.com>
arch/arm64/kernel/acpi.c

index 57ee31745af5ac3dcdd72501ac1452a7e51fde56..6884c7678453170f0da0cab583794072c9f34443 100644 (file)
@@ -66,17 +66,24 @@ static int __init dt_scan_depth1_nodes(unsigned long node,
                                       void *data)
 {
        /*
-        * Return 1 as soon as we encounter a node at depth 1 that is
-        * not the /chosen node, or /hypervisor node with compatible
-        * string "xen,xen".
+        * Ignore anything not directly under the root node; we'll
+        * catch its parent instead.
         */
-       if (depth == 1 && (strcmp(uname, "chosen") != 0)) {
-               if (strcmp(uname, "hypervisor") != 0 ||
-                   !of_flat_dt_is_compatible(node, "xen,xen"))
-                       return 1;
-       }
+       if (depth != 1)
+               return 0;
 
-       return 0;
+       if (strcmp(uname, "chosen") == 0)
+               return 0;
+
+       if (strcmp(uname, "hypervisor") == 0 &&
+           of_flat_dt_is_compatible(node, "xen,xen"))
+               return 0;
+
+       /*
+        * This node at depth 1 is neither a chosen node nor a xen node,
+        * which we do not expect.
+        */
+       return 1;
 }
 
 /*