Hi Rob,

Today's linux-next merge of the dt-rh tree got a conflict in
drivers/of/base.c include/linux/of.h between commit 3296193d1421 ("dt:
introduce for_each_available_child_of_node, of_get_next_available_child")
from Linus' tree and commit 9c19761a7ecd ("dt: introduce
of_get_child_by_name to get child node by name") from the dt-rh tree.

I fixed it up (see below) and can carry the fix as necessary (no action
is required).

-- 
Cheers,
Stephen Rothwell                    s...@canb.auug.org.au

diff --cc drivers/of/base.c
index 4a8d46f,e2e8136..0000000
--- a/drivers/of/base.c
+++ b/drivers/of/base.c
@@@ -364,33 -364,29 +364,56 @@@ struct device_node *of_get_next_child(c
  EXPORT_SYMBOL(of_get_next_child);
  
  /**
 + *    of_get_next_available_child - Find the next available child node
 + *    @node:  parent node
 + *    @prev:  previous child of the parent node, or NULL to get first
 + *
 + *      This function is like of_get_next_child(), except that it
 + *      automatically skips any disabled nodes (i.e. status = "disabled").
 + */
 +struct device_node *of_get_next_available_child(const struct device_node 
*node,
 +      struct device_node *prev)
 +{
 +      struct device_node *next;
 +
 +      read_lock(&devtree_lock);
 +      next = prev ? prev->sibling : node->child;
 +      for (; next; next = next->sibling) {
 +              if (!of_device_is_available(next))
 +                      continue;
 +              if (of_node_get(next))
 +                      break;
 +      }
 +      of_node_put(prev);
 +      read_unlock(&devtree_lock);
 +      return next;
 +}
 +EXPORT_SYMBOL(of_get_next_available_child);
 +
 +/**
+  *    of_get_child_by_name - Find the child node by name for a given parent
+  *    @node:  parent node
+  *    @name:  child name to look for.
+  *
+  *      This function looks for child node for given matching name
+  *
+  *    Returns a node pointer if found, with refcount incremented, use
+  *    of_node_put() on it when done.
+  *    Returns NULL if node is not found.
+  */
+ struct device_node *of_get_child_by_name(const struct device_node *node,
+                               const char *name)
+ {
+       struct device_node *child;
+ 
+       for_each_child_of_node(node, child)
+               if (child->name && (of_node_cmp(child->name, name) == 0))
+                       break;
+       return child;
+ }
+ EXPORT_SYMBOL(of_get_child_by_name);
+ 
+ /**
   *    of_find_node_by_path - Find a node matching a full OF path
   *    @path:  The full path to match
   *
diff --cc include/linux/of.h
index 5c7a158,fabb524..0000000
--- a/include/linux/of.h
+++ b/include/linux/of.h
@@@ -190,9 -190,8 +190,11 @@@ extern struct device_node *of_get_paren
  extern struct device_node *of_get_next_parent(struct device_node *node);
  extern struct device_node *of_get_next_child(const struct device_node *node,
                                             struct device_node *prev);
 +extern struct device_node *of_get_next_available_child(
 +      const struct device_node *node, struct device_node *prev);
 +
+ extern struct device_node *of_get_child_by_name(const struct device_node 
*node,
+                                       const char *name);
  #define for_each_child_of_node(parent, child) \
        for (child = of_get_next_child(parent, NULL); child != NULL; \
             child = of_get_next_child(parent, child))

Attachment: pgpW37k9sBScn.pgp
Description: PGP signature

Reply via email to