powerpc: Rework btext_find_display to use of_stdout and device_type helpers
authorRob Herring <robh@kernel.org>
Fri, 16 Nov 2018 22:10:59 +0000 (16:10 -0600)
committerMichael Ellerman <mpe@ellerman.id.au>
Mon, 26 Nov 2018 11:33:37 +0000 (22:33 +1100)
Remove directly accessing device_node.type pointer and use the
accessors instead. This will eventually allow removing the type
pointer.

In the process, the of_stdout pointer can be used instead of finding
the stdout node again.

Signed-off-by: Rob Herring <robh@kernel.org>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
arch/powerpc/kernel/btext.c

index b4241ed1456ece186edc936aa0c04c8a0afeb409..6dfceaa820e42f99e682c4860fdbf06c5fb69cc1 100644 (file)
@@ -232,20 +232,12 @@ static int btext_initialize(struct device_node *np)
 
 int __init btext_find_display(int allow_nonstdout)
 {
-       const char *name;
-       struct device_node *np = NULL; 
+       struct device_node *np = of_stdout;
        int rc = -ENODEV;
 
-       name = of_get_property(of_chosen, "linux,stdout-path", NULL);
-       if (name != NULL) {
-               np = of_find_node_by_path(name);
-               if (np != NULL) {
-                       if (strcmp(np->type, "display") != 0) {
-                               printk("boot stdout isn't a display !\n");
-                               of_node_put(np);
-                               np = NULL;
-                       }
-               }
+       if (!of_node_is_type(np, "display")) {
+               printk("boot stdout isn't a display !\n");
+               np = NULL;
        }
        if (np)
                rc = btext_initialize(np);