of: property: use new of_graph functions
authorKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Thu, 24 Oct 2024 02:20:11 +0000 (02:20 +0000)
committerRob Herring (Arm) <robh@kernel.org>
Thu, 24 Oct 2024 21:35:48 +0000 (16:35 -0500)
Current of_graph_get_next_endpoint() can be replaced by using
new of_graph_get_next_port().

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Link: https://lore.kernel.org/r/87iktib5t0.wl-kuninori.morimoto.gx@renesas.com
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
drivers/of/property.c

index ce0e081..519bf92 100644 (file)
@@ -717,13 +717,7 @@ struct device_node *of_graph_get_next_endpoint(const struct device_node *parent,
         * parent port node.
         */
        if (!prev) {
-               struct device_node *node __free(device_node) =
-                       of_get_child_by_name(parent, "ports");
-
-               if (node)
-                       parent = node;
-
-               port = of_get_child_by_name(parent, "port");
+               port = of_graph_get_next_port(parent, NULL);
                if (!port) {
                        pr_debug("graph: no port node found in %pOF\n", parent);
                        return NULL;
@@ -741,7 +735,7 @@ struct device_node *of_graph_get_next_endpoint(const struct device_node *parent,
                 * getting the next child. If the previous endpoint is NULL this
                 * will return the first child.
                 */
-               endpoint = of_get_next_child(port, prev);
+               endpoint = of_graph_get_next_port_endpoint(port, prev);
                if (endpoint) {
                        of_node_put(port);
                        return endpoint;
@@ -750,11 +744,9 @@ struct device_node *of_graph_get_next_endpoint(const struct device_node *parent,
                /* No more endpoints under this port, try the next one. */
                prev = NULL;
 
-               do {
-                       port = of_get_next_child(parent, port);
-                       if (!port)
-                               return NULL;
-               } while (!of_node_name_eq(port, "port"));
+               port = of_graph_get_next_port(parent, port);
+               if (!port)
+                       return NULL;
        }
 }
 EXPORT_SYMBOL(of_graph_get_next_endpoint);