net: dsa: use of_for_each_phandle
authorVivien Didelot <vivien.didelot@savoirfairelinux.com>
Mon, 6 Nov 2017 21:11:50 +0000 (16:11 -0500)
committerDavid S. Miller <davem@davemloft.net>
Thu, 9 Nov 2017 00:26:49 +0000 (09:26 +0900)
The OF code provides a of_for_each_phandle() helper to iterate over
phandles. Use it instead of arbitrary iterating ourselves over the list
of phandles hanging to the "link" property of the port's device node.

The of_phandle_iterator_next() helper calls of_node_put() itself on
it.node. Thus We must only do it ourselves if we break the loop.

Signed-off-by: Vivien Didelot <vivien.didelot@savoirfairelinux.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/dsa/dsa2.c

index 0f6f8c1..25ed412 100644 (file)
@@ -145,21 +145,18 @@ static int dsa_port_complete(struct dsa_switch_tree *dst,
                             struct dsa_port *port,
                             u32 src_port)
 {
-       struct device_node *link;
-       int index;
+       struct device_node *dn = port->dn;
+       struct of_phandle_iterator it;
        struct dsa_switch *dst_ds;
        struct dsa_port *link_dp;
+       int err;
 
-       for (index = 0;; index++) {
-               link = of_parse_phandle(port->dn, "link", index);
-               if (!link)
-                       break;
-
-               link_dp = dsa_tree_find_port_by_node(dst, link);
-               of_node_put(link);
-
-               if (!link_dp)
+       of_for_each_phandle(&it, err, dn, "link", NULL, 0) {
+               link_dp = dsa_tree_find_port_by_node(dst, it.node);
+               if (!link_dp) {
+                       of_node_put(it.node);
                        return 1;
+               }
 
                dst_ds = link_dp->ds;